]> git.koha-community.org Git - koha.git/commit
Bug 16847: Remove C4::Members::GetTitles
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 4 Jul 2016 14:03:18 +0000 (15:03 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 22 Jul 2016 17:23:42 +0000 (17:23 +0000)
commit131002e6230f2b69f0d6007b75fbc90444745b68
treef49e378e7f70df6d2cd84616ebe6761164fa2f9c
parent6ac4b20d3ab548621d6ffc8a931b947fdb3eb6f2
Bug 16847: Remove C4::Members::GetTitles

This subroutine just reads the content of a pref, split it, add an
empty string and returns.
It is not really useful and the code in the script (memberentry.pl) uses
the only occurrence of CGI::popup_menu
Let's remove it and build the dropdown list in the template.

Test plan:
1/ Empty BorrowersTitles, edit a patron and confirm that the "Salutation"
does not appear
2/ Fill BorrowersTitles with "Mr|Mrs|Miss|Ms", edit a patron and confirm
that the "Salutation" dropdown list is correctly filled.
The default option should be selected if you are editing a patron with a
title defined.
This should also be tested at the OPAC.

Followed test pan, works as expected in Staff and OPAC
Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
C4/Members.pm
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
members/memberentry.pl
opac/opac-memberentry.pl