Audit info setting fails in some cases
Bug #960624 reported by
Bill Erickson
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
High
|
Bill Erickson |
Bug Description
From bug #893645
AppUtils.pm set_audit_info expects the first parameter to be an open-ils.store OpenSRF:
I found a few examples:
open-ils.
Various API calls in Circ/StatCat.pm, Circ/Transit.pm and Circ/Survey.pm.
This may not be an exhaustive list. For most of these, it looks like a simple variable replacement (e.g. s/user_
Changed in evergreen: | |
assignee: | nobody → Bill Erickson (erickson-esilibrary) |
status: | New → In Progress |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Actually, it looks like I forgot to add the session variable at all to most of those calls, so where 4 arguments should be passed in only 3 were.
The branch below corrects that.
http:// git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ tsbere/ fix_set_ audit_info