Skip to content

feat: add PS_SHOP_MODE migration for 9.2.0 - #1858

Merged
Quetzacoalt91 merged 1 commit into
PrestaShop:7.6.xfrom
soledis-contributeur:B2B-CONTRIBUTION/EPIC1-US1.2-b2c-b2b-configuration
Jul 3, 2026
Merged

feat: add PS_SHOP_MODE migration for 9.2.0#1858
Quetzacoalt91 merged 1 commit into
PrestaShop:7.6.xfrom
soledis-contributeur:B2B-CONTRIBUTION/EPIC1-US1.2-b2c-b2b-configuration

Conversation

@soledis-contributeur

@soledis-contributeur soledis-contributeur commented Jun 30, 2026 •
edited by ingridusta
Loading

Copy link
Copy Markdown
Questions Answers
Description? Adds the PS_SHOP_MODE configuration migration in upgrade/sql/9.2.0.sql.
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? Related to PrestaShop/PrestaShop#40685
Sponsor company Soledis
How to test? Upgrade a shop from a pre-9.2.0 version to 9.2.0 using the autoupgrade module. After the upgrade, check that the PS_SHOP_MODE row exists in the ps_configuration table with the value b2c_only (e.g. SELECT value FROM ps_configuration WHERE name = 'PS_SHOP_MODE';). It must match the value a fresh 9.2.0 install gets from configuration.xml. Re-running the update must not error or change an existing value.

@ps-jarvis

ps-jarvis commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Hello @soledis-contributeur!

This is your first pull request on autoupgrade repository of the PrestaShop project.

Thank you, and welcome to this Open Source community!

@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Jun 30, 2026
@SachaFARINEL
SachaFARINEL force-pushed the B2B-CONTRIBUTION/EPIC1-US1.2-b2c-b2b-configuration branch from 49048fe to fcb5915 Compare June 30, 2026 15:39
Quetzacoalt91
Quetzacoalt91 reviewed Jun 30, 2026
View reviewed changes

@Quetzacoalt91 Quetzacoalt91 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @soledis-contributeur. Can you please target the branch 7.6.x so we can release quickly?

Comment thread upgrade/sql/9.2.0.sql Outdated
@SachaFARINEL
SachaFARINEL force-pushed the B2B-CONTRIBUTION/EPIC1-US1.2-b2c-b2b-configuration branch from fcb5915 to c6fd1df Compare June 30, 2026 17:13
@soledis-contributeur
soledis-contributeur changed the base branch from dev to 7.6.x June 30, 2026 17:19
@soledis-contributeur

soledis-contributeur commented Jun 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Author

Thanks @soledis-contributeur. Can you please target the branch 7.6.x so we can release quickly?

Done! Rebased onto 7.6.x and applied your suggestion using add_configuration_if_not_exists. Thanks!

@ps-jarvis ps-jarvis added the Waiting for QA Status: Action required, Waiting for test feedback label Jul 1, 2026
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Jul 1, 2026
@M0rgan01

M0rgan01 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Thanks @soledis-contributeur

@ingridusta ingridusta self-assigned this Jul 2, 2026
@ingridusta

ingridusta commented Jul 2, 2026

Copy link
Copy Markdown

@soledis-contributeur can u rebase please ?

@SachaFARINEL
feat: add PS_SHOP_MODE migration for 9.2.0
3ff6ca6 
Add the PS_SHOP_MODE configuration migration so existing shops upgraded
to 9.2.0 get the same configuration as a fresh install.

Uses add_configuration_if_not_exists so the migration is idempotent and
never overwrites a value already set on the shop.

Related to PrestaShop/PrestaShop#40685
@SachaFARINEL
SachaFARINEL force-pushed the B2B-CONTRIBUTION/EPIC1-US1.2-b2c-b2b-configuration branch from c6fd1df to 3ff6ca6 Compare July 2, 2026 16:09
@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Quetzacoalt91 Quetzacoalt91 removed Waiting for rebase Waiting for QA Status: Action required, Waiting for test feedback labels Jul 3, 2026
@Quetzacoalt91 Quetzacoalt91 added the Waiting for QA Status: Action required, Waiting for test feedback label Jul 3, 2026
ingridusta
ingridusta approved these changes Jul 3, 2026
View reviewed changes

@ingridusta ingridusta left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi @soledis-contributeur,

Thanks for your contribution !

When updating to 9.2.x, database is updated :

Image

It's QA approved ✅

@ingridusta ingridusta added QA ✔️ Status: Check done, Code approved and removed Waiting for QA Status: Action required, Waiting for test feedback labels Jul 3, 2026
@Quetzacoalt91
Quetzacoalt91 merged commit 84a6f4a into PrestaShop:7.6.x Jul 3, 2026
78 of 82 checks passed
@github-project-automation github-project-automation Bot moved this from To be tested to Merged in PR Dashboard Jul 3, 2026
@ps-jarvis ps-jarvis moved this from Merged to To be tested in PR Dashboard Jul 3, 2026
@ps-jarvis

ps-jarvis commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

PR merged, well done!

Message to @PrestaShop/committers: do not forget to milestone it before the merge.

@Quetzacoalt91 Quetzacoalt91 added this to the 7.6.6 milestone Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Quetzacoalt91 Quetzacoalt91 Quetzacoalt91 approved these changes
@M0rgan01 M0rgan01 M0rgan01 approved these changes
@ingridusta ingridusta ingridusta approved these changes

Assignees

@ingridusta ingridusta

Labels

Migration script QA ✔️ Status: Check done, Code approved

Projects

PR Dashboard
Archived in project

Milestone

7.6.6

Development

Successfully merging this pull request may close these issues.

6 participants

@soledis-contributeur @ps-jarvis @M0rgan01 @ingridusta @Quetzacoalt91 @SachaFARINEL