In OPAC and intra advanced search, more search options are displayed depending on expandedSearchOption syspres and user choice. This doesn't work anymore in HEAD.
Created attachment 6633 [details] [review] Proposed patch In OPAC and intra advanced search, more search options are displayed depending on expandedSearchOption syspref and user choice. This doesn't work anymore in HEAD. This patch restore this functionality. To test: - enable/disable expandedSearchOption. - Load advanced search page. Limited seach options version is always displayed - Apply this patch and see how it works...
Created attachment 6660 [details] [review] [SIGNED-OFF] Bug 7328 Expanded options in advanced search In OPAC and intra advanced search, more search options are displayed depending on expandedSearchOption syspres and user choice. This doesn't work anymore in HEAD. This patch restore this functionnality. To test: - enable/disable expandedSearchOption. - Load advanced search page. Limited seach options version is always displayed - Apply this patch and see how it works... Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> The problem only appeared in the staff client for me. Tested both OPAC and staff with patch applied.
QA comment : I can't see any difference with and without the patch. Frédéric, looking at your code, it seems it does something only if the syspref is not defined, which should not happend. Marking failed QA until I have more information about how to reproduce the problem
> QA comment : Thanks for going through this patch. > I can't see any difference with and without the patch. > Frédéric, looking at your code, it seems it does something only if the > syspref is not defined, which should not happend. I don't think so. Expanded search option are passed with a CGI param, it is used (and controlled), otherwise syspref is used. > Marking failed QA until I have more information about how to reproduce > the problem 1. On HEAD, in intra interface, enable expandedSearchOption syspref 2. Click Search link in the top links bar. 3. Next to Search button, you can see [More options] link. This is wrong. We should have [Fewer options] 4. Commit the patch 5. Repeat 1-2 6. Next to Search button, you have [Fewer options] link. This is correct.
Hi Paul, for me the problem only occured in the staff interface, not int he OPAC. Perhaps this helps.
QA comment: thanks for the explanation Frédéric, it worked now. I suspect I made something wrong in my 1st tests, because I hadn't this behaviour. perlcritic catalogue/search.pl catalogue/search.pl source OK passed QA patch pushed, please test