ACL Editor: Role not displayed at all

Steps to reproduce

Steps to reproduce:
1. Go to http://octestallinone.virtuos.uos.de:8080/admin-ng/index.html#/users/acls
2. Press "Add access policiy"
3. Enter title and go to next tab
4. Select template "Public"

Actual Results:
The role of the first ACE is not shown (see screenshot). While the role is shown in the next tab (summary), it is not possible to see it later anymore, i.e. when editing the ACL, there is no way to see what role that ACE is for.

Expected Results:
Always show the roles of all ACE

Workaround (if any):

Activity

Show:
Christoph Driessen
November 13, 2015, 1:08 AM

The first role is ROLE_INSTRUCTOR which is not available in the list of roles. That's why the role is not displayed/selected in the drop down box.

Sven Stauber
November 13, 2015, 1:16 AM
Edited

As I understood, ACLs can contain arbitrary roles ("strings") even if those roles are not in the table mh_roles.
I don't see a reason, however, to not show that roles in the ACL editor. If their not shown in the ACL editor, there is no place where a user could see the value of those ACEs in the UI.
Maybe dynamically including such roles (well, it's at most one such role per ACE) per ACE role drop down would be a feasible solution?

Christoph Driessen
November 13, 2015, 3:40 AM

I am not an expert on the admin UI side but what I can see is, that roles are a controlled vocabulary which is provided by so-called list providers. The admin UI fetches both, the list of available (aka known) roles and the ACL and matches them. AFAIK there cannot be any "free" roles.

Sven Stauber
November 13, 2015, 4:01 AM

I see your point. Maybe I was a little focused on that because we are using such free roles all the time - maybe this again is because there is no interface where you ever could add roles
But removing ROLE_INSTRUCTOR at least solves to problem for the Opencast 2.1 release, which is great. Thanks!

Sven Stauber
December 1, 2015, 8:32 AM

Thanks Christoph! All roles are now shown as expected.

Fixed and reviewed

Assignee

Christoph Driessen

Reporter

Sven Stauber

Severity

Incorrectly Functioning Without Workaround

Tags (folksonomy)

Components

Fix versions

Affects versions

Priority

Major