Permission to view Holdings Import Profiles

Bug #1845235 reported by Jeff Davis
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
New
Wishlist
Unassigned

Bug Description

Currently the permissions required to view holdings import profiles are the same perms needed to update the profile. But in some cases it's desirable to restrict the ability to edit profiles to admins, while allowing other staff to be able to view (and thus select) a holdings import profile to use when importing MARC records. So there should be a separate VIEW_IMPORT_ITEM_ATTR_DEF permission.

Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Working branch user/jeffdavis/lp1845235-holdings-import-profile-view-perm creates a new VIEW_IMPORT_ITEM_ATTR_DEF permission and uses it in the fieldmapper:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1845235-holdings-import-profile-view-perm

tags: added: pullrequest
Changed in evergreen:
milestone: none → 3.4-beta2
tags: added: permissions
Changed in evergreen:
milestone: 3.4-beta2 → 3.next
importance: Undecided → Wishlist
Changed in evergreen:
milestone: 3.next → 3.7-beta
Changed in evergreen:
milestone: 3.7-beta → 3.8-beta
Revision history for this message
Jennifer Weston (jweston) wrote (last edit ):

Tested by adding VIEW_IMPORT_ATTR_DEF to staff account with "Catalogers" perm group that does not have any pre-existing permissions for the IMPORT_ATTR_DEF. User: br1dbeck | demo123

One issue to report:
Using the updated staff account to login, I was able to see the Holdings Import Profile and I could click into the fields and make changes to the values (ex: Barcode, remove the value completely). The green success confirmation message was returned. The changes were not actually saved but the "Update Succeeded" message needs to be replaced with a failure message.

Tested on: https://pattypan.evergreencatalog.com/eg2/en-US/staff/cat/vandelay/holdings_profiles

This is a very nice option to have. Thanks for your work on it, Jeff!

Changed in evergreen:
milestone: 3.8-beta → 3.next
Revision history for this message
Jessica Woolford (jwoolford) wrote :

Tags updated based on Jennifer's comment.

tags: added: needswork
removed: pullrequest
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.