From: Garry Collum <gcollum@gmail.com>
Date: Thu, 11 Jun 2009 21:38:59 +0000 (-0400)
Subject: Bug 2889: Removed unused toggle variables from booksellers.pl. Added table highlighti... 
X-Git-Tag: v3.02.00-alpha~473
X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=commitdiff_plain;h=4cd469fe2ed69207514d556c3662af2e59a381ae;p=koha-ffzg.git

Bug 2889: Removed unused toggle variables from booksellers.pl. Added table highlighting to booksellers.tmpl.

Removed unused toggle and line(even) variables from booksellers.pl.  Added highlighting to the two display tables in booksellers.tmpl.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
---

diff --git a/acqui/booksellers.pl b/acqui/booksellers.pl
index 24677ec458..afda2a8d90 100755
--- a/acqui/booksellers.pl
+++ b/acqui/booksellers.pl
@@ -98,19 +98,12 @@ if ($query->param('op') eq 'close') {
 }
 
 #build result page
-my $toggle = 0;
 my @loop_suppliers;
 for ( my $i = 0 ; $i < $count ; $i++ ) {
     my $orders  = GetPendingOrders( $suppliers[$i]->{'id'}, "grouped" );
     my $ordcount = scalar @$orders;
     my %line;
-    if ( $toggle == 0 ) {
-        $line{even} = 1;
-        $toggle = 1;
-    } else {
-        $line{even} = 0;
-        $toggle = 0;
-    }
+
     $line{supplierid} = $suppliers[$i]->{'id'};
     $line{name}       = $suppliers[$i]->{'name'};
     $line{active}     = $suppliers[$i]->{'active'};
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tmpl
index 3a40109ac6..54d5508a32 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tmpl
@@ -30,7 +30,11 @@
 	</tr>
 	
 	<!-- TMPL_LOOP name="loop_suppliers" -->
+	<!-- TMPL_UNLESS name="__odd__" -->
+		<tr class="highlight">
+	<!-- TMPL_ELSE -->
 		<tr>
+	<!-- /TMPL_UNLESS -->
 			<td>	<!-- TMPL_IF name="active" -->
 					<a href="basket.pl?supplierid=<!-- TMPL_VAR name="supplierid" -->">Add order</a>
 				<!-- TMPL_ELSE -->
@@ -54,7 +58,11 @@
 							<th>&nbsp;</th>
 						</tr>
 						<!-- TMPL_LOOP NAME="loop_basket" -->
+						<!-- TMPL_UNLESS NAME="__odd__" -->
+							<tr class="highlight">
+						<!-- TMPL_ELSE -->
 							<tr>
+						<!-- /TMPL_UNLESS -->
 								<td><!-- TMPL_VAR NAME="basketno" --></td>
 								<td><TMPL_VAR NAME="total"></td>
 								<td>						<!-- TMPL_VAR name="firstname" -->