We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
basePath()
match
3.2
to be able to boot the app
it didn't boot the app
https://github.com/roots/acorn/blob/main/src/Roots/Acorn/Bootloader.php#L300-L319
Create an app without any of the folowing:
$_ENV['APP_BASE_PATH'], constant('ACORN_BASEPATH'), get_theme_file_path('composer.json') get_theme_file_path('app')
It's supposed to get the path from this match branch:
$vendor_path = (new Filesystem())->closest(dirname(__DIR__, 4), 'composer.json')) => dirname($vendor_path
However, it doesn't because match expression requires boolean value in, and $vendor_path is a string. As a result, it falls back to
$vendor_path
default => dirname(__DIR__, 3)
See: https://www.php.net/manual/en/control-structures.match.php
A match arm compares values strictly (===) instead of loosely as the switch statement does.
No response
Yes
The text was updated successfully, but these errors were encountered:
3143aa7
Thanks for the heads up! 🙏
Sorry, something went wrong.
No branches or pull requests
Version
3.2
What did you expect to happen?
to be able to boot the app
What actually happens?
it didn't boot the app
Steps to reproduce
https://github.com/roots/acorn/blob/main/src/Roots/Acorn/Bootloader.php#L300-L319
Create an app without any of the folowing:
It's supposed to get the path from this match branch:
However, it doesn't because match expression requires boolean value in, and
$vendor_path
is a string. As a result, it falls back toSee: https://www.php.net/manual/en/control-structures.match.php
System info
No response
Log output
No response
Please confirm this isn't a support request.
Yes
The text was updated successfully, but these errors were encountered: