Skip to content

Fix: restrict order detail price updates to current order (#607) - #608

Merged
Rom1-B merged 1 commit into
10.0/bugfixesfrom
litlle_fix_10
Sep 16, 2026
Merged

Rom1-B merged 1 commit into
10.0/bugfixesfrom
litlle_fix_10

Conversation

@stonebuzz

Copy link
Copy Markdown
Contributor

(cherry picked from commit bd5d9f3)

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

Backport for #607

When saving the detail lines of an order, the submitted price and discount values were applied to any item id received, even when that item belonged to another order.
A user editing one order could therefore alter the prices of items attached to an order they are not allowed to access.
Each submitted item is now checked against the order being edited, and lines that do not belong to it are ignored.

Screenshots (if appropriate):

Co-authored-by: Stanislas Kita <7335054+stonebuzz@users.noreply.github.com>
(cherry picked from commit bd5d9f3)
@stonebuzz stonebuzz self-assigned this Sep 16, 2026
@stonebuzz
stonebuzz requested a review from Rom1-B September 16, 2026 07:36
@Rom1-B
Rom1-B merged commit 97324fb into 10.0/bugfixes Sep 16, 2026
3 checks passed
@Rom1-B
Rom1-B deleted the litlle_fix_10 branch September 16, 2026 08:28
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