From 892034aa8142af0b89d57ca7c8ef3717eae07ea3 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Thu, 7 Mar 2013 13:30:27 -0500
Subject: [PATCH] Bug 9458 - Add sorting to lists - QA Followup 4

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
(cherry picked from commit f0c58df12e7bb7e9a42abf52cc113dcb85c8002f)

Solved Conflicts:
	koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
(cherry picked form commit f0c58df12e7bb7e9a42abf52cc113dcb85c8002f)
---
 .../prog/en/modules/virtualshelves/shelves.tt | 30 +++++++++----------
 1 file changed, 14 insertions(+), 16 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
index 746441cc59..06d7395387 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
@@ -204,7 +204,13 @@ function placeHold () {
      <span class="clearall"></span></p>
  [% END %]
 	
- [% IF direction == 'asc' %][% SET new_direction = 'desc' %][% ELSE %][% SET new_direction = 'asc' %][% END %]
+ [% IF direction == 'asc' %]
+     [% SET new_direction = 'desc' %]
+ [% ELSE %]
+     [% SET direction = 'desc' %]
+     [% SET new_direction = 'asc' %]
+ [% END %]
+
  <div class="pages">[% pagination_bar %]</div>
     <table>
             <tr>
@@ -212,28 +218,20 @@ function placeHold () {
 
                 [% UNLESS ( item_level_itypes ) %]<th>Item type</th>[% END %]
                 <th>
-                    <a href="/cgi-bin/koha/virtualshelves/shelves.pl?viewshelf=[% shelfnumber %]&sort=title&direction=[% new_direction %]">Title</a>
+                    <a href="/cgi-bin/koha/virtualshelves/shelves.pl?viewshelf=[% shelfnumber %]&sort=title&direction=[% IF sort != 'title' %]asc[% ELSE %][% new_direction %][% END %]">Title</a>
                     [% IF sort == 'title' %]
-                        [% IF direction == 'asc' %]
-                            <img src="/intranet-tmpl/prog/img/asc.gif" />
-                        [% ELSIF direction == 'desc' %]
-                            <img src="/intranet-tmpl/prog/img/desc.gif" />
-                        [% END %]
+                        <img src="/intranet-tmpl/prog/img/[% direction %].gif" />
                     [% ELSE %]
                         <img src="/intranet-tmpl/prog/img/ascdesc.gif" />
                     [% END %]
                 </th>
                 <th>
-                    <a href="/cgi-bin/koha/virtualshelves/shelves.pl?viewshelf=[% shelfnumber %]&sort=author&direction=[% new_direction %]">Author</a>
+                    <a href="/cgi-bin/koha/virtualshelves/shelves.pl?viewshelf=[% shelfnumber %]&sort=author&direction=[% IF sort != 'author' %]asc[% ELSE %][% new_direction %][% END %]">Author</a>
                    [% IF sort == 'author' %]
-                        [% IF direction == 'asc' %]
-                            <img src="/intranet-tmpl/prog/img/asc.gif" />
-                        [% ELSIF direction == 'desc' %]
-                            <img src="/intranet-tmpl/prog/img/desc.gif" />
-                        [% END %]
-                    [% ELSE %]
-                        <img src="/intranet-tmpl/prog/img/ascdesc.gif" />
-                    [% END %]
+                        <img src="/intranet-tmpl/prog/img/[% direction %].gif" />
+                   [% ELSE %]
+                       <img src="/intranet-tmpl/prog/img/ascdesc.gif" />
+                   [% END %]
                  </th>
                 <th>Date added</th>
                 <th>&nbsp;</th>