From e9357994a2b1910b17411fef16352bf4b94ba49c Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@biblibre.com>
Date: Fri, 13 Sep 2013 09:55:51 +0200
Subject: [PATCH] [PASSED QA] Bug 10856: If callnumbers are equals, order
 should be on itemnumbers

If a lot of items has the same callnumber, the order should be on the
itemnumbers. Otherwise the left side is always filled with the same
items.

+ Fix a bad c/p for the next link (when js is disabled).

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
---
 C4/ShelfBrowser.pm                                 |    2 +-
 .../opac-tmpl/prog/en/includes/shelfbrowser.inc    |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/C4/ShelfBrowser.pm b/C4/ShelfBrowser.pm
index 555ad05..763b406 100644
--- a/C4/ShelfBrowser.pm
+++ b/C4/ShelfBrowser.pm
@@ -171,7 +171,7 @@ sub GetNearbyItems {
 
     my @prev_items = @{
         $dbh->selectall_arrayref(
-            $prev_query . $query_cond . ' ORDER BY cn_sort DESC, itemnumber LIMIT ?',
+            $prev_query . $query_cond . ' ORDER BY cn_sort DESC, itemnumber DESC LIMIT ?',
             { Slice => {} },
             ( @params, $gap )
         )
diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc b/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc
index c41647a..f5672f9 100644
--- a/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc
+++ b/koha-tmpl/opac-tmpl/prog/en/includes/shelfbrowser.inc
@@ -67,7 +67,7 @@
                 <td rowspan="2" style="width:20px;">
                   [% IF shelfbrowser_next_item %]
                     <div id="browser_next">
-                        <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% shelfbrowser_prev_item.biblionumber %]&amp;shelfbrowse_itemnumber=[% shelfbrowser_prev_item.itemnumber %]#shelfbrowser">Next</a>
+                        <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% shelfbrowser_next_item.biblionumber %]&amp;shelfbrowse_itemnumber=[% shelfbrowser_next_item.itemnumber %]#shelfbrowser">Next</a>
                     </div>
                   [% END %]
                 </td>
-- 
1.7.9.5