Skip to content

Avoid using deleted products in Stock Planning#818

Merged
HaGuesto merged 1 commit intomasterfrom
fix-deleted-products-in-stock-planning
Mar 23, 2026
Merged

Avoid using deleted products in Stock Planning#818
HaGuesto merged 1 commit intomasterfrom
fix-deleted-products-in-stock-planning

Conversation

@pylipp
Copy link
Copy Markdown
Contributor

@pylipp pylipp commented Mar 23, 2026

https://trello.com/c/DKuBXKJ8

For testing, log in as user of base 1.

  1. comment the fix
  2. clone the "grouping" prouduct for e.g. trousers: insert into products (name, category_id, gender_id, sizegroup_id, camp_id) values ("Trousers", 2, 1, 1, 1)
  3. delete the "grouping" product for e.g. trousers: update products set deleted = utc_timestamp() where id = 1174
  4. reload, and click Bottoms -> Trousers -> Boy. The overview is now empty because the URL uses the deleted product 1174
  5. activate the fix and reload Stock Planning
  6. click Bottoms -> Trousers -> Boy. Overview shows two boxes, and the URL correctly uses the new product ID

@pylipp pylipp requested a review from HaGuesto March 23, 2026 11:51
@HaGuesto HaGuesto merged commit f498a9d into master Mar 23, 2026
6 checks passed
@HaGuesto HaGuesto deleted the fix-deleted-products-in-stock-planning branch March 23, 2026 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants