Summary: | Use checkout object in GetSoonestRenewDate | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Architecture, internals, and plumbing | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | CLOSED FIXED | QA Contact: | Katrin Fischer <katrin.fischer> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | david, lucas, m.de.rooy |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
22.11.00
|
Circulation function: | |
Bug Depends on: | 30167 | ||
Bug Blocks: | |||
Attachments: |
Bug 30168: Use checkout object in GetSoonestRenewDate
Bug 30168: Embed patron and item in issue for passing to GetSoonestRenewDate Bug 30168: Use checkout object in GetSoonestRenewDate Bug 30168: Use checkout object in GetSoonestRenewDate Bug 30168: Use checkout object in GetSoonestRenewDate Bug 30168: (QA follow-up) Fix POD |
Description
Nick Clemens (kidclamp)
2022-02-23 16:24:48 UTC
Created attachment 131048 [details] [review] Bug 30168: Use checkout object in GetSoonestRenewDate Created attachment 131049 [details] [review] Bug 30168: Embed patron and item in issue for passing to GetSoonestRenewDate (In reply to Nick Clemens from comment #2) > Created attachment 131049 [details] [review] [review] > Bug 30168: Embed patron and item in issue for passing to GetSoonestRenewDate Obsoleting, see bug 30169 Created attachment 134603 [details] [review] Bug 30168: Use checkout object in GetSoonestRenewDate Created attachment 140969 [details] [review] Bug 30168: Use checkout object in GetSoonestRenewDate Signed-off-by: David Nind <david@davidnind.com> I've signed off on the basis that the tests pass before and after the patch is applied: prove t/db_dependent/Circulation.t Let me know if there is anything else required to test this. I also ran similar tests to bug 30167, for what it is worth. - Tested on 09/25/2022 (American date format) with . 'no renewal before' set to 3 . 'loan period' set to 5 - Checked out on date is 09/25/2022 05:10 - For the patron from the checkouts: No renewal before 04/27/2022 - When trying to renew from Circulation > Renew, the message is "Intermediate Perl / ( 39999000005134 ) cannot be renewed before 09/27/2022." - When trying to renew by checking out to the same patron again, the message is: "No more renewals possible" - Date is calculated correctly including the current day (3 days - 25th, 26th + 27th) (In reply to David Nind from comment #7) > I also ran similar tests to bug 30167, for what it is worth. > > - Tested on 09/25/2022 (American date format) with > . 'no renewal before' set to 3 > . 'loan period' set to 5 > - Checked out on date is 09/25/2022 05:10 > - For the patron from the checkouts: No renewal before 04/27/2022 > - When trying to renew from Circulation > Renew, the message is > "Intermediate Perl / ( 39999000005134 ) cannot be renewed before 09/27/2022." > - When trying to renew by checking out to the same patron again, the message > is: "No more renewals possible" > - Date is calculated correctly including the current day (3 days - 25th, > 26th + 27th) Thanks David, your notes are always very helpful! Created attachment 141554 [details] [review] Bug 30168: Use checkout object in GetSoonestRenewDate Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Created attachment 141555 [details] [review] Bug 30168: (QA follow-up) Fix POD The POD had $issue in the example, but $checkout in explanation. Also standardized a bit of other terminology. (borrower, issue) = (patron, checkout) Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 22.11. Nice work everyone, thanks! |