https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22253Josef Moravec <
[hidden email]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #84640|0 |1
is obsolete| |
--- Comment #3 from Josef Moravec <
[hidden email]> ---
Created attachment 84657
-->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=84657&action=editBug 22253: Check we actually need to update the password
This patch makes memberentry.pl check if password needs to be updated
before attempting to call set_password. Above this there's a check that
won't raise any errors if no password is passed, or the default string (****)
is received.
So we could reach that line of code with no password, but the code
wouldn't check that.
To test:
- In master, edit any patron without changing the password
=> FAIL: It raises an exception
- Apply this patch
- Edit the patron withtout changing the password
=> SUCCESS: Edit successful
- Edit the patron, changing the password
- Try to login with the new password
=> SUCCESS: The password got changed correctly
- Sigh off :-D
Signed-off-by: Owen Leonard <
[hidden email]>
Signed-off-by: Josef Moravec <
[hidden email]>
--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugswebsite :
http://www.koha-community.org/git :
http://git.koha-community.org/bugs :
http://bugs.koha-community.org/