Attachment 175057 Details for Bug 38512 – Bug 38512: Fix display of recalls in items table

[patch] Bug 38512: Fix display of recalls in items table

Bug-38512-Fix-display-of-recalls-in-items-table.patch (text/plain), 3.57 KB, created by Lucas Gass (lukeg) on 2024-12-02 17:42:58 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Lucas Gass (lukeg)

Created: 2024-12-02 17:42:58 UTC

Size: 3.57 KB

patch

obsolete

>From 83ad374ffa5f157d07df99964730ae878a633923 Mon Sep 17 00:00:00 2001
>From: Lucas Gass <lucas@bywatersolutions.com>
>Date: Mon, 2 Dec 2024 17:40:54 +0000
>Subject: [PATCH] Bug 38512: Fix display of recalls in items table
>
>1. Turn on UseRecalls
>2. Set circ rules to allow for recalls ( Recalls allowed(total) and Recalls allowed(count))
>3. Check something out to a patron, choose a record with multiple items on it
>4. With another patron recall the item from the OPAC
>5. Choose "Recall next available item"
>6. Now go back to the item table and see that all items on the bib say something like: "Recalled by koha (42) on 11/21/2024"
>7. APPLY PATCH
>8. Do steps 3 - 6 again and now the items should correctly show as available.
>9. Place a recall again but choose 'Recall a specific item'.
>10. Look at the table again, it should accuretly show the recall in the status column. "recalled by koha (42) on 11/21/2024"
>11. Turn off UseRecalls, make sure the table still loads properly.
>---
> .../html_helpers/tables/items/catalogue_detail.inc | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc
>index 9fa81b87f7..37b95f254a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc
>@@ -547,20 +547,20 @@
> }
> 
> [% IF Koha.Preference('UseRecalls') %]
>- if ( row.recall ) {
>+ if ( row.recall && ( row.item_id === row.recall.item_id ) ) {
> if ( row.recall.waiting_date ) {
>- nodes += '%s'.format(_("Waiting at %s since %s").format(escape_str(row.recall.pickup_library_id.str), $date(row.recall.waiting_date)));
>+ nodes += '%s'.format(_("Waiting recall at %s since %s").format(escape_str(row.recall._strings.pickup_library_id.str), $date(row.recall.waiting_date)));
> } else {
> [%# Hacky for patron_to_html in case we simply want to display the patron's library name %]
> row.recall.patron.library = { name: libraries_names.get(row.recall.patron.library_id) };
> 
> let patron_to_html = $patron_to_html(row.recall.patron, {url: true, display_cardnumber: true, hide_patron_name });
>- nodes += '%s'.format(_("recalled by %s on %s").format(patron_to_html, $date(row.recall.created_date)))
>+ nodes += '%s'.format(_("Recalled by %s on %s").format(patron_to_html, $date(row.recall.created_date)))
> }
> }
> [% END %]
> 
>- if ( ! ( row.not_for_loan_status || item_types_notforloan.get(row.item_type_id) || row.checked_out_date || row.lost_status || row.withdrawn || row.damaged_status || row.transfer || row.first_hold || row.recall ) ) {
>+ if ( ! ( row.not_for_loan_status || item_types_notforloan.get(row.item_type_id) || row.checked_out_date || row.lost_status || row.withdrawn || row.damaged_status || row.transfer || row.first_hold || ( row.recall && ( row.item_id === row.recall.item_id ) ) )) {
> nodes += ' %s'.format(_("Available"))
> }
> 
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 38512: 174904 | 175057 | 175068 | 175571