Bug 35510: Non-patron guarantor missing from CollapseFieldsPatronAddForm options
This patch adds a missing option to the CollapseFieldsPatronAddForm system preference so that the "Non-patron guarantor" section can be collapsed by default. To test, apply the patch and go to Administration -> System preferences. - Search for CollapseFieldsPatronAddForm. - In the dropdown of options you should see an entry for "Non-patron guarantor." - Click "Select all" and save your changes. - Go to Patrons -> New patron and choose a patron category which can have a guarantor (for example J - Juvenile) - The patron entry page should load with all sections collapsed, including the Non-patron guarantor section. - Uncheck all options in the CollapseFieldsPatronAddForm preference and return to the patron entry form. All sections should now appear. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> (cherry picked from commite97dac3cbe
) Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> (cherry picked from commit036228a4e0
) Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
a267c38a3f
commit
83933dafe5
2 changed files with 6 additions and 1 deletions
|
@ -287,7 +287,8 @@ Patrons:
|
|||
- pref: CollapseFieldsPatronAddForm
|
||||
multiple:
|
||||
identity: "Patron identity"
|
||||
guarantor: "Guarantor information"
|
||||
guarantor: "Patron guarantor"
|
||||
nonpatron_guarantor: "Non-patron guarantor"
|
||||
primary_address: "Main address"
|
||||
primary_contact: "Contact information"
|
||||
alt_address: "Alternate address"
|
||||
|
|
|
@ -193,6 +193,7 @@ legend:hover {
|
|||
[% SWITCH field %]
|
||||
[% CASE 'identity' %] <span>Patron identity</span> |
|
||||
[% CASE 'guarantor' %] <span>Guarantor information</span> |
|
||||
[% CASE 'nonpatron_guarantor' %] <span>Non-patron guarantor</span> |
|
||||
[% CASE 'primary_address' %] <span>Main address</span> |
|
||||
[% CASE 'primary_contact' %] <span>Contact information</span> |
|
||||
[% CASE 'alt_address' %] <span>Alternate address</span> |
|
||||
|
@ -1666,6 +1667,9 @@ legend:hover {
|
|||
$("#memberentry_guarantor").find('legend').nextAll().toggle();
|
||||
$("#guarantor_template").toggle();
|
||||
[% END %]
|
||||
[% IF fieldstohide.match('nonpatron_guarantor') %]
|
||||
$("#non_patron_guarantor").find('legend').nextAll().toggle();
|
||||
[% END %]
|
||||
[% END %]
|
||||
[% IF fieldstohide.match('primary_address') %]
|
||||
$("#memberentry_mainaddress").find('legend').nextAll().toggle();
|
||||
|
|
Loading…
Reference in a new issue