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

missing implementation in RoleService: hard-coded return values in isController / isAdmin #3331

Open
saschaszott opened this issue Sep 18, 2024 · 0 comments
Labels
bug component: MyDSpace help wanted Needs a volunteer to claim to move forward

Comments

@saschaszott
Copy link
Contributor

saschaszott commented Sep 18, 2024

Describe the bug

RoleService provides methods to check whether a given user is submitter / controller / admin. The implementations of both methods isController and isAdmin are missing (TODO comment was added 5 years ago by @atarix83):

return observableOf(true);

return observableOf(false);

Consequently, every logged-in user has the controller "role" and the dropdown field in "MyDSpace" shows inappropriate option values. The screenshot below shows the options available to the demo user [email protected] (which has only the submitter role):

image

@saschaszott saschaszott added bug needs triage New issue needs triage and/or scheduling labels Sep 18, 2024
@github-project-automation github-project-automation bot moved this to 🆕 Triage in DSpace Backlog Sep 18, 2024
@tdonohue tdonohue added help wanted Needs a volunteer to claim to move forward component: MyDSpace and removed needs triage New issue needs triage and/or scheduling labels Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component: MyDSpace help wanted Needs a volunteer to claim to move forward
Projects
Development

No branches or pull requests

2 participants