gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 11342: fix error in OPAC self registration form if BorrowersTitles is empty 

If BorrowersTitles is empty, it causes the library pulldown on the self
registration page to be empty, and to have the "Saluation" field have
the option "branches".

This patch also fixes a minor string capitalization issue.

Test Plan:
1) Enable OPAC self registration
2) Set the system preference BorrowersTitles to be empty
3) View the self registration page
4) Note the lack of branches in the home library pulldown
5) Apply this patch
6) Note the branches now display in the pulldown

Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested in bootstrap and prog OPAC, with BorrowersTitle configured
and emptied.
Passes all tests and QA script.

Note: The titles pull down has 2 empty entries in master with
and without the patch.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 9a5f737dcb)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Kyle M Hall 2013-12-05 09:13:35 -05:00 committed by Fridolin Somers
parent 335b3059d0
commit cc86857dd6
3 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
View File

@ -68,7 +68,7 @@
[% ELSE %]
<label for="borrower_branchcode">
[% END %]
Home Library:</label>
Home library:</label>
<select id="borrower_branchcode" name="borrower_branchcode">
[% FOREACH b IN branches %]
@ -100,7 +100,7 @@
<legend id="identity_legend">Identity</legend>
<ol>
[% UNLESS hidden.defined('title') %]
[% UNLESS hidden.defined('title') || !member_titles %]
<li>
[% IF mandatory.defined('title') %]
<label for="borrower_title" class="required">

4
koha-tmpl/opac-tmpl/prog/en/modules/opac-memberentry.tt
View File

@ -77,7 +77,7 @@
[% ELSE %]
<label for="borrower_branchcode">
[% END %]
Home Library:</label>
Home library:</label>
<select id="borrower_branchcode" name="borrower_branchcode">
[% FOREACH b IN branches %]
@ -107,7 +107,7 @@
<legend id="identity_legend">Identity</legend>
<ol>
[% UNLESS hidden.defined('title') %]
[% UNLESS hidden.defined('title') || !member_titles %]
<li>
[% IF mandatory.defined('title') %]
<label for="borrower_title" class="required">

2
opac/opac-memberentry.pl
View File

@ -60,7 +60,7 @@ $template->param(
action => $action,
hidden => GetHiddenFields(),
mandatory => GetMandatoryFields($action),
member_titles => GetTitles(),
member_titles => GetTitles() || undef,
branches => GetBranchesLoop(),
OPACPatronDetails => C4::Context->preference('OPACPatronDetails'),
);