-
Notifications
You must be signed in to change notification settings - Fork 113
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 required PHP version to 7.1 #706
Conversation
M0rgan01
commented
May 23, 2024
•
edited by Quetzacoalt91
Loading
edited by Quetzacoalt91
Questions | Answers |
---|---|
Description? | Switching to php 7.1. Added typing where easily possible. Addition of an alert on installation and on display of the configuration page if the PHP version is not >= 7.1 |
Type? | improvement |
BC breaks? | yes |
Deprecations? | no |
Fixed ticket? | Fixes PrestaShop/PrestaShop#27087 |
Sponsor company | - |
How to test? | Check an installation with a PHP version lower than 7.1 to see the alert. View the configuration page with a PHP version lower than 7.1 to see the alert. The upgrade should work the same way with a PHP version >= 7.1 |
autoupgrade.php
Outdated
$autoloadPath = __DIR__ . '/../../vendor/autoload.php'; | ||
if (file_exists($autoloadPath)) { | ||
require_once $autoloadPath; | ||
} |
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.
I recommend moving this to a place where you have checked the current PHP version is compatible. For instance in the __construct
after a call to VersionCheck::_isActualPHPVersionCompatible()
.
3baf6d7
to
d4721af
Compare
5065237
to
ac2fa4f
Compare
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.
Hi @M0rgan01
Thank you for your PR, I tested it and it seems to works as you can see :
recording.231.webm
Tested with php 5.6, 7.0 and 7.1
Tested from :
1.7.1.0 to 1.7.8.11
1.7.8.11 to 8.1.6
8.0.5 to 8.1.6
8.1.5 to 8.1.6
8.1.6 to 9.0.0
8.0.5 to 9.0.0
8.1.5 to 9.0.0
Because the PR seems to works as expected, It's QA ✔️
Thank you