Bug 31562: Respect mandatory fields pref
To test: 1 - Set gonenoaddress and lost in BorrowerMandatoryField 2 - Edit a patron, confirm fields are required Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
parent
8c50fe3890
commit
15039ab66f
1 changed files with 17 additions and 2 deletions
|
@ -1315,7 +1315,11 @@ legend:hover {
|
|||
<ol class="radio">
|
||||
[% UNLESS nogonenoaddress %]
|
||||
<li>
|
||||
<label class="radio" for="yesgonenoaddress">
|
||||
[% IF mandatorygonenoaddress %]
|
||||
<label class="radio required" for="yesgonenoaddress">
|
||||
[% ELSE %]
|
||||
<label class="radio" for="yesgonenoaddress">
|
||||
[% END %]
|
||||
Gone no address:
|
||||
</label>
|
||||
[% IF CAN_user_circulate_manage_restrictions %]
|
||||
|
@ -1338,11 +1342,19 @@ legend:hover {
|
|||
[% ELSE %]
|
||||
[% IF borrower_data.gonenoaddress %]<span>Yes</span>[% ELSE %]<span>No</span>[% END %]
|
||||
[% END # /IF CAN_user_circulate_manage_restrictions %]
|
||||
[% IF mandatorygonenoaddress %]
|
||||
<span class="required">Required</span>
|
||||
[% END %]
|
||||
</li>
|
||||
|
||||
[% END # /UNLESS nogonenoaddress %]
|
||||
[% UNLESS nolost %]
|
||||
<li>
|
||||
<label class="radio" for="yeslost">
|
||||
[% IF mandatorylost %]
|
||||
<label class="radio required" for="yeslost">
|
||||
[% ELSE %]
|
||||
<label class="radio" for="yeslost">
|
||||
[% END %]
|
||||
Lost card:
|
||||
</label>
|
||||
[% IF CAN_user_circulate_manage_restrictions %]
|
||||
|
@ -1365,6 +1377,9 @@ legend:hover {
|
|||
[% ELSE %]
|
||||
[% IF borrower_data.lost %]<span>Yes</span>[% ELSE %]<span>No</span>[% END %]
|
||||
[% END # /IF CAN_user_circulate_manage_restrictions %]
|
||||
[% IF mandatorylost %]
|
||||
<span class="required">Required</span>
|
||||
[% END %]
|
||||
</li>
|
||||
[% END # /UNLESS nogonenoaddress %]
|
||||
</ol>
|
||||
|
|
Loading…
Reference in a new issue