On Feb 12, 2008, at 3:16 PM, David Jencks wrote:

I started looking at the UserManager and Permissions and have a couple immediate questions...

Is the set of possible actions finite and known (when roller is compiled) or is it extensible by the user? If it's known I'm going to propose essentially a bitset implementation for actions.

Are the following really intentional?

oops, these are supposed to be Collections.singletonList() etc.

new GlobalPermission(Arrays.emptyList()).implies(new GlobalPermission(Arrays.singletonList("ADMIN"))) == true

new GlobalPermission(Arrays.singletonList("FOO")).implies(new GlobalPermision(Arrays.singletonList("BAR"))) == true

and many other similar examples with WebLogPermission. This seems to me like asking for trouble.

Similarly, is the name really supposed to be ignored in both implies methods?

new WebLogPermission(wl1, Collections.singletonList("Admin")).implies (new WebLogPermission(wl2, Collections.singletonList("Admin"))) == true

similarly for GlobalPermission's user...

etc etc

thanks again

thanks
david jencks


Reply via email to