Hide the search string "kw,wrdl:" (and it's variants) from the Search Results.
Created attachment 5752 [details] Screenshot of Search Results
Screenshot of Search Results: http://screencast.com/t/kolmH6ha
There are a couple of ways this could be fixed: 1. Don't show it at all in the OPAC 2. Change the text to something else "Your search returned" 3. Use local jquery to do either of the above. Does this need to be a systemic change, or can we handle it with jquery?
+1 I think it's quite hard to do with jQuery right now, because the part is not formatted nicely. If people want to keep it, some reformatting and adding of ids would be good. I think I would prefer a global change here. "Your search returned XX results."
Created attachment 7331 [details] [review] bug 6985 - remove kw,wrdl from the search results To test: * apply patch * do a search in the OPAC, the result page should now say "Your search returned X results" instead of "Your search for "kw,wrdl:yyy" returned x results"
These strings also appear in a patron's search history. Where something more explicit would be better, imho.
Created attachment 7369 [details] [review] Bug 6985 - remove kw,wrdl from the search results To test: * Apply patch * Do a search in the OPAC, the result page should now say "Your search returned X results" instead of "Your search for "kw,wrdl:yyy" returned x results" Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
This works fine when we have a result, but if there is none we do still have a variant of "kw,wrdl" appearing. Example: No results match your search for “kw,wrdl: tsfdfs branch:Bibliothèque de Benquet” in Catalog. Also, like mentionned earlier by Gaetan, the search history displays lines like: Date Search Results 03/02/2012 15:49:18 au,wrdl: test2 0 03/02/2012 15:49:04 se,wrdl: test 0 03/02/2012 15:48:51 nb: test 0 03/02/2012 15:47:09 kw,wrdl: tsfdfs 0 03/02/2012 15:47:01 kw,wrdl: tsfdfs 0 It's a little bit out of this bug scope, though (but shares the same goal of avoiding strange strings displayed to users).
good point. Two shakes and I'll get the "no results" on there too.
Created attachment 7449 [details] [review] Bug 6985 - remove kw,wrdl from the search results to test: apply patch do a search, result found text should not contain kw,wrdl do a search that returns no result. No result text should not contain kw,wrdl.
Oh, looks like this patch only include the last modification, but the 1st was marked as obsolete. I did install this one and now it works with no result but not anymore when I have results. Could you make a patch merging the two modifications, please? Thanks.
Created attachment 7454 [details] [review] Bug 6985 - remove kw,wrdl from the search results To test: * Apply patch * Do a search in the OPAC, the result page should now say "Your search returned X results" instead of "Your search for "kw,wrdl:yyy" returned x results" * Do a search that will have no results, kw,wrdl should not appear in the text. Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7497 Created to address the search history problem.
Created attachment 7468 [details] [review] Bug 6985 - remove kw,wrdl from the search results Bug 6985 - remove kw,wrdl from the search results Signed off.
*** Bug 7209 has been marked as a duplicate of this bug. ***
Removes this 'debugging' kind of information from the OPAC in a template-only change. Marking as Passed QA.
This fix was included in the 3.6.x branch prior to 3.6.4.