sandstorm / NeosAcl

14 stars 8 forks source link

Selecting the Root-Node leads to Exception #11

Closed DrillSergeant closed 4 years ago

DrillSergeant commented 4 years ago

When you create a dynamic role and select no nodes that role has access to all nodes, even if it is not stated in the UI like "Leave empty to grant access to all nodes". I (my project team) thought it is easier to understand that a role has access to all nodes if we check the root-node (instead of checking no node at all). After we did that (checked the root node and saved) we encoutered a bug:

Steps to reproduce

Expected behaviour You can edit the role and the root-node is selected

Actual behaviour You cannot edit the role, you end up in an exception.

Bug locally already fixed, PR will follow (in a few minutes...)

skurfuerst commented 4 years ago

thank you so much :)