Summary: | Filling a hold should update the timestamp | ||
---|---|---|---|
Product: | Koha | Reporter: | Johanna Räisä <johanna.raisa> |
Component: | Circulation | Assignee: | Johanna Räisä <johanna.raisa> |
Status: | Pushed to main --- | QA Contact: | Lucas Gass (lukeg) <lucas> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | gmcharlt, kebliss, kyle.m.hall, lucas, martin.renvoize |
Version: | Main | Keywords: | release-notes-needed |
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | Sponsored | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
24.11.00
|
|
Circulation function: | holds | ||
Attachments: |
Bug 37023: Update timestamp when filling a hold
Bug 37023: Update timestamp when filling a hold Bug 37023: Update timestamp when filling a hold |
Description
Johanna Räisä
2024-06-04 10:28:25 UTC
Created attachment 167378 [details] [review] Bug 37023: Update timestamp when filling a hold This patch updates the timestamp of the hold when it is filled and moved to old_reserves. Test plan: 1) Apply the patch 2) prove t/db_dependent/Koha/Hold.t Sponsored-by: Koha-Suomi Oy Created attachment 167457 [details] [review] Bug 37023: Update timestamp when filling a hold This patch updates the timestamp of the hold when it is filled and moved to old_reserves. Test plan: 1) Apply the patch 2) prove t/db_dependent/Koha/Hold.t Sponsored-by: Koha-Suomi Oy Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Trivial change, does what it is suppose to, includes tests to prove the chane, QA script is happy. Created attachment 171904 [details] [review] Bug 37023: Update timestamp when filling a hold This patch updates the timestamp of the hold when it is filled and moved to old_reserves. Test plan: 1) Apply the patch 2) prove t/db_dependent/Koha/Hold.t Sponsored-by: Koha-Suomi Oy Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> This is probably a bug, but maybe the change is better for a major release? Pushed for 24.11! Well done everyone, thank you! |