Bug 28074

Summary:	Browse controls on staff detail pages are sometimes weird
Product:	Koha	Reporter:	Lucas Gass (lukeg) <lucas>
Component:	Searching	Assignee:	Lucas Gass (lukeg) <lucas>
Status:	CLOSED FIXED	QA Contact:	Testopia <testopia>
Severity:	normal
Priority:	P5 - low	CC:	andrew, fridolin.somers, victor
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	Small patch
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:	21.05.00,20.11.05,20.05.11
Circulation function:
Attachments:	Bug 28074: Make current_search.offset an integer instead of string Bug 28074: Make current_search.offset an integer instead of string Bug 28074: Make current_search.offset an integer instead of string

Description Lucas Gass (lukeg) 2021-04-01 22:49:03 UTC

The browse control feature doesn't work quite right in the OPAC detail. 

1. If you do a search with 3-4 results, go to the record and use the browse controls, when you get to the last record if you attempt to go to the 'next' you'll end up on a page which says 'record not found'.

2. Do a search with exactly 10 results. Click on the first record, the 'next' arrow is not a link, you cannot move to the next result.

This is because in js/browser.js, if you look at the current_search Object and specifically at current_search.offset you will see that current_search.offset is a string.

We try to do calculations with that variable and JavaScript performs string concatenation instead of addtion. 

For example:

current_search.offset + me.curPos == current_search.total

When the offset is '1' and we add it to 0, the result is 10. Therefore if the offset is the first and the total is 10, it thinks we are on the last result (10) and doesn't let use move forward. 


Converting current_search.offset to an integer solves the problem.

Comment 1 Lucas Gass (lukeg) 2021-04-01 22:53:18 UTC

When I said OPAC detail, I meant staff detail

Comment 2 Lucas Gass (lukeg) 2021-04-01 22:54:05 UTC

Created attachment 119090 [details] [review]
Bug 28074: Make current_search.offset an integer instead of string

To test:
1. do a search with 3-4 results, go to the record and use the browse controls, when you get to the last record if you attempt to go to the 'next' you'll end up on a page which says 'record not found'.2. Do a search with exactly 10 results. Click on the first record, the 'next' arrow is not a link, you cannot move to the next result
3. Apply patch
4. repeat steps 1-2 and the problems should be gone

Comment 3 Owen Leonard 2021-04-02 13:22:15 UTC

Created attachment 119122 [details] [review]
Bug 28074: Make current_search.offset an integer instead of string

To test:
1. Do a search with 3-4 results, go to the record and use the browse
   controls, when you get to the last record if you attempt to go to the
   'next' you'll end up on a page which says 'record not found.'
2. Do a search with exactly 10 results. Click on the first record, the
   'next' arrow is not a link, you cannot move to the next result.
3. Apply patch
4. Repeat steps 1-2 and the problems should be gone.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Comment 4 Katrin Fischer 2021-04-03 14:06:27 UTC

Created attachment 119154 [details] [review]
Bug 28074: Make current_search.offset an integer instead of string

To test:
1. Do a search with 3-4 results, go to the record and use the browse
   controls, when you get to the last record if you attempt to go to the
   'next' you'll end up on a page which says 'record not found.'
2. Do a search with exactly 10 results. Click on the first record, the
   'next' arrow is not a link, you cannot move to the next result.
3. Apply patch
4. Repeat steps 1-2 and the problems should be gone.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Comment 5 Jonathan Druart 2021-04-06 14:00:03 UTC

Pushed to master for 21.05, thanks to everybody involved!

Comment 6 Fridolin Somers 2021-04-15 10:43:27 UTC

Pushed to 20.11.x for 20.11.05

Comment 7 Andrew Fuerste-Henry 2021-04-24 17:01:57 UTC

Pushed to 20.05.x for 20.05.11

Comment 8 Victor Grousset/tuxayo 2021-04-24 17:19:38 UTC

Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.