[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Fridolin SOMERS changed: What|Removed |Added Blocks||13265 Referenced Bugs: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13265 [Bug 13265] Still too many search cursor cookies -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to Master CC||tomasco...@gmail.com Blocks|12261 | Depends on||12261 --- Comment #20 from Tomás Cohen Arazi --- Pushed to master. Thanks Juhani! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Jonathan Druart changed: What|Removed |Added See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=12510 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #19 from Juhani Seppälä --- (In reply to Jonathan Druart from comment #18) > (In reply to Juhani Seppälä from comment #16) > > I can create a new bug for this. > > Did you create it? Yes, was about to post it here for Marc as well: Bug 12510(In reply to Jonathan Druart from comment #18) > (In reply to Juhani Seppälä from comment #16) > > I can create a new bug for this. > > Did you create it? Yes, sorry, was about to post it here for Marc as well, see Bug 12510. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Jonathan Druart changed: What|Removed |Added Status|Signed Off |Passed QA --- Comment #18 from Jonathan Druart --- (In reply to Juhani Seppälä from comment #16) > I can create a new bug for this. Did you create it? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Jonathan Druart changed: What|Removed |Added Attachment #29403|0 |1 is obsolete|| --- Comment #17 from Jonathan Druart --- Created attachment 29408 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29408&action=edit 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. http://bugs.koha-community.org/show_bug.cgi?id=12481 This patch prevents the "Next" button on detail view to be grayed out at the end of e result page. Signed-off-by: Marc Véron Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #16 from Juhani Seppälä --- (In reply to Marc Véron from comment #15) > (In reply to Juhani Seppälä from comment #9) > > (...) > > This is not the only interesting behaviour I've come across with the > > navigation. There's another issue where the results of the page get > > concatenated infinitely on page reload, which originally led me to believe > > the results-array in the cookie was the whole results set. These should > > probably be reported separately? > > I propose that you report this separately. Thanks for looking into this Marc and helping with my regression. I should've been more precise there: it's the result array in the search cookie that's getting concatenated needlessly on page reloads - not sure if this can actually cause any harm currently, but the concatenation on page reloads doesn't look intentional in any case. I can create a new bug for this. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #15 from Marc Véron --- (In reply to Juhani Seppälä from comment #9) (...) > This is not the only interesting behaviour I've come across with the > navigation. There's another issue where the results of the page get > concatenated infinitely on page reload, which originally led me to believe > the results-array in the cookie was the whole results set. These should > probably be reported separately? I propose that you report this separately. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Marc Véron changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Marc Véron changed: What|Removed |Added Attachment #29276|0 |1 is obsolete|| --- Comment #14 from Marc Véron --- Created attachment 29403 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29403&action=edit 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. http://bugs.koha-community.org/show_bug.cgi?id=12481 This patch prevents the "Next" button on detail view to be grayed out at the end of e result page. Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Marc Véron changed: What|Removed |Added Attachment #29386|0 |1 is obsolete|| --- Comment #13 from Marc Véron --- Comment on attachment 29386 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29386 Bug 12481 - Follow up: Link to results is not refreshed when item is on next or prev result page Hi Juhani, Strange, I tested again with your patch only and can not longer reproduce the behaviour. I used a result set with about 10 pages. Yesterday, I had two tabs open, in one I had the result list (>10 pages) and in the other, I stepped through the single views andgot the odd behaviour. I obsolete now my follow up patch and will sign-off your patch. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #12 from Juhani Seppälä --- (In reply to Marc Véron from comment #11) > Created attachment 29386 [details] [review] > Bug 12481 - Follow up: Link to results is not refreshed when item is on > next or prev result page > > Test plan for this follow up > > Reproduce issue without patch: > > Do a search that results in 3 or more pages. > Print out list or do the same search in an other browser (to have a > reference for the following) > Go to a detail view and step through details to a detail that ist on result > #3 > Click on Link "Return to details" > Result: The first page will display (instead of page #2) > > With patch, the correct page should display. Hi, I'm unable to reproduce what you're describing with only my patch applied. What I'm doing: 1. Do a search with 3 or more pages of results. 2. Select a result from the first page and navigate into the third page using the detail-view. 3. Select "return to results" on a results on the third page. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #11 from Marc Véron --- Created attachment 29386 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29386&action=edit Bug 12481 - Follow up: Link to results is not refreshed when item is on next or prev result page Test plan for this follow up Reproduce issue without patch: Do a search that results in 3 or more pages. Print out list or do the same search in an other browser (to have a reference for the following) Go to a detail view and step through details to a detail that ist on result #3 Click on Link "Return to details" Result: The first page will display (instead of page #2) With patch, the correct page should display. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Marc Véron changed: What|Removed |Added Attachment #29275|0 |1 is obsolete|| --- Comment #10 from Marc Véron --- Comment on attachment 29275 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29275 Staff client detail-view "next" link is greyed out when the last search result of any results page is clicked or navigated into Obsoleting patch #1 (suppose it is replaced by patch #2) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481
--- Comment #9 from Juhani Seppälä ---
(In reply to Marc Véron from comment #8)
> I suppose that the first patch is obsolete? Seems to be the same code.
>
> I applied the second patch, did a search wit a couple of pages as result,
> went to the first detail from one of the result pages and clicked "Next"
> until the end of this page. Then it seems that I am "trapped" on this result
> page, after the last entry the first entry of the same result page is
> displayed (should be the first entry of the next page).
This is not the only interesting behaviour I've come across with the
navigation. There's another issue where the results of the page get
concatenated infinitely on page reload, which originally led me to believe the
results-array in the cookie was the whole results set. These should probably be
reported separately? The behaviour you're describing should be unrelated to my
patch and I've come across it before doing any changes. Can't be 100%,
obviously.
Related js (which I have not touched):
var browseRecords = function (movement) {
var newSearchPos = me.curPos + movement;
if (newSearchPos > me.searchCookie.results.length - 1) {
window.location = '/cgi-bin/koha/catalogue/search.pl?' +
decodeURIComponent(me.searchCookie.query) + '&limit=' +
decodeURIComponent(me.searchCookie.limit) + '&sort=' + me.searchCookie.sort +
'&gotoPage=detail.pl&gotoNumber=first&searchid=' + me.searchid + '&offset=' +
newSearchPos;
} else if (newSearchPos < 0) {
window.location = '/cgi-bin/koha/catalogue/search.pl?' +
decodeURIComponent(me.searchCookie.query) + '&limit=' +
decodeURIComponent(me.searchCookie.limit) + '&sort=' + me.searchCookie.sort +
'&gotoPage=detail.pl&gotoNumber=last&searchid=' + me.searchid + '&offset=' +
(me.offset - me.searchCookie.pagelen);
} else {
window.location = window.location.href.replace('biblionumber=' +
biblionumber, 'biblionumber=' + me.searchCookie.results[newSearchPos]);
}
}
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Marc Véron changed: What|Removed |Added CC||ve...@veron.ch --- Comment #8 from Marc Véron --- I suppose that the first patch is obsolete? Seems to be the same code. I applied the second patch, did a search wit a couple of pages as result, went to the first detail from one of the result pages and clicked "Next" until the end of this page. Then it seems that I am "trapped" on this result page, after the last entry the first entry of the same result page is displayed (should be the first entry of the next page). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Juhani Seppälä changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Juhani Seppälä changed: What|Removed |Added Attachment #29272|0 |1 is obsolete|| --- Comment #7 from Juhani Seppälä --- Created attachment 29276 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29276&action=edit 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. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #6 from Juhani Seppälä --- Created attachment 29275 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29275&action=edit 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. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Juhani Seppälä changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #5 from Juhani Seppälä --- Newest patch does not apply - does not apply to master, attaching again. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Juhani Seppälä changed: What|Removed |Added Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Juhani Seppälä changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 Juhani Seppälä changed: What|Removed |Added Attachment #29210|0 |1 is obsolete|| --- Comment #4 from Juhani Seppälä --- Created attachment 29272 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29272&action=edit 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. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #3 from Juhani Seppälä --- (In reply to Jonathan Druart from comment #2) > Does this patch needs signoff? > If yes, you missed to change the status :) > And if yes, I will change the status to failed QA, because the patch does > not work if you get more than 1 page of results. > me.curPos is the position of the result on the current page, not the > position on the total number of results. You're right, I'm obviously a bit confused with regards to the fundamental purpose of some of the js I'm looking at. However, think that making the total results available to the navigation js is a starting point. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #2 from Jonathan Druart --- Does this patch needs signoff? If yes, you missed to change the status :) And if yes, I will change the status to failed QA, because the patch does not work if you get more than 1 page of results. me.curPos is the position of the result on the current page, not the position on the total number of results. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [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
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12481 --- Comment #1 from Juhani Seppälä --- Created attachment 29210 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29210&action=edit 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. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
