Attachment 174438 Details for Bug 37866 – Bug 37866: Unable to resolve claim from patron details page if syspref ClaimReturnedLostValue is disabled

[patch] Bug 37866: Unable to resolve claim from patron details page if syspref ClaimReturnedLostValue is disabled

Bug-37866-Unable-to-resolve-claim-from-patron-deta.patch (text/plain), 2.19 KB, created by Emily Lamancusa (emlam) on 2024-11-12 20:47:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Emily Lamancusa (emlam)

Created: 2024-11-12 20:47:57 UTC

Size: 2.19 KB

patch

obsolete

>From f25b257e251485b4a395760e305e431451ea5ec0 Mon Sep 17 00:00:00 2001
>From: Emmi Takkinen <emmi.takkinen@koha-suomi.fi>
>Date: Wed, 11 Sep 2024 14:05:33 +0300
>Subject: [PATCH] Bug 37866: Unable to resolve claim from patron details page
> if syspref ClaimReturnedLostValue is disabled
>
>If ones tries to resolve a claim from patrons detail page
>and syspref ClaimReturnedLostValue is disabled,
>"Resolve return claim" pop up isn't displayed. Page just
>jumps to the top of the page. However this works from the
>patron check out page.
>
>This happens because in moremember.tt line 765 we only
>check if  ClaimReturnedLostValue is enabled. In
>circulation.tt line 1023 we check if either
>ClaimReturnedLostValue or BundleLostValue syspref is enabled.
>
>To test:
>1. Disable syspref "ClaimReturnedLostValue" (set used value empty).
>2. Find patron with claims (or create them).
>3. Navigate to patrons detail page and from there open "Claims" tab.
>4. Press "Resolve" button.
>=> Pop up isn't displayed and page jumps to the top of the page.
>5. Repeat steps on patrons check out page.
>=> Pop up is displayed.
>6. Apply this patch.
>=> Pop up should now be displayed both in patrons details and check
>out pages.
>
>Sponsored-by: Koha-Suomi Oy
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>index 43760ac154..166d291887 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>@@ -767,7 +767,7 @@
>         var SuspendHoldsIntranet = [% ( Koha.Preference('SuspendHoldsIntranet') ) ? 1 : 0 | html %];
>     </script>
>     [% Asset.js("js/pages/circulation.js") | $raw %]
>-    [% IF Koha.Preference('ClaimReturnedLostValue') %]
>+    [% IF Koha.Preference('ClaimReturnedLostValue') || Koha.Preference('BundleLostValue') %]
>         [% Asset.js("js/resolve_claim_modal.js") | $raw %]
>     [% END %]
>     [% Asset.js("js/holds.js") | $raw %]
>-- 
>2.34.1

Actions: View | Diff | Splinter Review

Attachments on bug 37866: 171301 | 171352 | 174438 | 174439