opac-shevles - fix for CSS alternating highlight lines

Signed-off-by: Joshua Ferraro <jmf@liblime.com>

diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-shelves.tmpl b/koha-tmpl/opac-tmpl/prog/en/modules/opac-shelves.tmpl
index f0692b9..66c670e 100644 (file)
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-shelves.tmpl
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-shelves.tmpl
@@ -157,7 +157,9 @@
         <table>
         <tr><th>List Name</th><th>Contents</th></tr>
             <!-- TMPL_LOOP NAME="shelvesloop" -->
-                <!-- TMPL_IF NAME"toggle" --> <tr class="highlight"><!--TMPL_ELSE--><tr><!-- /TMPL_IF --><td><a href="opac-shelves.pl?viewshelf=<!-- TMPL_VAR NAME="shelf" -->"><!-- TMPL_VAR NAME="shelfname" --></a></td><td><!-- TMPL_VAR NAME="shelfvirtualcount" --> item(s)</td></tr>
+                <!-- TMPL_IF NAME="toggle" --><tr class="highlight"><!--TMPL_ELSE--><tr><!-- /TMPL_IF -->
+               <td><a href="opac-shelves.pl?viewshelf=<!-- TMPL_VAR NAME="shelf" -->"><!-- TMPL_VAR NAME="shelfname" --></a></td>
+               <td><!-- TMPL_VAR NAME="shelfvirtualcount" --> item(s)</td></tr>
             <!-- /TMPL_LOOP -->
         </table>
         <!-- TMPL_ELSE -->

diff --git a/opac/opac-shelves.pl b/opac/opac-shelves.pl
index d618c52..d92823d 100755 (executable)
--- a/opac/opac-shelves.pl
+++ b/opac/opac-shelves.pl
@@ -98,6 +98,7 @@ if ( $query->param('modifyshelfcontents') ) {
         AddToShelf( $biblio->{'biblionumber'}, $shelfnumber );
         foreach ( $query->param ) {
             /REM-(\d+)/ or next;
+                       $debug and warn "SHELVES: user $loggedinuser removing item $1 from shelf $shelfnumber";
             DelFromShelf( $1, $shelfnumber );  # $1 is biblionumber
         }
     }
@@ -150,6 +151,10 @@ SWITCH: {
         if ( ShelfPossibleAction( $loggedinuser, $shelfnumber, 'view' ) ) {
             my $items = GetShelfContents($shelfnumber);
                        $showadd = 1;
+                       my $i = 0;
+                       foreach (grep {$i++ % 2} @$items) {             # every other item
+                               $_->{toggle} = 1;
+                       }
             $template->param(
                 shelfname   => $shelflist->{$shelfnumber}->{'shelfname'},
                 shelfnumber => $shelfnumber,
@@ -208,7 +213,7 @@ my @shelveslooppriv;
 foreach my $element (sort { lc($shelflist->{$a}->{'shelfname'}) cmp lc($shelflist->{$b}->{'shelfname'}) } keys %$shelflist) {
        my %line;
        $color = ($color) ? 0 : 1;
-       $line{'toggle'} = $color;
+       $color and $line{'toggle'} = $color;
        $line{'shelf'} = $element;
        $line{'shelfname'} = $shelflist->{$element}->{'shelfname'};
        $line{'sortfield'} = $shelflist->{$element}->{'sortfield'};