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

Issue with Laravel 10.17 #1529

Open
tuxudo opened this issue Aug 4, 2023 · 5 comments
Open

Issue with Laravel 10.17 #1529

tuxudo opened this issue Aug 4, 2023 · 5 comments

Comments

@tuxudo
Copy link
Contributor

tuxudo commented Aug 4, 2023

Starting with Laravel 10.17, there is an issue when calling php please migrate. Fix is to version lock illuminate/console, illuminate/database, illuminate/events, and illuminate/filesystem to 10.16 until bug is fixed upstream.

Only affects MunkiReport 5.8 with PHP 8.

Error is: Fatal error: Uncaught Error: Call to undefined method Illuminate\Container\Container::runningUnitTests() in /var/munkireport/vendor/illuminate/console/Concerns/ConfiguresPrompts.php:27 Stack trace: #0 /var/munkireport/vendor/illuminate/console/Command.php(177): Illuminate\Console\Command->configurePrompts(Object(Symfony\Component\Console\Input\ArgvInput)) #1 /var/munkireport/vendor/symfony/console/Application.php(1063): Illuminate\Console\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput)) #2 /var/munkireport/vendor/symfony/console/Application.php(320): Symfony\Component\Console\Application->doRunCommand(Object(App\Console\Commands\MigrateCommand), Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput)) #3 /var/munkireport/vendor/symfony/console/Application.php(174): Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput)) #4 /var/munkireport/please(39): Symfony\Component\Console\Application->run() #5 {main} thrown in /var/munkireport/vendor/illuminate/console/Concerns/ConfiguresPrompts.php on line 27

tuxudo added a commit to tuxudo/munkireport-php that referenced this issue Aug 4, 2023
sphen13 added a commit to sphen13/munkireport-php that referenced this issue Aug 4, 2023
@sphen13
Copy link
Contributor

sphen13 commented Aug 4, 2023

in discussing this further with @tuxudo we saw that other illuminate dependencies were installing 10.17+. to resolve this in case the version mismatch causes issues i expanded the change with this commit sphen13@3c67fe3 - i could PR That if desired or @tuxudo could decide to update this one perhaps.

@tuxudo
Copy link
Contributor Author

tuxudo commented Aug 4, 2023

This also breaks Module Marketplace as discovered by @jby

@SamdyPrum
Copy link

Hello All!

I got a problem with run ./please migrate


PHP Fatal error:  Uncaught Error: Call to undefined method Illuminate\Container\Container::runningUnitTests() in /usr/share/nginx/html/report/vendor/illuminate/console/Concerns/ConfiguresPrompts.php:30
Stack trace:
#0 /usr/share/nginx/html/report/vendor/illuminate/console/Command.php(177): Illuminate\Console\Command->configurePrompts()
#1 /usr/share/nginx/html/report/vendor/symfony/console/Application.php(1063): Illuminate\Console\Command->run()
#2 /usr/share/nginx/html/report/vendor/symfony/console/Application.php(320): Symfony\Component\Console\Application->doRunCommand()
#3 /usr/share/nginx/html/report/vendor/symfony/console/Application.php(174): Symfony\Component\Console\Application->doRun()
#4 /usr/share/nginx/html/report/please(39): Symfony\Component\Console\Application->run()
#5 {main}
  thrown in /usr/share/nginx/html/report/vendor/illuminate/console/Concerns/ConfiguresPrompts.php on line 30

If anyone know how to fix it let know, thank you!

@sphen13
Copy link
Contributor

sphen13 commented Nov 6, 2023

@SamdyPrum we limited laravel to 10.16 to address this for now - was pushed via this commit - are you up to date? #1531

@SamdyPrum
Copy link

@sphen13 It's worked
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants