From 54a17d97dcdcf31e9d4228c89473c82373e48e68 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Fr=C3=A9d=C3=A9rick?= <frederick.capovilla@libeo.com>
Date: Mon, 9 Dec 2013 13:45:52 -0500
Subject: [PATCH] Bug 11359: Add the borrower's cardnumber and phonenumber to
 the branch overdues report

The report also known as "Overdues with fines"

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>

All tests pass, this adds data to the Patron column on the
overdues with fines report to show the patron's cardnumber
and phone number.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
This works as described and passes all tests and QA script.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---
 C4/Overdues.pm                                            | 1 +
 circ/branchoverdues.pl                                    | 1 +
 .../intranet-tmpl/prog/en/modules/circ/branchoverdues.tt  | 8 ++++++--
 3 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/C4/Overdues.pm b/C4/Overdues.pm
index 33dbd9874e..af83e0f30b 100644
--- a/C4/Overdues.pm
+++ b/C4/Overdues.pm
@@ -796,6 +796,7 @@ sub GetOverduesForBranch {
     my $dbh = C4::Context->dbh;
     my $select = "
     SELECT
+            borrowers.cardnumber,
             borrowers.borrowernumber,
             borrowers.surname,
             borrowers.firstname,
diff --git a/circ/branchoverdues.pl b/circ/branchoverdues.pl
index c81547ed43..31e2b44a43 100755
--- a/circ/branchoverdues.pl
+++ b/circ/branchoverdues.pl
@@ -129,6 +129,7 @@ foreach my $num (@getoverdues) {
     $overdueforbranch{'itemcallnumber'}    = $num->{'itemcallnumber'};
     $overdueforbranch{'borrowernumber'}    = $num->{'borrowernumber'};
     $overdueforbranch{'itemnumber'}        = $num->{'itemnumber'};
+    $overdueforbranch{'cardnumber'}        = $num->{'cardnumber'};
 
     # now we add on the template, the differents values of notify_level
     # FIXME: numerical comparison, not string eq.
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt
index 87eeac6efb..dd2d7e6718 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt
@@ -48,7 +48,9 @@
                             <br />Barcode : [% overduesloo.barcode %]
                     </td>
                     <td>
-                        <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% overduesloo.borrowernumber %]">[% overduesloo.borrowersurname %], [% overduesloo.borrowerfirstname %]</a><br />[% overduesloo.borrowerphone %]<br />
+                        <a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% overduesloo.borrowernumber %]">[% overduesloo.borrowersurname %], [% overduesloo.borrowerfirstname %]</a>
+                        <br />[% overduesloo.cardnumber %]
+                        <br />[% overduesloo.borrowerphone %]<br />
                     [% IF ( overduesloo.borroweremail ) %]<a href="mailto:[% overduesloo.borroweremail %]?subject=Overdue: [% overduesloo.title |html %]">
                     [% overduesloo.borroweremail %]</a>[% END %]
                     </td>
@@ -126,7 +128,9 @@
                             </p>
                         </td>
                         <td>
-                            <p><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% todayoverduesloo.borrowernumber %]">[% todayoverduesloo.borrowersurname %] &nbsp; [% todayoverduesloo.borrowerfirstname %]</a><br />[% todayoverduesloo.borrowerphone %]<br />
+                            <p><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% todayoverduesloo.borrowernumber %]">[% todayoverduesloo.borrowersurname %] &nbsp; [% todayoverduesloo.borrowerfirstname %]</a>
+                            <br />[% todayoverduesloo.cardnumber %]<br />
+                            <br />[% todayoverduesloo.borrowerphone %]<br />
                         [% IF ( todayoverduesloo.borroweremail ) %]<a href="mailto:[% todayoverduesloo.borroweremail %]?subject=Overdue: [% todayoverduesloo.title |html %]">
                         [% todayoverduesloo.borroweremail %]</a>[% END %]
                             </p>
-- 
2.39.5