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

Bug 10297 - categorycode and branchcode should be required when adding patrons 

categorycode and branchcode should show as required under all
circumstances, not just when they are included in the
BorrowerMandatoryField system preference. This patch adds the correct
classes and hints to those fields.

To test, view the patron entry form for a new or existing patron.
Category and Library should show that they are required.

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 81bfae9364)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Logical addition
This commit is contained in:
Owen Leonard 2013-06-10 11:14:41 -04:00 committed by Tomas Cohen Arazi
parent 10f5822daa
commit 545504e604
1 changed files with 4 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
View File

@ -989,13 +989,8 @@
[% END %]
[% UNLESS nobranchcode %]
<li>
[%- IF ( mandatorybranchcode ) -%]
<label for="branchcode" class="required">
[%- ELSE -%]
<label for="branchcode">
[%- END -%]
Library:</label>
<select name="branchcode" size="1" id="branchcode">
<label for="branchcode" class="required">Library:</label>
<select name="branchcode" size="1" id="branchcode">
[%- FOREACH branchloo IN branchloop %]
[% IF ( branchloo.selected ) -%]
<option value="[% branchloo.branchcode %]" selected="selected">[% branchloo.branchname %]</option>
@ -1004,13 +999,11 @@
[%- END -%]
[%- END %]
</select>
[% IF ( mandatorybranchcode ) -%]
<span class="required">Required</span>
[%- END %]
</li>
[% END %]
<li>
<label for="categorycode">Category: </label>
<label for="categorycode" class="required">Category: </label>
<select id="categorycode" name="categorycode" onchange="update_category_code(this);">
[% FOREACH typeloo IN typeloop %]
[% FOREACH categoryloo IN typeloo.categoryloop %]
@ -1033,6 +1026,7 @@
[% END %]
[% END %]
</select>
<span class="required">Required</span>
</li>
[% UNLESS nosort1 %]
<li>