Contentify: Getting Error while installing Contentify 2.4

[GaneshKandu]

I am Getting Error while installing Contentify 2.4

Getting this error on button click of database fill up step

Contentify 2.4 trying to get database details for table creation and insertion from database.ini which is field with default value

you should update database.ini first before table creation and insertion while install so that Contentify 2.2 can connect with right database details instead of default value

Laravel Log --> laravel.log

[GaneshKandu]

Contentify 2.4

Laravel Log --> laravel.log

[chriskonnertz]

Hello,

seems like my comment disappeared...

So, again: Thank you for creating this issue. So the database credentials in the storage/app/databse.ini are correct?

[GaneshKandu]

yes, credentials are correct in storage/app/databse.ini
but my database credentials are different from default value in storage/app/databse.ini

; Auto-generated file with database connection settings.
; See config/database.php for more settings.

host = 127.0.0.1
database = contentify2
username = root
password = 

[chriskonnertz]

What part is different? Do you have a special port? You can change the port in this file: https://github.com/Contentify/Contentify/blob/v2.4/config/database.php#L45

[GaneshKandu]

No i don't have special port i am using default port 3306

[GaneshKandu]

Did you able to reproduce ? i got solution of it

[chriskonnertz]

@GaneshKandu What solution did you find? :)

[GaneshKandu]

https://github.com/Contentify/Contentify/pull/383
Hi @chriskonnertz Please look at the changed i made

[chriskonnertz]

Hi, yes I will take a closer look tomorrow. I believe I understand the problem now, I have to look at it from different angles and do some testing but you might have a point there...

[chriskonnertz]

Hello,

finally had time to write a bug fix. I hope this fixes excatly this problem.

In your pull request you were right about what the problem was, but I was also right that there had to be a redirect (okay, acutally I've added a new step instead of a redirect), so I kinda combined our solutions. c9d58d4

Thank you for your contribution, it helped a lot (even though it did not get merged)!