Attachment 33604 Details for Bug 8096 – Bug 8096 - card number twice on patron search

[patch] Bug 8096 - card number twice on patron search

Bug-8096---card-number-twice-on-patron-search.patch (text/plain), 4.49 KB, created by Marc Véron on 2014-11-16 22:02:30 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Véron

Created: 2014-11-16 22:02:30 UTC

Size: 4.49 KB

patch

obsolete

>From 89c4b1e70b1191e92452671b7168a7f99ffc7d65 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch>
>Date: Sun, 16 Nov 2014 22:52:31 +0100
>Subject: [PATCH] Bug 8096 - card number twice on patron search
>
>This patch removes double patron card numbers from patron search results.
>
>To test:
>
>Search patrons
>See card number 12345 in first column and in second column like Testman, John (12345)
>Apply patch
>Patron card number no longer dispays in second colum
>
>Further testing:
>Search for other places in intranet where patron-title.inc is used (git grep 'patron-title.inc')
>The card number still should display at the other places.
>---
> koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc | 8 ++++++--
> .../prog/en/modules/members/tables/members_results.tt | 2 +-
> 2 files changed, 7 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc
>index 3a3b90c..f51807e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-title.inc
>@@ -8,7 +8,9 @@
> [%- borrower.firstname %] [% IF borrower.othernames %] ([% borrower.othernames %]) [% END %] [% borrower.surname %]
> [%- END -%]
> [%- END -%]
>- ([% borrower.cardnumber %])
>+ [% IF ( borrower.cardnumber ) %]
>+ ([% borrower.cardnumber %])
>+ [% END %]
> [%- ELSIF ( borrowernumber ) %]
> [%- IF category_type == 'I' %]
> [%- surname %] [% IF othernames %] ([% othernames %]) [% END %]
>@@ -19,5 +21,7 @@
> [%- firstname %] [% IF othernames %] ([% othernames %]) [% END %] [% surname %]
> [%- END %]
> [%- END -%]
>- ([% cardnumber -%])
>+ [% IF ( borrower.cardnumber ) %]
>+ ([% borrower.cardnumber %])
>+ [% END %]
> [%- END -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt
>index e519289..00b02af 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt
>@@ -13,7 +13,7 @@
> "dt_cardnumber":
> "[% data.cardnumber %]",
> "dt_name":
>- "<a href='/cgi-bin/koha/members/moremember.pl?borrowernumber=[% data.borrowernumber %]'>[% INCLUDE 'patron-title.inc' borrowernumber = data.borrowernumber category_type = data.category_type firstname = data.firstname surname = data.surname othernames = data.othernames cardnumber = data.cardnumber invert_name = 1%]</a> [% IF ( data.streetnumber ) %][% data.streetnumber %] [% END %][% IF ( data.streettype ) %][% AuthorisedValues.GetByCode( 'ROADTYPE', data.streettype ) %] [% END %][% data.address %][% IF ( data.address2 ) %] [% data.address2 %][% END %][% IF ( data.city || data.state || data.zipcode || data.country) %] [% END %][% IF ( data.city ) %][% data.city %][% IF ( data.state ) %],[% END %][% END %][% IF ( data.state ) %] [% data.state %][% END %] [% IF ( data.zipcode ) %] [% data.zipcode %][% END %][% IF ( data.country ) %], [% data.country %][% END %][% IF data.email %] Email&#58; <a href='mailto&#58;[% data.email %]'>[% data.email %]</a>[% END %]",
>+ "<a href='/cgi-bin/koha/members/moremember.pl?borrowernumber=[% data.borrowernumber %]'>[% INCLUDE 'patron-title.inc' borrowernumber = data.borrowernumber category_type = data.category_type firstname = data.firstname surname = data.surname othernames = data.othernames invert_name = 1%]</a> [% IF ( data.streetnumber ) %][% data.streetnumber %] [% END %][% IF ( data.streettype ) %][% AuthorisedValues.GetByCode( 'ROADTYPE', data.streettype ) %] [% END %][% data.address %][% IF ( data.address2 ) %] [% data.address2 %][% END %][% IF ( data.city || data.state || data.zipcode || data.country) %] [% END %][% IF ( data.city ) %][% data.city %][% IF ( data.state ) %],[% END %][% END %][% IF ( data.state ) %] [% data.state %][% END %] [% IF ( data.zipcode ) %] [% data.zipcode %][% END %][% IF ( data.country ) %], [% data.country %][% END %][% IF data.email %] Email&#58; <a href='mailto&#58;[% data.email %]'>[% data.email %]</a>[% END %]",
> "dt_category":
> "[% data.category_description |html %] ([% data.category_type |html %])",
> "dt_branch":
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 8096: 9597 | 33604 | 33626 | 34608 | 34721