https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15343
--- Comment #22 from Jonathan Druart <jonathan.dru...@bugs.koha-community.org> --- Comment on attachment 47407 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=47407 Bug 15343 - Allow user to set password on opac-memberentry.pl Review of attachment 47407: --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=15343&attachment=47407) ----------------------------------------------------------------- There is a third case to check: if the password contain leading or trailing whitespaces, see bug 14509. ::: C4/Members.pm @@ +2482,5 @@ > $borrower{'cardnumber'} = fixup_cardnumber(); > > my $borrowernumber = AddMember(%borrower); > > + return ( $borrowernumber, $borrower{'password'} ); Why do you return the password? You should compare the hashed password (Koha::AuthUtils::hash_password) in the test. ::: koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt @@ +60,5 @@ > [% IF field == "email" %]<li>Contact > information: <a href="#borrower_email">primary email address</a></li>[% END %] > [% IF field == "emailpro" %]<li>Contact > information: <a href="#borrower_emailpro">secondary email address</a></li>[% > END %] > [% IF field == "B_email" %]<li>Alternate > address information: <a href="#borrower_B_email">email address</a></li>[% END > %] > + [% IF field == "password_match" > %]<li>Passwords do not match! <a href="#password">password</a></li>[% END %] > + [% IF field == "password_invalid" > %]<li>Password does not meet minium requirements! <a > href="#password">password</a></li>[% END %] typo minimum. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/