http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3674

--- Comment #4 from Frédéric Demians <[email protected]> 2011-05-14 06:43:01 
UTC ---
(In reply to comment #2)
> Created attachment 4006 [details]
> reimplementation
> 
> This re-implements the fix to provide disabled logins and also adds
> auto-generated logins when none provided.

This patch contains code from your patch for bug 6224. So there will be
a merge conflict. It would be great to base it on a branch containing
already bug 6224 patch and note that it must be apply after 6224.

Your patch solves the bug, ie put a correct value in password field, but
there is no UI feedback telling the librarian what has been done
internally. I think it's confusing. And it's also not clear what happens
when password is modified and cleared for an existing borrower. 

So I sign-off this patch, since it fixes the describe bug but something
has to be done to improve userid/password management. The merge conflict
should be manageable by the RM...

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
_______________________________________________
Koha-bugs mailing list
[email protected]
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/

Reply via email to