Attachment 29272 Details for Bug 12481 – Bug 12481: Staff client detail-view "next" link is greyed out when the last search result of any results page is clicked or navigated into

[patch] Bug 12481: Staff client detail-view "next" link is greyed out when the last search result of any results page is clicked or navigated into

Bug-12481-Staff-client-detail-view-next-link-is-gr.patch (text/plain), 3.46 KB, created by Juhani Seppälä on 2014-06-26 07:55:49 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Juhani Seppälä

Created: 2014-06-26 07:55:49 UTC

Size: 3.46 KB

patch

obsolete

>From c200ffdf9bfa690c6a1124312831afe945bc0327 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Juhani=20Sepp=C3=A4l=C3=A4?= <jseppal@student.uef.fi>
>Date: Thu, 26 Jun 2014 10:42:00 +0300
>Subject: [PATCH] Bug 12481: Staff client detail-view "next" link is greyed out
> when the last search result of any results page is clicked or navigated into
>
>When doing a staff client catalog search with more than 1 page of results
>and clicking the last result on any search result page, the top-left navigation
>button for "next" is greyed out.
>
>Tested on newest Firefox & Chromium. Attempts to restore originally planned checks for navigation
>with the exception of not using results.length due to buggy behaviour where the results get concatenated
>upon page reload & "return to results" navigation - a separate issue?
>
>To test:
>1) Do an intranet catalog search that has more than 1 page of results.
>2) Click on the last result on a page that is not the last one & confirm that the navigation button "next" is greyed out.
>4) Apply patch.
>5) Do an intranet search with more than 1 but less results than fit on a single page.
>6) Click on the last result on the page and confirm that the "Next" navigation button is greyed out.
>6) Do an intranet search with more than 1 page of results.
>7) Click on the last result of a page that's not the final page of the results & confirm that "Next" is not grey out.
>8) Navigate to the last page of the results and click on the final result & confirm that "Next" is greyd out.
>---
> koha-tmpl/intranet-tmpl/js/browser.js                        | 2 +-
> koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/js/browser.js b/koha-tmpl/intranet-tmpl/js/browser.js
>index f0bd106..b0676a5 100644
>--- a/koha-tmpl/intranet-tmpl/js/browser.js
>+++ b/koha-tmpl/intranet-tmpl/js/browser.js
>@@ -82,7 +82,7 @@ KOHA.browser = function (searchid, biblionumber) {
>                     } else {
>                         prevbutton = '<a href="#" id="browse-previous" class="browse-button">Â« ' + BROWSER_PREVIOUS + '</a>';
>                     }
>-                    if (me.curPos === me.searchCookie.total - 1) {
>+                    if (me.searchCookie.offset + me.curPos == me.searchCookie.total) {
>                         nextbutton = '<span id="browse-next" class="browse-button">' + BROWSER_NEXT + ' Â»</span>';
>                     } else {
>                         nextbutton = '<a href="#" id="browse-next" class="browse-button">' + BROWSER_NEXT + ' Â»</a>';
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
>index 6538f33..8065580 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
>@@ -139,8 +139,8 @@ $('#sort_by').change(function() {
>             [%- END -%]
>         ];
>         var browser = KOHA.browser('[% searchid %]', parseInt('[% biblionumber %]', 10));
>-        browser.create([% SEARCH_RESULTS.first.result_number %], '[% query_cgi | uri %]',
>-                        '[% limit_cgi | uri %]','[% sort_cgi | uri %]', newresults, '[% total %]');
>+        browser.create([% SEARCH_RESULTS.first.result_number %], '[% query_cgi | uri %]', '[% limit_cgi | uri %]','[% sort_cgi | uri %]',
>+                        newresults, '[% total %]');
>     [% END %]
>     [% IF (gotoPage && gotoNumber) %]
>         [% IF (gotoNumber == 'first') %]
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 12481: 29210 | 29272 | 29275 | 29276 | 29386 | 29403 | 29408