Attachment 28674 Details for Bug 11857 – Bug 11857 - Number of patrons on a patron list not accurate

[patch] Bug 11857 - Number of patrons on a patron list not accurate

Bug-11857---Number-of-patrons-on-a-patron-list-not.patch (text/plain), 2.04 KB, created by Fridolin Somers on 2014-06-05 15:31:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fridolin Somers

Created: 2014-06-05 15:31:08 UTC

Size: 2.04 KB

patch

obsolete

>From a72bf1f6d791a2ce872d3cafc6e68e25f6cdaad4 Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Thu, 5 Jun 2014 17:21:28 +0200
>Subject: [PATCH] Bug 11857 - Number of patrons on a patron list not accurate
>
>When creating a patron list with only 1 patron, the patron lists page says that the list contains 3 patrons.
>When creating a patron list with only 1 patron, it works fine.
>
>This is caused by the fact that access on "patron_list_patrons" Koha::Schema::Result::PatronList object retruns an hash if one result and an array if more results.
>See similar problem at http://permalink.gmane.org/gmane.comp.lang.perl.modules.template-toolkit/7250
>
>This patch replaces by the call on resultset and then the use of "count" method.
>
>Test plan :
>- Go to patron lists : /cgi-bin/koha/patron_lists/lists.pl
>- Create a new patron list LIO
>- Add one patron
>- Create a new patron list LIM
>- Add 2 patrons
>- Go to patron lists : /cgi-bin/koha/patron_lists/lists.pl
>=> Without this patch you see :
>  Name  Patrons in list
>  LIO   3
>  LIM   2
>=> With this patch you see :
>  Name  Patrons in list
>  LIO   1
>  LIM   2
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
>index 8be8035..2703144 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/lists.tt
>@@ -51,7 +51,7 @@
>                 [% FOREACH l IN lists %]
>                     <tr>
>                         <td>[% l.name %]</td>
>-                        <td>[% l.patron_list_patrons.size || 0 %]</td>
>+                        <td>[% l.patron_list_patrons_rs.count || 0 %]</td>
>                         <td>
>                             <a class="btn btn-mini" href="list.pl?patron_list_id=[% l.patron_list_id %]">
>                                 <i class="icon-plus-sign"></i> Add patrons <i class="icon-user"></i>
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 11857: 28674 | 28725 | 28728