Attachment 90885 Details for Bug 23097 – Bug 23097: Fix regression on overdues report and link patrons to moremember

[patch] Bug 23097: Fix regression on overdues report and link patrons to moremember

Bug-23097-Fix-regression-on-overdues-report-and-li.patch (text/plain), 2.29 KB, created by Marcel de Rooy on 2019-06-21 09:38:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2019-06-21 09:38:27 UTC

Size: 2.29 KB

patch

obsolete

>From 718451310b8965511fff7c13c36ba11fdd811157 Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <katrin.fischer.83@web.de>
>Date: Mon, 17 Jun 2019 21:35:19 +0000
>Subject: [PATCH] Bug 23097: Fix regression on overdues report and link patrons
> to moremember
>Content-Type: text/plain; charset=utf-8
>
>The link from the patron name in the circulation > overdues report
>shouldn't point to the reserves tab, as this report is about checkouts.
>
>The patch restores the behaviour as it was in 17.11.
>
>To test:
>- Make sure you have a patron account with overdue items
>- If you don't have one, check out an item with specified due date in the
>  past
>- Go to Circulation > Overdues
>- Veriy the overdue shows
>- Click on the patron name that is linked
>- Verify you go to the checkouts tab in patron account with holds tab selected
>- Apply patch
>- Reload Overdues report
>- Verify the link now leads to the Details tab with the checkouts tab in front
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt
>index 57cb911509..ad0d0594b8 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/overdue.tt
>@@ -61,7 +61,7 @@
>         <tr>
>           <td><span title="[% overdueloo.duedate | html %]">[% overdueloo.duedate | $KohaDates %]</span></td>
>           <td>
>-            [% INCLUDE 'patron-title.inc' patron=overdueloo.patron hide_patron_infos_if_needed=1 link_to="circulation_reserves" %]
>+            [% INCLUDE 'patron-title.inc' patron=overdueloo.patron hide_patron_infos_if_needed=1 %]
>             [% IF logged_in_user.can_see_patron_infos( overdueloo.patron ) %]
>                 [% IF ( overdueloo.email ) %][<a href="mailto:[% overdueloo.email | uri %]?subject=[% INCLUDE subject %] [% overdueloo.title | uri %]">email</a>][% END %]
>                 [% IF ( overdueloo.phone ) %]([% overdueloo.phone | html %])[% ELSIF ( overdueloo.mobile ) %]([% overdueloo.mobile | html %])[% ELSIF ( overdueloo.phonepro ) %]([% overdueloo.phonepro | html %])[% END %]</td>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 23097: 90705 | 90768 | 90885