Lineitem alerts not displayed scoped to org unit

Bug #996033 reported by Lebbeous Fogle-Weekley
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Bill Erickson

Bug Description

To quote Sharon Herbert of Sitka:

---

Instead of scoping to the owning org unit, line item alerts are visible to all org units. This especially
causes a problem when multiple libraries have created the same line item alerts and cannot tell which
one belongs to their org unit.

---

I can address this issue myself.

For reference, lineitem alerts are embodied by rows in acq.lineitem_note where the alert_text column is not null and refers to a row in acq.lineitem_alert_text. That latter table has an owning_lib column that we can use for scoping.

The controlled vocabulary of acq lineitem alert text is set up in an interface labeled "Line Item Alerts" in the staff client admin menu. In the PO and selection list interfaces (where alerts can be applied), we will limit the user to selecting alerts from those owned at the user's workstation org unit (or its ancestors).

When we display applied alerts, we can also display the shortname of the org unit at which the alert text is defined (the owning_lib column). This should help clarify what's going on with alerts that have already been applied using alert text from another org unit.

Revision history for this message
Lebbeous Fogle-Weekley (lebbeous) wrote :

Here's a patch that implements what I outlined in the description of this Launchpad bug: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/senator/lp996033

This is suitable for master and backport to 2.2 at least.

Changed in evergreen:
milestone: none → 2.2.0rc1
importance: Undecided → Medium
tags: added: pullrequest
Changed in evergreen:
milestone: 2.2.0rc1 → 2.2.0
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 merged.

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.