Skip to content

[MODORDSTOR-527] PO line version history does not consistently highlight Holding changes when changing instance connection with "Find or create" - #650

Open
ykashtalian wants to merge 1 commit into
masterfrom
MODORDSTOR-527
Open

[MODORDSTOR-527] PO line version history does not consistently highlight Holding changes when changing instance connection with "Find or create"#650
ykashtalian wants to merge 1 commit into
masterfrom
MODORDSTOR-527

Conversation

@ykashtalian

@ykashtalian ykashtalian commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Purpose

MODORDSTOR-527 - PO line version history does not consistently highlight Holding changes when changing instance connection with "Find or create"

Approach

  • In batch mode, the intermediate PO line version is no longer emitted; the direct PATCH transaction produces the single authoritative version.
  • The PATCH now re-reads the PO line under SELECT FOR UPDATE inside the transaction and rebases the instance/location changes on that fresh row, preventing the lost update. Existing lock ordering is preserved (pieces -> po_line -> titles).

Pre-Review Checklist

  • Self-reviewed Code — Reviewed code for issues, unnecessary parts, and overall quality.
  • Change Notes — NEWS.md updated with clear description and issue key.
  • Testing — Confirmed changes were tested locally or on dev environment.
  • Logging: Confirmed that logging is appropriately handled.
  • Breaking Changes — Handled all required actions if changes affect API, DB, or interface versions.
    • API/schema changes
    • Interface version updates
    • DB schema changes / migration scripts
  • New Properties / Environment Variables — Updated README.md if new configs were added.

@ykashtalian ykashtalian self-assigned this Jul 27, 2026
@sonarqubecloud

Copy link
Copy Markdown

@ykashtalian ykashtalian added the ai label Jul 28, 2026
@ykashtalian
ykashtalian marked this pull request as ready for review July 28, 2026 07:31
@ykashtalian
ykashtalian requested a review from a team as a code owner July 28, 2026 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant