Bug 37023 - Filling a hold should update the timestamp
Summary: Filling a hold should update the timestamp
Status: Passed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Johanna Räisä
QA Contact: Lucas Gass (lukeg)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-06-04 10:28 UTC by Johanna Räisä
Modified: 2024-09-30 14:39 UTC (History)
5 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 37023: Update timestamp when filling a hold (2.30 KB, patch)
2024-06-04 11:50 UTC, Johanna Räisä
Details | Diff | Splinter Review
Bug 37023: Update timestamp when filling a hold (2.31 KB, patch)
2024-06-05 15:43 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 37023: Update timestamp when filling a hold (2.35 KB, patch)
2024-09-23 22:38 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Johanna Räisä 2024-06-04 10:28:25 UTC
At the moment when a hold is filled the timestamp on database doesn't get updated. The row in the reserves table is copied to old_reserves and it is filled with the same timestamp value as before.
Comment 1 Johanna Räisä 2024-06-04 11:50:21 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
Comment 2 Matt Blenkinsop 2024-06-05 15:43:39 UTC
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>
Comment 3 Lucas Gass (lukeg) 2024-09-23 22:35:18 UTC
Trivial change, does what it is suppose to, includes tests to prove the chane, QA script is happy.
Comment 4 Lucas Gass (lukeg) 2024-09-23 22:38:40 UTC
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>