]> git.koha-community.org Git - koha.git/commit
Bug 10808: make authority search form retain drop-down selections
authorFridolyn SOMERS <fridolyn.somers@biblibre.com>
Fri, 30 Aug 2013 08:54:00 +0000 (10:54 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 14 Jan 2014 15:11:17 +0000 (12:11 -0300)
commit935a9a6a7e7301f885e7a7ce42482264616c468f
treeb6b6f0d9097cd6c95ec38601868bfaebe1a9883f
parentf3c8be69c9dfa7a4a5eac9e30e980299cfc0e650
Bug 10808: make authority search form retain drop-down selections

When cataloging a field defined with a thesaurus, an authority search
popup is displayed with a search from. Once operators, values and sort
selected in this form the search can be performed. The bug is that the
values entered are kept but not the selected operators and sort.

The same bug was existing in authorities module, solved by Bug 8692.

This patch corrects the bug.

Also uses in 'sort by' options the same text as search in authorities
module : Heading A-Z (default), Heading Z-A, None.

Also removes duplicated code in auth_finder.pl :
    value_mainstr => $query->param('value_mainstr') || "", ...

Test plan :
- Create a new biblio with a framework containing a field linked to a
  thesaurus. For example : 600
- Click on small icon of main entry. For example : 600$a
=> You get a search form with all operators to "contains" and sort by
   "Heading A-Z"
- Enter a value in each text box and perform search
=> You get a search form with values in text boxes and all operators
   to "contains"
- Select "starts with" in all operator comboboxes and perform search
=> You get a search form with all operators to "starts with"
- Select "is exactly" in all operator comboboxes and perform search
=> You get a search form with all operators to "is exactly"
- Select "Heading Z-A" in sort by and perform search
=> You get a search form with "Heading Z-A" in sort by
- Select "None" in sort by and perform search
=> You get a search form with "None" in sort by

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Comment: With both patches applied no koha-qa errors

Test
1) Original behavior is whatever selection you do before patch,
search form returns to default options
2) After patch, selection remains

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
All tests and QA script pass, further comments on second patch.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 857547c992ea2b548712583d09602eda3787556c)
Signed-off-by: Fridolin SOMERS <fridolin.somers@biblibre.com>
(cherry picked from commit 9959fca67510356ab9d7acd128c453d6008a2b73)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
authorities/auth_finder.pl
koha-tmpl/intranet-tmpl/prog/en/includes/auth-finder-search.inc