Attachment 36266 Details for Bug 13657 – Bug 13657 - Don't show extra comma after guarantor name on patron details

[patch] Bug 13657 - Don't show extra comma after guarantor name on patron details

Bug-13657---Dont-show-extra-comma-after-guarantor-.patch (text/plain), 1.65 KB, created by Jesse Maseto on 2015-03-02 09:43:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jesse Maseto

Created: 2015-03-02 09:43:59 UTC

Size: 1.65 KB

patch

obsolete

>From 1e302947d6b8ef38fe39cd161824ff390a25084b Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Marc=20V=C3=A9ron?= <veron@veron.ch>
>Date: Tue, 10 Feb 2015 16:37:41 +0100
>Subject: [PATCH] Bug 13657 - Don't show extra comma after guarantor name on patron details
>
>This patch suppresses the first name part (including comma) if first name is empty. This way, first name is still displays with natural persons, but not with organizations.
>
>To test:
>Apply this patch (without the first patch).
>Have one patron with a natural person as guarantor and one patron with an organization as guarantor. Verify that both display properly on the patron details page.
>---
> .../prog/en/modules/members/moremember.tt          |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>index 3b89653..d01573b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>@@ -228,7 +228,7 @@ function validate1(date) {
>         [% END %]
>     [% ELSE %]
>         [% IF ( guarantorborrowernumber ) %]
>-            <li><span class="label">Guarantor:</span><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% guarantorborrowernumber %]">[% guarantorsurname %], [% guarantorfirstname %]</a></li>
>+            <li><span class="label">Guarantor:</span><a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% guarantorborrowernumber %]">[% guarantorsurname %][%IF ( guarantorfirstname ) %], [% guarantorfirstname %] [% END %]</a></li>
>         [% END %]
>     [% END %]
> </ol>
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 13657: 35775 | 35805 | 36266 | 36548