Summary: | Last returned by and last/previous borrower doesn't display if patron's cardnumber is empty | ||
---|---|---|---|
Product: | Koha | Reporter: | hebah |
Component: | Staff interface | Assignee: | Lucas Gass (lukeg) <lucas> |
Status: | Pushed to main --- | QA Contact: | Matt Blenkinsop <matt.blenkinsop> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, lucas, matt.blenkinsop, mteal |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: |
This fixes a bug where the 'Last borrower' and 'Previous borrower' links did not appear when the borrower lacked a cardnumber. This patch makes it fall back to borrowernumber in those cases.
|
Version(s) released in: |
25.05.00
|
Circulation function: | |||
Attachments: |
Bug 34681: Fallback to borrowernumber when there is no cardnumber
Bug 34681: Fallback to borrowernumber when there is no cardnumber Bug 34681: Follow up - fix typo Bug 34681: Fallback to borrowernumber when there is no cardnumber Bug 34681: Follow up - fix typo |
Description
hebah
2023-08-31 18:42:46 UTC
Created attachment 176876 [details] [review] Bug 34681: Fallback to borrowernumber when there is no cardnumber To test: 1. Have StoreLastBorrower enabled to store Last Returned By. 2. Create account and leave cardnumber empty. 3. Check out and return item with that account. 4. Go to items detail page. 5. See that Last Returned By and, if return is not anonymized, Last Borrower appears blank (you can inspect the fields and see that borrower information is there but not visible). Previous borrower will similarly be blank if it is from an account with no cardnumber. 6. APPLY PATCH 7. Try the items detail page again. You should see the link for 'Last borrower' and/or 'Previous borrower' now as the borrowernumber. Created attachment 176940 [details] [review] Bug 34681: Fallback to borrowernumber when there is no cardnumber To test: 1. Have StoreLastBorrower enabled to store Last Returned By. 2. Create account and leave cardnumber empty. 3. Check out and return item with that account. 4. Go to items detail page. 5. See that Last Returned By and, if return is not anonymized, Last Borrower appears blank (you can inspect the fields and see that borrower information is there but not visible). Previous borrower will similarly be blank if it is from an account with no cardnumber. 6. APPLY PATCH 7. Try the items detail page again. You should see the link for 'Last borrower' and/or 'Previous borrower' now as the borrowernumber. Signed-off-by: Magnus Enger <magnus@libriotech.no> Works as advertised, with the tiny followup I am providing. Created attachment 176941 [details] [review] Bug 34681: Follow up - fix typo Change from: [% last_retruned_by | html %] To: [% last_returned_by | html %] Created attachment 177117 [details] [review] Bug 34681: Fallback to borrowernumber when there is no cardnumber To test: 1. Have StoreLastBorrower enabled to store Last Returned By. 2. Create account and leave cardnumber empty. 3. Check out and return item with that account. 4. Go to items detail page. 5. See that Last Returned By and, if return is not anonymized, Last Borrower appears blank (you can inspect the fields and see that borrower information is there but not visible). Previous borrower will similarly be blank if it is from an account with no cardnumber. 6. APPLY PATCH 7. Try the items detail page again. You should see the link for 'Last borrower' and/or 'Previous borrower' now as the borrowernumber. Signed-off-by: Magnus Enger <magnus@libriotech.no> Works as advertised, with the tiny followup I am providing. Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Created attachment 177118 [details] [review] Bug 34681: Follow up - fix typo Change from: [% last_retruned_by | html %] To: [% last_returned_by | html %] Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Pushed for 25.05! Well done everyone, thank you! |