[Koha-bugs] [Bug 16694] Limit SIP2 auth by patron attribute

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Oct 11 17:21:16 CEST 2016


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16694

Magnus Enger <magnus at libriotech.no> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #3 from Magnus Enger <magnus at libriotech.no> ---
I can't seem to be able to turn "64  " into "64YY", no matter what I set the
extended patron attribute is set to. I think this is because the default values
for charge_ok and renew_ok are 1, so by only setting them to 1 the patches
never changes them. Something like this should work, I think:

            if ($attr || $attr == "1") {
                syslog( "LOG_ERR", "attr OK" );
                $patron->{charge_ok} = 1; 
                $patron->{renew_ok} = 1; 
            } else {
                $patron->{charge_ok} = 0; 
                $patron->{renew_ok} = 0; 
            }

Otherwise, this looks OK, so it should be an easy fix/followup.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list