I've given this some more thought, I think I've come up with a way to give PINES what they want WRT "who set this penalty" while retaining the benefits of the existing design.
Really, it seems obvious now that I'm writing it down... c'est la vie.
1) add a new column to ausp.org_unit called originating_org_unit that captures the ws_ou in open-ils.actor.user.penalty.apply
2) surface that in appropriate UIs (via IDL addition, fleshing, minor UI addition) next to ausp.org_unit
For backfilling that data we can either copy ausp.org_unit, or leave it NULL. I lean pretty strongly towards towards NULL for both technical (less painful upgrade) and semantics (we really don't know, and already have the ausp.org_unit "effective org" data).
I've given this some more thought, I think I've come up with a way to give PINES what they want WRT "who set this penalty" while retaining the benefits of the existing design.
Really, it seems obvious now that I'm writing it down... c'est la vie.
1) add a new column to ausp.org_unit called originating_ org_unit that captures the ws_ou in open-ils. actor.user. penalty. apply
2) surface that in appropriate UIs (via IDL addition, fleshing, minor UI addition) next to ausp.org_unit
For backfilling that data we can either copy ausp.org_unit, or leave it NULL. I lean pretty strongly towards towards NULL for both technical (less painful upgrade) and semantics (we really don't know, and already have the ausp.org_unit "effective org" data).
Thoughts?