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

Remove 1.6 related code #616

Merged
merged 1 commit into from Nov 23, 2023
Merged

Remove 1.6 related code #616

merged 1 commit into from Nov 23, 2023

Conversation

Hlavtox
Copy link
Contributor

@Hlavtox Hlavtox commented Aug 24, 2023

Questions Answers
Description? Removes old 1.6 related code and upgrade scripts. Fixed return value when enableTheme failed - issue was hidden behind mixed return value.
Type? refactor
BC breaks? yes
Deprecations? no
Fixed ticket?
Sponsor company
How to test? Run upgrade from some old version, like 1.7.6, should go without issues.

Copy link
Contributor

@matks matks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hlavtox You made the CI cry 😆

@darkWolf-PR
Copy link

What about to ... AT LEAST ... before removing 1.6 stuff, release a downloadable 4.14.3 version with this line:

return version_compare($this->getModuleVersion(), $upgrader->autoupgrade_last_version, '>=');

changed to:

return true;

So people, who actually needs to update from some older version can somehow do it?

Because writing this: "The minimum PrestaShop compatibility version has been bumped to 1.7. If you upgrade from a version older than 1.7, please use a previous version of this module." is just plain stupid as noone is able to actually use older version because of that check.

U just have to love PS for the need to hack stuff that should work... One line that I had to find and after which I can run my update from 1.6.1.9 to 1.6.1.24.

Sorry to mess this PR, I would write it into issues, but there are no issues or I don´t see em.

@matks
Copy link
Contributor

matks commented Sep 8, 2023

What about to ... AT LEAST ... before removing 1.6 stuff, release a downloadable 4.14.3 version with this line:

return version_compare($this->getModuleVersion(), $upgrader->autoupgrade_last_version, '>=');

changed to:

return true;

So people, who actually needs to update from some older version can somehow do it?

Because writing this: "The minimum PrestaShop compatibility version has been bumped to 1.7. If you upgrade from a version older than 1.7, please use a previous version of this module." is just plain stupid as noone is able to actually use older version because of that check.

U just have to love PS for the need to hack stuff that should work... One line that I had to find and after which I can run my update from 1.6.1.9 to 1.6.1.24.

Sorry to mess this PR, I would write it into issues, but there are no issues or I don´t see em.

It's actually a good idea

@matks
Copy link
Contributor

matks commented Sep 8, 2023

@darkWolf-PR See #626

@Hlavtox Hlavtox requested review from matks and a team October 3, 2023 08:22
@M0rgan01 M0rgan01 added this to the 4.16.5 milestone Nov 17, 2023
Copy link

@florine2623 florine2623 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @Hlavtox ,

Tested the following :

Upgrade went well from 1.7.6.4 to 1.7.8.0 ✅
Screenshot 2023-11-23 at 15 40 15

Screen.Recording.2023-11-23.at.15.40.45.mov

@Hlavtox Hlavtox merged commit f363210 into PrestaShop:dev Nov 23, 2023
23 checks passed
@Hlavtox
Copy link
Contributor Author

Hlavtox commented Nov 23, 2023

Thank you @florine2623! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
6 participants