Summary: | Stock rotation: cancelled transfer result in stockrotation failures | ||
---|---|---|---|
Product: | Koha | Reporter: | Martin Renvoize (ashimema) <martin.renvoize> |
Component: | Staff interface | Assignee: | Martin Renvoize (ashimema) <martin.renvoize> |
Status: | CLOSED FIXED | QA Contact: | Nick Clemens (kidclamp) <nick> |
Severity: | major | ||
Priority: | P5 - low | CC: | aleisha, bugzilla, gmcharlt, lucas, martin.renvoize, nick, testopia |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
20.11.00, 20.05.06
|
|
Circulation function: | |||
Bug Depends on: | 11897, 24296 | ||
Bug Blocks: | |||
Attachments: |
Bug 23432: Replace 'die' with 'warn'
Bug 23432: Replace 'die' with 'warn' Bug 23432: Replace 'die' with 'warn' Bug 23432: (QA follow-up) Add a unit test |
Description
Martin Renvoize (ashimema)
2019-08-05 16:12:22 UTC
Created attachment 110591 [details] [review] Bug 23432: Replace 'die' with 'warn' This patch simply replaced the die statement with a warn such that the cron script may continue to rotate other items even if this item cannot be properly rotated due to corrupt data. The patch here simply replaces the die with a warn. It does not attempt to fix the original cause of the data corruption that can take place... we attempt to resolve that in the transfers re-write tree where we significantly clean up how the transfers processes all interact. Created attachment 112405 [details] [review] Bug 23432: Replace 'die' with 'warn' This patch simply replaced the die statement with a warn such that the cron script may continue to rotate other items even if this item cannot be properly rotated due to corrupt data. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Created attachment 112424 [details] [review] Bug 23432: Replace 'die' with 'warn' This patch simply replaced the die statement with a warn such that the cron script may continue to rotate other items even if this item cannot be properly rotated due to corrupt data. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Created attachment 112425 [details] [review] Bug 23432: (QA follow-up) Add a unit test prove -v t/db_dependent/StockRotationItems.t Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed to master for 20.11, thanks to everybody involved! backported to 20.05.x for 20.05.06 missing dependencies, not backported to 19.11.x |