Bug 22903

Summary: Elasticsearch, dynamic display when you refine your search
Product: Koha Reporter: axel Amghar <axel.amghar>
Component: Searching - ElasticsearchAssignee: axel Amghar <axel.amghar>
Status: RESOLVED DUPLICATE QA Contact:
Severity: enhancement    
Priority: P5 - low CC: e.betemps, fridolin.somers, nick
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14419
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23132
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 22924    
Bug Blocks:    
Attachments: Bug 22903 - Elasticsearch, dynamic display when you refine your search
Bug 22903: Elasticsearch, dynamic display when you refine your search
Bug 22903: Elasticsearch, dynamic display when you refine your search
Bug 22903: Elasticsearch, dynamic display when you refine your search
Bug 22903: Unit test for sub getRecords in C4/Search.pm

Description axel Amghar 2019-05-14 12:36:36 UTC
With elastic search when you use characters like é, à, ç ... the first reasearch show up correctly but when you refine your search with "Show more", the encoding becomes bad and so, results of the reasearch too.
Comment 1 axel Amghar 2019-05-15 11:50:14 UTC
The solution could be to show dynamically the results when you click on "Show more".
Comment 2 axel Amghar 2019-05-16 09:16:29 UTC Comment hidden (obsolete)
Comment 3 axel Amghar 2019-05-16 09:19:24 UTC Comment hidden (obsolete)
Comment 4 axel Amghar 2019-05-16 11:30:14 UTC Comment hidden (obsolete)
Comment 5 axel Amghar 2019-05-16 14:34:05 UTC
Created attachment 89850 [details] [review]
Bug 22903: Elasticsearch, dynamic display when you refine your search

To test:

With Elasticsearch->
Without the patch:
- check if you have your SysPref SearchEngine with Elasticsearch as value,
- put your SysPref FacetMaxCount at 10,
- search in the catalog "a" and then in Refine your search click on show more (just to see how it works),
- then search with special characters like "é", ç" ... and make the same test, the research should be wrong when you click on show more and the page is realoaded.

Then apply the patch:
- and make the same with a classic reasearch, then with special characters,
- verify that when you click on show more the page isn't realoded and the reasearch is correct.
- same with show less
- and finnaly, add some facets and test show more/less

With Zerbra ->
- put your SysPref SearchEngine with Zebra as value,
- and make the same thing as elasticsearch
Comment 6 axel Amghar 2019-05-17 07:42:56 UTC
Created attachment 89880 [details] [review]
Bug 22903: Unit test for sub getRecords in C4/Search.pm
Comment 7 Nick Clemens 2019-07-05 20:26:24 UTC

*** This bug has been marked as a duplicate of bug 14419 ***