mirror of
https://github.com/opensourcepos/opensourcepos.git
synced 2026-09-13 13:57:34 -04:00
c0abf80c86
* fix: wrap postSave() in single transaction for atomicity - Remove internal transaction from Item_taxes->save_value() to allow controller-level transaction - Wrap entire save sequence (item, taxes, quantities, inventory, attributes) in single transaction - Ensure all operations succeed or all fail together - Prevents partial writes when saveItemAttributes() fails after item/tax/quantity saves succeed Fixes #4474 * fix: Use explicit transBegin/transCommit/transRollback for atomicity - Replace transStart/transComplete with transBegin/transCommit/transRollback - Check all success conditions before committing - Explicit rollback on failure Address CodeRabbit review feedback * refactor(items): rename postSave locals to camelCase per PSR-12 Convert snake_case variables to camelCase in postSave() and related item-save logic to match project naming convention for new methods. Signed-off-by: Travis Garrison <travis@chiraqbookstore.com> --------- Signed-off-by: Travis Garrison <travis@chiraqbookstore.com> Co-authored-by: Ollama <ollama@steganos.dev> Co-authored-by: objecttothis <17935339+objecttothis@users.noreply.github.com> Co-authored-by: Travis Garrison <travis@chiraqbookstore.com>