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.
The solution could be to show dynamically the results when you click on "Show more".
Created attachment 89822 [details] [review] Bug 22903 - Elasticsearch, dynamic display when you refine your search To test: 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
Created attachment 89823 [details] [review] Bug 22903: Elasticsearch, dynamic display when you refine your search To test: 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
Created attachment 89829 [details] [review] Bug 22903: Elasticsearch, dynamic display when you refine your search To test: - First you have to apply the patch 22924 Then -> 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
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
Created attachment 89880 [details] [review] Bug 22903: Unit test for sub getRecords in C4/Search.pm
*** This bug has been marked as a duplicate of bug 14419 ***