From 5d6c092921919526ade501facb1220f8a108a08f Mon Sep 17 00:00:00 2001 From: David Roberts Date: Mon, 30 Jun 2014 12:22:46 +0100 Subject: [PATCH] Bug 12495 - Include streetnumber in hold alert address Steetnumber is not included in the addess in the hold alert add the field to those displayed To test this, place a hold on an item that is on-loan, having first made sure that the requesting user has a value in the street number field of his address. Check the item in, and an alert should appear giving the name and address of the requesting user. This address should include the contents of the street number address field. Signed-off-by: Owen Leonard I was able to confirm three of the four template changes for the following cases: - Item placed on hold for a patron at the current library, not waiting - Item placed on hold for a patron at the current library, already waiting - Item placed on hold for a patron at another library, not waiting I don't know how to trigger the case [% IF ( diffbranch ) %]. Signed-off-by: Katrin Fischer Passes QA script and tests. Note: Maybe sample slip templates should also include street number? Signed-off-by: Tomas Cohen Arazi --- circ/returns.pl | 2 ++ koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt | 8 ++++---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/circ/returns.pl b/circ/returns.pl index a95a17ca85..428edb1512 100755 --- a/circ/returns.pl +++ b/circ/returns.pl @@ -384,6 +384,7 @@ if ( $messages->{'WrongTransfer'} and not $messages->{'WasTransfered'}) { wbortitle => $borr->{'title'}, wborphone => $borr->{'phone'}, wboremail => $borr->{'email'}, + wborstnum => $borr->{streetnumber}, wboraddress => $borr->{'address'}, wboraddress2 => $borr->{'address2'}, wborcity => $borr->{'city'}, @@ -429,6 +430,7 @@ if ( $messages->{'ResFound'}) { boremail => $borr->{'email'}, boraddress => $borr->{'address'}, boraddress2 => $borr->{'address2'}, + borstnum => $borr->{streetnumber}, borcity => $borr->{'city'}, borzip => $borr->{'zipcode'}, borcnum => $borr->{'cardnumber'}, 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 72f9ee367f..5e6704478b 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt @@ -149,7 +149,7 @@ $(document).ready(function () { [% IF ( wborcnum ) %]
Hold for:
  • [% borsurname %], [% borfirstname %] ([% borcnum %])
  • -
  • [% wboraddress %][% IF ( wboraddress2 ) %]
    +
  • [% wborstnum %] [% wboraddress %][% IF ( wboraddress2 ) %]
    [% wboraddress2 %]
    [% END %] [% wborcity %] [% wborzip %]
  • [% IF ( wborphone ) %]
  • [% wborphone %]
  • [% END %] @@ -182,7 +182,7 @@ $(document).ready(function () {

    Hold for:

    • [% borsurname %], [% borfirstname %] ([% borcnum %])
    • -
    • [% boraddress %]
      +
    • [% borstnum %] [% boraddress %]
      [% IF ( boraddress2 ) %][% boraddress2 %]
      [% END %] [% borcity %] [% borzip %]
    • [% IF ( borphone ) %]
    • [% borphone %]
    • [% END %] @@ -223,7 +223,7 @@ $(document).ready(function () {

      Hold for:

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