Summary: | if parentheses used in ccode then searches against ccode fail | ||
---|---|---|---|
Product: | Koha | Reporter: | Reed Wade <reed> |
Component: | Searching | Assignee: | Reed Wade <reed> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, julian.maurice, jwagner, nengard |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22344 | ||
Change sponsored?: | Sponsored | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
patch to wrap mc-ccode search limit in quotes
Follow up fix |
Description
Reed Wade
2011-02-04 03:11:19 UTC
Created attachment 3065 [details] [review] patch to wrap mc-ccode search limit in quotes Work is sponsored by Opus This bug is mentioned in: bug 5693 - mc-ccode parens fix http://lists.koha-community.org/pipermail/koha-patches/2011-February/013766.html Pushed, please test and mark resolved This now ends up with a url like this a href="/cgi-bin/koha/opac-search.pl?idx=kw&q=a&limit=mc-ccode:"AF"&sort_by=relevance&limit=su-geo:Alabama." title="Alabama.">Alabama.</a> With the "AF" breaking it obviously, working on a follow up patch to escape the html where limit_cgi is used in the templates Follow up patch submitted Created attachment 3296 [details] [review] Follow up fix Follow up patch pushed, please test and mark resolved This bug is mentioned in: bug 5693 - mc-ccode parens fix http://lists.koha-community.org/pipermail/koha-patches/2011-March/013985.html Bug 5693 : Follow up patch, escaping HTML in limit_cgi so "" dont break hrefs http://lists.koha-community.org/pipermail/koha-patches/2011-March/014046.html Bug 5693 : Follow up patch, escaping HTML in limit_cgi so "" dont break hrefs http://lists.koha-community.org/pipermail/koha-patches/2011-March/014048.html |