Bug 26374

Summary: Update for 19974 is not idempotent
Product: Koha Reporter: Nick Clemens <nick>
Component: Architecture, internals, and plumbingAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Martin Renvoize <martin.renvoize>
Severity: major    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, kyle, martin.renvoize, victor, wainuiwitikapark
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:
21.11.00,21.05.05,20.11.12,20.05.18
Bug Depends on: 19974    
Bug Blocks:    
Attachments: Bug 26374: update from bug 19974 is not idempotent
Bug 26388: Do not show 'Renew all' or 'Renew selected' if no renewable items
Bug 26374: update from bug 19974 is not idempotent
Bug 26374: update from bug 19974 is not idempotent

Description Nick Clemens 2020-09-03 16:35:37 UTC

    
Comment 1 Nick Clemens 2020-09-03 16:36:42 UTC
Created attachment 109633 [details] [review]
Bug 26374: update from bug 19974 is not idempotent
Comment 2 Nick Clemens 2020-09-04 14:30:30 UTC Comment hidden (obsolete)
Comment 3 Nick Clemens 2020-09-04 18:05:50 UTC
Created attachment 109681 [details] [review]
Bug 26374: update from bug 19974 is not idempotent
Comment 4 Aleisha Amohia 2021-01-19 22:30:42 UTC
Does this depend on Bug 19974? Can you please provide a test plan?
Comment 5 Nick Clemens 2021-04-21 18:27:15 UTC
(In reply to Aleisha Amohia from comment #4)
> Does this depend on Bug 19974? Can you please provide a test plan?

It does depend on 19974

Essentially run the update statement, set some values, run it again, they are deleted.

I believe this was noticed because of internal backports, filing here because it could happen to others
Comment 6 Martin Renvoize 2021-10-28 08:12:57 UTC
Created attachment 127027 [details] [review]
Bug 26374: update from bug 19974 is not idempotent

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 7 Martin Renvoize 2021-10-28 08:13:23 UTC
Clear and understandable fix.. tested.

Going straight for QA
Comment 8 Jonathan Druart 2021-10-28 10:29:42 UTC
Pushed to master for 21.11, thanks to everybody involved!
Comment 9 Kyle M Hall 2021-10-29 15:46:49 UTC
Pushed to 21.05.x for 21.05.05
Comment 10 Fridolin Somers 2021-11-10 23:45:35 UTC
Pushed to 20.11.x for 20.11.12
Comment 11 Victor Grousset/tuxayo 2021-11-15 04:41:47 UTC
Backported: Pushed to 20.05.x branch for 20.05.18
Comment 12 wainuiwitikapark 2021-11-23 00:11:32 UTC
Not backporting to 19.11.x - LTS version will just backport security patches