Audit info setting fails in some cases

Bug #960624 reported by Bill Erickson
6
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::AppSession. In some contexts, it's passed an authtoken instead. This results in failures with (for example) creating payments.

I found a few examples:

open-ils.circ.money.grocery.create
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_session/session/) will suffice.

Tags: pullrequest
Revision history for this message
Thomas Berezansky (tsbere) wrote :

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

tags: added: pullrequest
Changed in evergreen:
milestone: none → 2.2.0beta1
importance: Undecided → High
Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (erickson-esilibrary)
status: New → In Progress
Revision history for this message
Bill Erickson (berick) wrote :

Tested and pushed to master. Thanks!

Changed in evergreen:
status: In Progress → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.