Attachment 16161 Details for Bug 9780 – [PATCH 2] Bug 9780: Don't show comma at the end of the line

[patch] [PATCH 2] Bug 9780: Don't show comma at the end of the line

0006-DontShowComma.patch (text/plain), 3.11 KB, created by Mathieu Saby on 2013-03-15 11:59:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mathieu Saby

Created: 2013-03-15 11:59:55 UTC

Size: 3.11 KB

patch

obsolete

>From 29f23dbd7fec17d5d62586785d8ca3eec532f5a5 Mon Sep 17 00:00:00 2001
>From: Mathieu Saby <mathieu.saby@univ-rennes2.fr>
>Date: Fri, 15 Mar 2013 12:55:31 +0100
>Subject: [PATCH 2] Bug 9780: Don't show comma at the end of the line
>
>Little improvement :
>This patch uses loop.last in detail.tt template to suppress useless comma.
>
>---
> .../prog/en/modules/catalogue/detail.tt            |   24 +++++++++++++-------
> 1 file changed, 16 insertions(+), 8 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>index bcf56a4..927d922 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>@@ -178,9 +178,11 @@ function verify_images() {
>                         [% countorders %] active order(s) in basket(s)
> 			[% FOREACH basketo IN basketsorders %]
> 			   [% IF ( CAN_user_acquisition_order_manage ) %]
>-			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketo %]">#[% basketo %]</a>,
>+			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketo %]">#[% basketo %]</a>
>+			    [% UNLESS loop.last %],[% END %]		    
> 			   [% ELSE %]
>-			    [% basketo %],
>+			    [% basketo %]
>+			    [% UNLESS loop.last %],[% END %]
> 			   [% END %]
> 			[% END %]
> 		    [% END %]
>@@ -191,9 +193,11 @@ function verify_images() {
>                         [% countdeletedorders %] deleted order(s) in basket(s)
> 			[% FOREACH basketdo IN basketsdeletedorders %]
> 			   [% IF ( CAN_user_acquisition_order_manage ) %]
>-			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketdo %]">#[% basketdo %]</a>,
>+			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketdo %]">#[% basketdo %]</a>
>+			    [% UNLESS loop.last %],[% END %]
> 			   [% ELSE %]
>-			    [% basketdo %],
>+			    [% basketdo %]
>+			    [% UNLESS loop.last %],[% END %]
> 			   [% END %]
> 			[% END %]
> 		    [% END %]
>@@ -340,9 +344,11 @@ function verify_images() {
>                         [% countorders %] active order(s) in basket(s)
> 			[% FOREACH basketo IN basketsorders %]
> 			   [% IF ( CAN_user_acquisition_order_manage ) %]
>-			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketo %]">#[% basketo %]</a>,
>+			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketo %]">#[% basketo %]</a>
>+			    [% UNLESS loop.last %],[% END %]
> 			   [% ELSE %]
>-			    [% basketo %],
>+			    [% basketo %]
>+			    [% UNLESS loop.last %],[% END %]
> 			   [% END %]
> 			[% END %]
> 		    [% END %]
>@@ -353,9 +359,11 @@ function verify_images() {
>                         [% countdeletedorders %] deleted order(s) in basket(s)
> 			[% FOREACH basketdo IN basketsdeletedorders %]
> 			   [% IF ( CAN_user_acquisition_order_manage ) %]
>-			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketdo %]">#[% basketdo %]</a>,
>+			    <a href="/cgi-bin/koha/acqui/basket.pl?basketno=[% basketdo %]">#[% basketdo %]</a>
>+			    [% UNLESS loop.last %],[% END %]
> 			   [% ELSE %]
>-			    [% basketdo %],
>+			    [% basketdo %]
>+			    [% UNLESS loop.last %],[% END %]
> 			   [% END %]
> 			[% END %]
> 		    [% END %]
>-- 
>1.7.9.5
>

Actions: View | Diff | Splinter Review

Attachments on bug 9780: 16008 | 16009 | 16161 | 16164 | 16167 | 16233 | 16305 | 16774 | 17390 | 21254 | 22128 | 22168