Attachment 167269 Details for Bug 36984 – bug 36984 Fixes "Transit pending from..." message in bibliographic record detail page

[patch] bug 36984 Fixes "Transit pending from..." message in bibliographic record detail page

bug-36984-Fixes-Transit-pending-from-message-in-bi.patch (text/plain), 2.87 KB, created by Adolfo Rodríguez Taboada on 2024-05-30 10:09:49 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Adolfo Rodríguez Taboada

Created: 2024-05-30 10:09:49 UTC

Size: 2.87 KB

patch

obsolete

>From f5170d6766530783e5f9795daad4c0fc2b2610af Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Adolfo=20Rodr=C3=ADguez?= <adolfo.rodriguez@xercode.es>
>Date: Thu, 30 May 2024 12:07:21 +0200
>Subject: [PATCH] bug 36984 Fixes "Transit pending from..." message in
> bibliographic record detail page
>
>Transfering a rotating collection will mark the items contained in the collection as "Transit pending..." and it will trigger a javascript error in the bibliographic record detail page that breaks the holdings table, leaving it caught in the "Processing..." message. This patch fixes that.
>
>The javascript messages gives this error in the console tab:
>Uncaught TypeError: Cannot read properties of undefined (reading 'str')
>
>The error comes from using from_library_ insted of from_library in the file koha-tmpl/intranet-tmpl/prog/en/includes/html_helpers/tables/items/catalogue_detail.inc
>
>Test plan:
>1 Create a rotating collection
>2 Add at least an item to the collection
>3 Transfer the rotating collection
>4 Check the detail page of the bibliography record of any of the items that belong to that collection in the Intranet. Notice that the holdings table gets stuck "Processing..." and that there is a Javascript error
>5 Apply patch, restart services
>6 Repeat step 4. Now the holdings tab works as intended and the item from the collection shows up with the text "Transit pending from..." in the Status column
>7 Sign off
>---
> .../en/includes/html_helpers/tables/items/catalogue_detail.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>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 3c08fe3ec7..922a2192e5 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
>@@ -474,7 +474,7 @@
> if ( row.transfer.datesent ) {
> nodes += '%s'.format(_("In transit from %s to %s since %s").format(escape_str(row.transfer._strings.from_library.str), escape_str(row.transfer._strings.to_library.str), $date(row.transfer.datesent)));
> } else {
>- nodes += '%s'.format(_("Transit pending from %s to %s since %s").format(escape_str(row.transfer._strings.from_library_.str), escape_str(row.transfer._strings.to_library.str), $date(row.transfer.daterequested)));
>+ nodes += '%s'.format(_("Transit pending from %s to %s since %s").format(escape_str(row.transfer._strings.from_library.str), escape_str(row.transfer._strings.to_library.str), $date(row.transfer.daterequested)));
> }
> }
> 
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 36984: 167239 | 167269 | 167291 | 167759