Bug 13458: Display the correct patron categories
authorJonathan Druart <jonathan.druart@biblibre.com>
Fri, 26 Dec 2014 09:22:34 +0000 (10:22 +0100)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Sun, 28 Dec 2014 23:34:33 +0000 (20:34 -0300)
commit2b39823c0f4cec1607078de1f2547afc95ae87cd
treeaf3627c435f2adb39d9a1757c0fd07d386d2177d
parent90ebffcffae59b5ded2db3dce875be0293b845e1
Bug 13458: Display the correct patron categories

Bug 9811 removes useful code.
Actually the AddPatronLists pref is not sent to the template from
members/member.pl.

To fix this issue, we can use the existing not clean way, or compare the
syspref value directly in the template. This second solution is
implemented in this patch.

Test plan:
1/ Set the AddPatronLists pref to 'specific'
2/ On the patron home page (members/members-home/pl), the patron search
result page (members/member.pl after launching a search) and on the
checkouts page/patron search result (circ/circulation.pl after searching
a patron using the check out), verify that the patron category list is
the specific ones.
3/ Test there are no regression with the AddPatronLists pref set to
'general'.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Works as described and fixes the problem.

Note: I am not sure if AddPatronLists makes sense -
if you set it to general patron types, it still preselects the
wrong category type (tried organization, a child patron category
was selected). Also the name is confusiong nowadays with the
Patron list feature.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
circ/circulation.pl
koha-tmpl/intranet-tmpl/prog/en/includes/patron-toolbar.inc
members/members-home.pl