Missing override permission for PATRON_EXCEEDS_LOST_COUNT

Bug #1254826 reported by Ben Shum on 2013-11-25
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Josh Stompro

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.

Ben Shum (bshum) on 2013-12-06
Changed in evergreen:
milestone: 2.5.1 → 2.5.2
status: New → Triaged
importance: Undecided → Medium
Dan Wells (dbw2) on 2014-01-17
Changed in evergreen:
assignee: nobody → Ben Shum (bshum)
Ben Shum (bshum) on 2014-01-17
Changed in evergreen:
status: Triaged → In Progress
Dan Wells (dbw2) on 2014-01-17
Changed in evergreen:
milestone: 2.5.2 → 2.6.0-alpha1
Ben Shum (bshum) on 2014-01-17
Changed in evergreen:
milestone: 2.6.0-alpha1 → none
Ben Shum (bshum) wrote :

Working branch in progress at: 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  Edit
Everyone can see this information.

Other bug subscribers