Bug 9043: The comma (,) should be kept to separate multi-valuated prefs
authorJonathan Druart <jonathan.druart@biblibre.com>
Mon, 17 Nov 2014 15:55:26 +0000 (16:55 +0100)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Mon, 17 Nov 2014 17:49:59 +0000 (14:49 -0300)
commitdddde58967137edb01dfe5bae261f52fec867484
treea5d0e60bd3b2d692495722631b92ef4f29b63b64
parent47e030ea753c160b166fa85e09fadee744225053
Bug 9043: The comma (,) should be kept to separate multi-valuated prefs

The prefs language and opaclanguages used the comma to separate the
different values.

The new prefs OpacAdvSearchMoreOptions and OpacAdvSearchOptions should
do the same.

To reproduce the issue: update the language pref (or opaclanguages) and
refresh the page.
=> The pref values are not checked anymore and the language selection
(bottom of the page) does not appear.

Test plan:
1/ Verify that the behavior described above is fixed.
2/ Verify that the original test plan of bug 9043 still passes.

Note: The 2 OpacAdvSearchMoreOptions and OpacAdvSearchOptions pref
values are overwritten but the feature have just been pushed recently.
It should not affect a production environment.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
I was able to reproduce the issue and verify that this patch corrected
the problem. Langage selection and OpacAdvSearchOptions worked
correctly.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
admin/preferences.pl
installer/data/mysql/sysprefs.sql
installer/data/mysql/updatedatabase.pl
opac/opac-search.pl
svc/config/systempreferences