Bug 28074

Summary: Browse controls on staff detail pages are sometimes weird
Product: Koha Reporter: Lucas Gass <lucas>
Component: SearchingAssignee: Lucas Gass <lucas>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: andrewfh, fridolin.somers, victor
Version: master   
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
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 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 2021-04-01 22:53:18 UTC
When I said OPAC detail, I meant staff detail
Comment 2 Lucas Gass 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.