Missing override permission for PATRON_EXCEEDS_LOST_COUNT

Bug #1254826 reported by Ben Shum
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
In Progress
Medium
Josh Stompro
2.5
New
Undecided
Unassigned

Bug Description

Evergreen master / 2.5

It looks like when new functionality was added for a new standing penalty of "PATRON_EXCEEDS_LOST_COUNT" (see bug 1154258), there wasn't a new permission added for overriding that penalty.

Should be just as simple as adding a new permission called "PATRON_EXCEEDS_LOST_COUNT.override" to the perm_list table and then seeing which stock users need it applied (maybe the circ admins?)

Working branch forthcoming.

Tags: permissions
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.5.1 → 2.5.2
status: New → Triaged
importance: Undecided → Medium
Dan Wells (dbw2)
Changed in evergreen:
assignee: nobody → Ben Shum (bshum)
Ben Shum (bshum)
Changed in evergreen:
status: Triaged → In Progress
Dan Wells (dbw2)
Changed in evergreen:
milestone: 2.5.2 → 2.6.0-alpha1
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.6.0-alpha1 → none
Revision history for this message
Ben Shum (bshum) wrote :

Working branch in progress at: user/bshum/lp1254826-add-override-for-lost-count

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/lp1254826-add-override-for-lost-count

The branch in progress only adds the permission to the seed data. We have to figure out how to safely add this as an upgrade script without breaking systems that may have already added the necessary override permission. Also if it's required by default for any groups in the seed data (like Circulators get a few overrides to start with, etc.)

Along the way, we may want to also check and see if the PATRON_EXCEEDS_LONGOVERDUE_COUNT also needs an override.

tags: added: permissions
Changed in evergreen:
assignee: Ben Shum (bshum) → Josh Stompro (u-launchpad-stompro-org)
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.