Skip to content
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

Update InstallTool.php #84

Closed
wants to merge 1 commit into from
Closed

Update InstallTool.php #84

wants to merge 1 commit into from

Conversation

Wusch
Copy link

@Wusch Wusch commented Jan 6, 2018

If $vendor is not set at line 222 because SELECT @@version as Version only returns a version number you get an error:

An exception occurred. {"exception":"[object]
(Symfony\Component\Debug\Exception\FatalThrowableError(code: 0): Type error: strtolower() expects parameter 1 to be string, null given at /vendor/contao/installation-bundle/src/InstallTool.php:274)

If $vendor is not set at line 222 because "SELECT @@Version as Version" only returns a version number you get an error:
An exception occurred. {"exception":"[object] (Symfony\\Component\\Debug\\Exception\\FatalThrowableError(code: 0): Type error: strtolower() expects parameter 1 to be string, null given at /vendor/contao/installation-bundle/src/InstallTool.php:274)
@leofeyer
Copy link
Member

leofeyer commented Jan 6, 2018

What exactly does SELECT @@version as Version return on your system?

@Wusch
Copy link
Author

Wusch commented Jan 7, 2018

It returns 5.7.20

@leofeyer
Copy link
Member

Fixed in 0ce2e6d.

@leofeyer leofeyer closed this Jan 11, 2018
@leofeyer leofeyer modified the milestones: 4.5.2, 4.5 May 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants