From f3322257d5a2fe003656a25ce3f519c9c5690f70 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Fri, 14 Oct 2011 10:37:56 -0400 Subject: [PATCH] Bug 13134 - Add patron category to returns confirmation dialogs Some librarians find it useful to know what category a patron is before confirming a reserve or transfer from the checkin screen. This patch adds the patron category to the hold and transfer popups to the patron information already displayed. The li tags that contain the patron category have the class "patron-category" to allow this data to be easily hidden. Test Plan: 1) Apply this patch 2) Trap a hold for a patron, note the patron category is now displayed 3) Trap a hold for pickup at another loation, note the patron category is now displayed Signed-off-by: Owen Leonard Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall --- circ/returns.pl | 3 +++ koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt | 8 +++++--- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/circ/returns.pl b/circ/returns.pl index 0f227dd3a2..e849a46e1a 100755 --- a/circ/returns.pl +++ b/circ/returns.pl @@ -175,6 +175,7 @@ if ( $query->param('reserve_id') ) { borcnum => $borr->{'cardnumber'}, borfirstname => $borr->{'firstname'}, borsurname => $borr->{'surname'}, + borcategory => $borr->{'description'}, diffbranch => 1, ); } @@ -405,6 +406,7 @@ if ( $messages->{'WrongTransfer'} and not $messages->{'WasTransfered'}) { wname => $name, wborfirstname => $borr->{'firstname'}, wborsurname => $borr->{'surname'}, + wborcategory => $borr->{'description'}, wbortitle => $borr->{'title'}, wborphone => $borr->{'phone'}, wboremail => $borr->{'email'}, @@ -446,6 +448,7 @@ if ( $messages->{'ResFound'}) { name => $borr->{'surname'} . ", " . $borr->{'title'} . " " . $borr->{'firstname'}, borfirstname => $borr->{'firstname'}, borsurname => $borr->{'surname'}, + borcategory => $borr->{'description'}, bortitle => $borr->{'title'}, borphone => $borr->{'phone'}, boremail => $borr->{'email'}, diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt index 8e1849aa77..631150251e 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt @@ -228,8 +228,8 @@ $(document).ready(function () { [% IF ( wborcnum ) %]
Hold for:
  • - [% borsurname %], [% borfirstname %] ([% borcnum %])
  • -
  • [% wborstnum %] [% wboraddress %][% IF ( wboraddress2 ) %]
    + [% borsurname %], [% borfirstname %] ([% borcnum %]) - [% wborcategory %]
  • +
  • [% wborstnum %] [% wboraddress %][% IF ( wboraddress2 ) %]
    [% wboraddress2 %]
    [% END %] [% wborcity %] [% wborzip %]
  • [% IF ( wborphone ) %]
  • [% wborphone %]
  • [% END %] @@ -274,6 +274,7 @@ $(document).ready(function () {
  • [% borsurname %], [% borfirstname %] ([% borcnum %]) + - [% wborcategory %]
  • @@ -352,7 +353,7 @@ $(document).ready(function () {

    [% itembarcode |html %]: [% title |html %]

    Hold for:

      -
    • [% borsurname %], [% borfirstname %] ([% borcnum %])
    • +
    • [% borsurname %], [% borfirstname %] ([% borcnum %]) - [% wborcategory %]
    • [% borstnum %] [% boraddress %]
      [% IF ( boraddress2 ) %][% boraddress2 %]
      [% END %] [% borcity %] [% borzip %]
    • @@ -477,6 +478,7 @@ $(document).ready(function () { [% borsurname %], [% borfirstname %] ([% borcnum %]) + - [% wborcategory %]
    • -- 2.39.5