Add more permission checks to menu options

Bug #1089494 reported by Mark Cooper
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Triaged
Wishlist
Unassigned

Bug Description

Evergreen master:

See - https://bugs.launchpad.net/evergreen/+bug/1066158

This is a first attempt to add additional permissions to various menu options - both admin and standard.
The problem is that often staff can access parts of the staff client interface that they don't have appropriate permissions for. This creates confusion for staff and is not very user friendly.

Where possible I'm taking the approach of identifying a single base permission that would be necessary to use the interface i.e.

cmd_acq_create_invoice => CREATE_INVOICE

Here we go:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=15afea95e599edaa311d57f1dbae637fa6064a8e

tags: added: wishlist
Ben Shum (bshum)
Changed in evergreen:
status: New → Triaged
importance: Undecided → Wishlist
milestone: none → 2.4.0-alpha
tags: removed: wishlist
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-alpha1 → 2.4.0-beta
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-beta → 2.4.0-rc
Ben Shum (bshum)
Changed in evergreen:
milestone: 2.4.0-rc → 2.5.0-alpha
Revision history for this message
Dan Wells (dbw2) wrote :

Want to add a pullrequest, but the commit link is a 404. Does this code still exist?

Changed in evergreen:
milestone: 2.5.0-m1 → none
Revision history for this message
Mark Cooper (markchristophercooper) wrote :

This worked well for us during a solid 4 month testing period. However the original push is stale at this point. Just updated and resolved conflicts with master. Want to test again before adding pullrequest tag.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=commit;h=4d8b0b317339ecd40cd3d6826474e515a893eccf

Revision history for this message
Terran McCanna (tmccanna) wrote :

Noting that this is still a valid usability issue, but the previous proposed fix was for XUL so is no longer valid.

tags: added: usability
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.