When one uses characters like é, à, ç ... the first search show up correctly but when you refine your search with "Show more", the encoding becomes bad and so, results of the search too.
A quick fix of Bug 22903 problem
Same fix as commit : Bug 21526: Fix search result pages (url vs uri vs raw)
Created attachment 90673 [details] [review] Bug 23132: Fix encoding issues in facets with show more link When one uses characters like Ã, ç ... the first searcshow up correctly but when you refine your search with "Show more", the encoding becomes bad and so, results of the search too. Test plan : 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
(In reply to Fridolin SOMERS from comment #2) > Same fix as commit : > Bug 21526: Fix search result pages (url vs uri vs raw) I do not understand which part of this patch you are referring to.
(In reply to Jonathan Druart from comment #4) > (In reply to Fridolin SOMERS from comment #2) > > Same fix as commit : > > Bug 21526: Fix search result pages (url vs uri vs raw) > > I do not understand which part of this patch you are referring to. Ok got it. The $raw was used in opac-facets.inc, not facets.inc
Created attachment 90767 [details] [review] Bug 23132: Fix encoding issues in facets with show more link When one uses characters like Ã, ç ... the first searcshow up correctly but when you refine your search with "Show more", the encoding becomes bad and so, results of the search too. Test plan : 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 Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 90788 [details] [review] Bug 23132: Fix encoding issues in facets with show more link When one uses characters like Ã, ç ... the first searcshow up correctly but when you refine your search with "Show more", the encoding becomes bad and so, results of the search too. Test plan : 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 Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Nice work! Pushed to master for 19.11.00
Pushed to 19.05.x for 19.05.02
backported to 18.11.x for 18.11.09