http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6328 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ian.walls@bywatersolutions. | |com Patch Status|Signed Off |Failed QA --- Comment #32 from Ian Walls <ian.walls@bywatersolutions.com> 2011-10-07 20:04:52 UTC --- Doesn't update all instances of $borrower->{'debarred'}; there are usages on the following files that are not updated to reflect the change data type for 'debarred': opac/opac-reserve.pl opac/opac-user.pl members/setdebar.pl misc/cronjobs/notifyMailsOp.pl C4/ILSDI/Utility.pm C4::Members::patronflags() is not touched, so the DBARRED flag in GetMemberDetails is never set; thus any place in the code to use this is not handled: This includes: C4/SIP/ILS/Patron.pm C4/Circulation::CanBookBeIssued circ/circulation.pl There is some documentation on C4/Auth_with_ldap.pm that needs updating as well. This change will likely break opac-reserve.pl, ILSDI and SIP2 connections, as well as return erroneous values for CanBookBeIssued. Marking as Failed QA. -- 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.