Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix TypeError with recent version of Composer #17

Merged
merged 1 commit into from
Apr 15, 2020
Merged

Fix TypeError with recent version of Composer #17

merged 1 commit into from
Apr 15, 2020

Conversation

villfa
Copy link
Contributor

@villfa villfa commented Apr 7, 2020

This PR fixes the following error that I got with composer 1.10.1:

PHP Fatal error:  Uncaught TypeError: Return value of "ScriptsDev\DevScriptProxyCommand::execute()" must be of the type int, "NULL" returned. in /.../vendor/symfony/console/Command/Command.php:258
Stack trace:
#0 /.../vendor/symfony/console/Application.php(912): Symfony\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#1 /.../vendor/symfony/console/Application.php(264): Symfony\Component\Console\Application->doRunCommand(Object(ScriptsDev\DevScriptProxyCommand), Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#2 /.../vendor/composer/composer/src/Composer/Console/Application.php(281): Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Comp in /.../vendor/symfony/console/Command/Command.php on line 258

See also:
https://github.com/symfony/console/blob/5fa1caadc8cdaa17bcfb25219f3b53fe294a9935/Command/Command.php#L255-L259

@neronmoon neronmoon merged commit 433a61a into neronmoon:master Apr 15, 2020
@neronmoon
Copy link
Owner

Thank you!

@villfa villfa deleted the patch-2 branch April 19, 2020 18:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants