Bug 38531

Summary: Include action_logs.diff when reverting hold
Product: Koha Reporter: Andrew Fuerste-Henry <andrew>
Component: ToolsAssignee: Andrew Fuerste-Henry <andrew>
Status: Pushed to main --- QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: kebliss, m.de.rooy, sam.sowanick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.05.00
Circulation function:
Attachments: Bug 38531: add diff when reverting hold
Bug 38531: add diff when reverting hold
Bug 38531: add diff when reverting hold

Description Andrew Fuerste-Henry 2024-11-25 20:30:48 UTC
Bug 25159 added a diff column to action logs and made it populate for changes to holds. We missed adding that diff when reverting the hold waiting status.
Comment 1 Andrew Fuerste-Henry 2024-11-25 20:33:59 UTC
Created attachment 174983 [details] [review]
Bug 38531: add diff when reverting hold

To test:
1 - place, capture, and revert a hold
2 - view action logs for your hold via SQL so you can see the diff column
3 - confirm reversion of waiting hold generates a MODIFY line in logs with no diff
4 - apply patch, restart_all
5 - capture and revert your hold again
6 - confirm logs include diff showing change to found and priority
Comment 2 Biblibre Sandboxes 2024-11-25 22:57:38 UTC
Created attachment 174984 [details] [review]
Bug 38531: add diff when reverting hold

To test:
1 - place, capture, and revert a hold
2 - view action logs for your hold via SQL so you can see the diff column
3 - confirm reversion of waiting hold generates a MODIFY line in logs with no diff
4 - apply patch, restart_all
5 - capture and revert your hold again
6 - confirm logs include diff showing change to found and priority

Signed-off-by: Sam Sowanick <sam.sowanick@corvallisoregon.gov>
Comment 3 Marcel de Rooy 2024-12-13 08:48:40 UTC
Created attachment 175438 [details] [review]
Bug 38531: add diff when reverting hold

To test:
1 - place, capture, and revert a hold
2 - view action logs for your hold via SQL so you can see the diff column
3 - confirm reversion of waiting hold generates a MODIFY line in logs with no diff
4 - apply patch, restart_all
5 - capture and revert your hold again
6 - confirm logs include diff showing change to found and priority

Signed-off-by: Sam Sowanick <sam.sowanick@corvallisoregon.gov>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Katrin Fischer 2024-12-23 08:38:20 UTC
Pushed for 25.05!

Well done everyone, thank you!