Bug 14867: userid not generated when defined in BorrowerUnwantedField
authorFridolin Somers <fridolin.somers@biblibre.com>
Tue, 22 Sep 2015 07:49:39 +0000 (09:49 +0200)
committerFrédéric Demians <f.demians@tamil.fr>
Sun, 29 Nov 2015 20:45:23 +0000 (21:45 +0100)
commitee185cfd1afed08d20f226ef8a2c62df2c96b6e8
treecac4265a3218cbd201b8f4c9282a1243b7a967be
parent1cd654313bd539e22eacf4322fc419331ed80d44
Bug 14867: userid not generated when defined in BorrowerUnwantedField

When userid is not provided when creating a new patron, it is generated
using surname and firstname.  The bug is when userid is defined in
syspref BorrowerUnwantedField, the input text is missing in patron
creation form. When saving you always get an alert message :
"Username/password already exists". No patron can be created.

This patch corrects by adding this case to userid generation conditions.

Test plan :
- add 'userid' in syspref BorrowerUnwantedField
- try to create a new patron : /cgi-bin/koha/members/memberentry.pl
- there is not input text for userid
- choose non-existing surname and firstname
- click on save
  => Without patch : patron is not created, you see the alert message
     "Username/password already exists"
  => With patch : patron is created, userid is generated with surname
     and firstname
- remove 'userid' in syspref BorrowerUnwantedField and check it can be
  defined in patron creation form

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
  The issue is there, and this patch fixes it.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit 0cbc65111c735e699d94d50ad09c17fb7b7b8913)
members/memberentry.pl