Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding second manager removes edit access of first manager #564

Closed
mjgiarlo opened this issue Mar 15, 2017 · 0 comments
Closed

Adding second manager removes edit access of first manager #564

mjgiarlo opened this issue Mar 15, 2017 · 0 comments
Assignees
Labels
bug Collection impacts the Collection part of PCDM Model
Milestone

Comments

@mjgiarlo
Copy link
Member

mjgiarlo commented Mar 15, 2017

First reported in Sufia: samvera-deprecated/sufia#3148 Should be verified in Hyrax first.

Descriptive summary

When adding a Manager to an Admin Set, the Manager user gets edit_access to the Admin Set. When you add a second Manager to the Admin Set, the first user, who is still a manager, no longer has edit_access. The second Manager does have edit_access.

Expected behavior

When the second Manager is added, both managers will have edit_access to the Admin Set, as seen in the solr document for the Admin Set in Solr.

Actual behavior

When the second Manager is added, only the second Manager has edit_access to the Admin Set, as seen in the solr document for the Admin Set in Solr.

Steps to reproduce the behavior

Prerequisite: You will need 1 admin user and 2 non-admin users.

Primary Validation

  1. Create Admin Set
  2. Login as an admin user
  3. Create an Admin Set and Save
  4. Edit the Admin Set
  5. View the Admin Set's show page
  6. Edit the Admin Set
  7. Click the Participants tab
  8. Set the first Manager
  9. Add Participants -> Add user: -> set to non-admin user
  10. Set role to Manager
  11. Click Add button
  12. VERIFY: Check solr document for the new Admin Set in Solr and verify that the selected user is in the edit_access_person_ssim list.
  13. Set the second Manager
  14. Add Participants -> Add user: -> set to another non-admin user
  15. Set role to Manager
  16. Click Add button
  17. VERIFY: Check solr document for the new Admin Set in Solr and verify that only the second manager user is in the edit_access_person_ssim list.

EXPECTED: I would expect the edit_access_person_ssim to include both manager users.

Secondary Validation: Manager can edit metadata in an Admin Set

  1. Login as each Manager user
  2. Attempt to edit the Admin Set -- NOT AVAILABLE

I would expect that Managers of an Admin Set would be able to edit the metadata and set participants for an Admin Set. But I can't find a way through the UI for a non-admin user to edit Admin Sets.

Third Validation: Manager can add works to Admin Set

  1. Login as each Manager user
  2. In Top Menu, select Works -> New Work -> select work type
  3. In New Work form, select Relationships tab
  4. VERIFY: 'Add as member of administrative set' selection list includes the Admin Set

This verification is CORRECT for both users.

So something other than edit_access on the Admin Set is driving ability to select the Admin Set on New Work -> Relationships tab.

(Thanks for writing this up, @elrayle)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Collection impacts the Collection part of PCDM Model
Projects
None yet
Development

No branches or pull requests

2 participants