From 84494966d44d659dcb10e6403bc43945e4a32eab 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] Bug 10856: (follow-up) if callnumbers are equal, order should
 be on itemnumber

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>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---
 C4/ShelfBrowser.pm                                    | 2 +-
 koha-tmpl/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 555ad05342..763b406452 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 c41647a243..f5672f9d25 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>
-- 
2.39.5