From b1d76c5d08af71c06cc03cfa06d46b46959dcbcf Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 15 Oct 2020 10:47:55 +0200 Subject: [PATCH] Bug 26445: Fix "back to result" link of the search result browser Not sure what was the expected behaviour, I am suggesting the following one: If you are browsing and you click "back to result", you will be redirected to the page where the record appear on the search result list. Say you search for "something" that returns 10 page (20 results per page) Click on first result first page Click back to result => You see the first page with the first result at the top Click on second page, 3rd result Click back to result => You see the second page with the biblio you clicked at the 3rd position Click on second page, 3rd result Click next/previous Click back to result => You see the page where you last result were. I think the existing expected behaviour was to have the result you clicked at the first position of the result page, but I am not sure it really makes sense. Hope this makes sense however! Signed-off-by: Owen Leonard Signed-off-by: Martin Renvoize Signed-off-by: Jonathan Druart (cherry picked from commit 3edbe7d7dc9358e55b4ecc9da5b292a32cadcce9) Signed-off-by: Lucas Gass --- koha-tmpl/intranet-tmpl/js/browser.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/js/browser.js b/koha-tmpl/intranet-tmpl/js/browser.js index 5ad0ed97ab..2b9ee2f9db 100644 --- a/koha-tmpl/intranet-tmpl/js/browser.js +++ b/koha-tmpl/intranet-tmpl/js/browser.js @@ -73,7 +73,11 @@ KOHA.browser = function (searchid, biblionumber) { me.show = function () { if (current_search) { me.curPos = $.inArray(biblionumber, current_search.results); - me.offset = Math.floor((current_search.offset + me.curPos - 1) / current_search.pagelen) * current_search.pagelen; + if ( current_search.offset + me.curPos <= current_search.pagelen ) { // First page + me.offset = 0; + } else { + me.offset = current_search.offset - 1; + } $(document).ready(function () { if (me.curPos > -1) { -- 2.39.5