Skip to content

Add category_shop.active migration to 9.2.0 upgrade - #1890

Open
PrestaEdit wants to merge 1 commit into
PrestaShop:7.6.xfrom
PrestaEdit:feat-category-shop-active-9.2.0
Open

Add category_shop.active migration to 9.2.0 upgrade#1890
PrestaEdit wants to merge 1 commit into
PrestaShop:7.6.xfrom
PrestaEdit:feat-category-shop-active-9.2.0

Conversation

@PrestaEdit

Copy link
Copy Markdown
Contributor
Questions Answers
Branch? 7.6.x
Type? improvement
Category? CO
BC breaks? no
Deprecations? no
Fixed issue? Companion for PrestaShop/PrestaShop#41643

Core PR PrestaShop/PrestaShop#41643 moves the category active flag from category to category_shop so visibility can be configured per shop in a multistore setup. Fresh installs get the column via db_structure.sql; existing shops need the same column added and back-filled during upgrade, otherwise every read on category_shop.active (Category, Search, Manufacturer, Supplier, the two grid query builders and CategoryRepository) fails after upgrade.

Follows the same idiom as #1887: add_column() for idempotence and an explicit AFTER position so upgraded shops match fresh-install column order.

🤖 Generated with Claude Code

Companion for PrestaShop/PrestaShop#41643, which stores the category
'active' flag per shop instead of globally. Fresh installs get the
column via db_structure.sql; upgraded shops need the same column,
seeded from the existing category.active value.

Uses add_column() so the directive is idempotent and receives the same
AFTER clause as the fresh-install order, matching the pattern of PrestaShop#1887.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

@Quetzacoalt91 Quetzacoalt91 added the Blocked Status: The issue is blocked by another task label Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Blocked Status: The issue is blocked by another task Migration script

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

3 participants