https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25322

ByWater Sandboxes <bws.sandbo...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #105419|0                           |1
        is obsolete|                            |

--- Comment #17 from ByWater Sandboxes <bws.sandbo...@gmail.com> ---
Created attachment 105420
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=105420&action=edit
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 <nug...@gmail.com>

Signed-off-by: Holly Cooper <h...@interleaf.ie>

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to