-
Notifications
You must be signed in to change notification settings - Fork 888
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixing installation with dbal (#552) and PHP 7.4 (#554) #567
base: master
Are you sure you want to change the base?
Conversation
Previous versions aren't supported anymore
I've also juste updated your travis file so the tests goes green on PHP |
Hi, installed php 7.4 and I got installation error. Changed doctrine/dbal to < 2.10 and php to 7.3 and no problem with installation. @Tchekda Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR. I've just got this issue during installation.
Maybe another day I will take time to update the whole project to latest versions of laravel and dbal, but for now this fix is working like a charm |
Same issue and you PR does fix the case, Thanks! |
@Tchekda Solved. Thank you. |
Thank you so much @Tchekda. Your modifications made the installation to work. Thanks for the clarification and your time. |
As said above, I solved my problem, thank you |
During installation I encountered some problems due librairy updates that broke compatibility.
doctrine/dbal
not to use higher version than2.10.0.
by setting the compatibility at<2.10.0
: "Whoops, looks like something went wrong." when finishing setup #552 : Migrating a varchar to a bigint causes invalid charset instructions to appear laravel/framework#305397.3
because the is a backwards incompatible change in7.4', so I've set
<7.5` : PHP7.4: ErrorException: Trying to access array offset on value of type int [Solution] #554 https://stackoverflow.com/a/60413095Feel free to tell me what to change to make this PR accepted.