From 0261fb9149ff026a83b2316e6c78f31cd3e142fd Mon Sep 17 00:00:00 2001 From: Petro Vashchuk Date: Thu, 28 May 2020 18:20:11 +0300 Subject: [PATCH] Bug 25322: fix for don't default "relationship" dropdown to father When you create a patron's guarantor on cgi-bin/koha/members/memberentry.pl but don't select the relationship from a dropdown, the relationship defaults to first default value, which in this case is father that may or may not be correct as that is not a conscious choice from the user. The solution is to make the "Relationship" field mandatory, starting with an empty option but not allowing the user to save empty, so choice will be conscious. To reproduce: 1) Create a new patron who is assumed to have a guarantor or modify the existing one. 2) Under "Guarantor Information" click on "Search to add" button. After performing the search, select a user to act as guarantor. Don't use the dropdown menu to select a relationship. Save your changes. 3) Observe that relationship is set as "father". 4) Apply the patch. 5) Repeat steps 1 and 2. 6) Observe that it doesn't allow you to save the form until you pick a relationship type. Mentored-by: Andrew Nugged --- .../prog/en/modules/members/memberentrygen.tt | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt index 8cc1c9ec00..16d7220240 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt @@ -474,12 +474,14 @@ legend:hover {
  • - - + [% FOREACH pr IN possible_relationships.split('\|') %] [% END %] + Required
  • @@ -510,12 +512,14 @@ legend:hover {
  • - - + [% FOREACH pr IN possible_relationships.split('\|') %] [% END %] + Required
  • -- 2.17.1