Bug 24371

Summary: OPAC 'Showing only available items/Show all items' is double encoded
Product: Koha Reporter: Lucas Gass <lucas>
Component: OPACAssignee: Lucas Gass <lucas>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: 1joynelson, george, hayleypelham, jonathan.druart, rkuiper
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.03, 19.05.08, 18.11.14
Bug Depends on: 13618    
Bug Blocks:    
Attachments: Bug 24371: Change URI filter to $raw
Bug 24371: Change URI filter to $raw
Bug 24371: Fix "Show all items" avaibility link (use $raw filter)
Bug 24371: Fix "Show all items" avaibility link (use $raw filter) - intra

Description Lucas Gass 2020-01-07 18:52:58 UTC
The show all items link is double encoded when attempt to limit a seach via the facets and toggle 'Showing only available items/Show all items'  

To Recreate:
1. Make an OPAC search with plenty of results.
2. Use the search facets to limit the search in some way, item type, author, etc.
3. Click on 'Limit to currently available items', everything is still fine.
4. Attempt to return to 'Show all items', there will be no search results.


The ':' is being double encoded here. (%253A). I think...
Comment 1 Lucas Gass 2020-01-07 18:57:29 UTC
Created attachment 96953 [details] [review]
Bug 24371: Change URI filter to $raw

TO TEST:
1. Make an OPAC search with plenty of results.
2. Use the search facets to limit the search in some way, item type, author, etc.
3. Click on 'Limit to currently available items', everything is still fine.
4. Attempt to return to 'Show all items', there will be no search results.
5. Apply patch
6. Ateempt steps 1-4 again.
7. Should work this time.
Comment 2 ByWater Sandboxes 2020-01-08 23:13:44 UTC
Created attachment 97060 [details] [review]
Bug 24371: Change URI filter to $raw

Signed-off-by: Rhonda Kuiper <rkuiper@roundrocktexas.gov>
Comment 3 Jonathan Druart 2020-01-10 08:57:08 UTC
Created attachment 97164 [details] [review]
Bug 24371: Fix "Show all items" avaibility link (use $raw filter)

TO TEST:
1. Make an OPAC search with plenty of results.
2. Use the search facets to limit the search in some way, item type, author, etc.
3. Click on 'Limit to currently available items', everything is still fine.
4. Attempt to return to 'Show all items', there will be no search results.
5. Apply patch
6. Ateempt steps 1-4 again.
7. Should work this time.

Signed-off-by: Rhonda Kuiper <rkuiper@roundrocktexas.gov>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2020-01-10 08:57:11 UTC
Created attachment 97165 [details] [review]
Bug 24371: Fix "Show all items" avaibility link (use $raw filter) - intra

Same fix for the intranet side

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Martin Renvoize 2020-01-10 10:51:26 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 6 Joy Nelson 2020-01-30 23:57:06 UTC
Pushed to 19.11.x branch for 19.11.03
Comment 7 Lucas Gass 2020-02-05 22:05:28 UTC
backported to 19.05.x for 19.05.08
Comment 8 Hayley Pelham 2020-02-10 03:56:11 UTC
Backported to 18.11.x for 18.11.14.