Attachment 29210 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.96 KB, created by Juhani Seppälä on 2014-06-25 10:51:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Juhani Seppälä

Created: 2014-06-25 10:51:32 UTC

Size: 3.96 KB

patch

obsolete

>From c5fe00977661609e6106ce28ecf8a7d77dae0713 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Juhani=20Sepp=C3=A4l=C3=A4?= <jseppal@student.uef.fi>
>Date: Wed, 25 Jun 2014 13:39:46 +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.
>
>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                        | 7 ++++---
> koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt | 3 ++-
> 2 files changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/js/browser.js b/koha-tmpl/intranet-tmpl/js/browser.js
>index 4d0fc36..f0bd106 100644
>--- a/koha-tmpl/intranet-tmpl/js/browser.js
>+++ b/koha-tmpl/intranet-tmpl/js/browser.js
>@@ -25,7 +25,7 @@ KOHA.browser = function (searchid, biblionumber) {
>         }
>     }
> 
>-    this.create = function (offset, query, limit, sort, newresults) {
>+    this.create = function (offset, query, limit, sort, newresults, total) {
>         if (me.searchCookie) {
>             if (offset === me.searchCookie.offset - newresults.length) {
>                 me.searchCookie.results = newresults.concat(me.searchCookie.results);
>@@ -41,7 +41,8 @@ KOHA.browser = function (searchid, biblionumber) {
>                 limit: limit,
>                 sort:  sort,
>                 pagelen: newresults.length,
>-                results: newresults
>+                results: newresults,
>+                total: total
>             };
> 
>             //Bug_11369 Cleaning up excess searchCookies to prevent cookie overflow in the browser memory.
>@@ -81,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.pagelen - 1) {
>+                    if (me.curPos === me.searchCookie.total - 1) {
>                         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 21c2866..6538f33 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
>@@ -139,7 +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);
>+        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