Repository: wicket Updated Branches: refs/heads/master b088c6967 -> 1f805dd5a
WICKET-6463 Please add additional constructor to Roles Project: http://git-wip-us.apache.org/repos/asf/wicket/repo Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/1f805dd5 Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/1f805dd5 Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/1f805dd5 Branch: refs/heads/master Commit: 1f805dd5af6d06e9209907f33f3c7ab4a4678e76 Parents: b088c69 Author: Martin Tzvetanov Grigorov <[email protected]> Authored: Mon Sep 4 22:11:39 2017 +0300 Committer: Martin Tzvetanov Grigorov <[email protected]> Committed: Mon Sep 4 22:11:39 2017 +0300 ---------------------------------------------------------------------- .../authroles/authorization/strategies/role/Roles.java | 9 +++++++++ .../annotations/AnnotationsRoleAuthorizationStrategy.java | 2 +- 2 files changed, 10 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/wicket/blob/1f805dd5/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/Roles.java ---------------------------------------------------------------------- diff --git a/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/Roles.java b/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/Roles.java index 5d272de..fa5251d 100644 --- a/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/Roles.java +++ b/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/Roles.java @@ -16,9 +16,11 @@ */ package org.apache.wicket.authroles.authorization.strategies.role; +import java.util.Collection; import java.util.HashSet; import org.apache.wicket.util.io.IClusterable; +import org.apache.wicket.util.lang.Args; import org.apache.wicket.util.string.StringList; @@ -53,6 +55,7 @@ public class Roles extends HashSet<String> implements IClusterable */ public Roles(final String roles) { + Args.notNull(roles, "roles"); for (final String role : roles.split("\\s*,\\s*")) { add(role); @@ -67,12 +70,18 @@ public class Roles extends HashSet<String> implements IClusterable */ public Roles(final String[] roles) { + Args.notNull(roles, "roles"); for (final String role : roles) { add(role); } } + public Roles(final Collection<String> roles) { + Args.notNull(roles, "roles"); + addAll(roles); + } + /** * Whether this roles object containes the provided role. * http://git-wip-us.apache.org/repos/asf/wicket/blob/1f805dd5/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/annotations/AnnotationsRoleAuthorizationStrategy.java ---------------------------------------------------------------------- diff --git a/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/annotations/AnnotationsRoleAuthorizationStrategy.java b/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/annotations/AnnotationsRoleAuthorizationStrategy.java index ff7b9dd..96163e1 100644 --- a/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/annotations/AnnotationsRoleAuthorizationStrategy.java +++ b/wicket-auth-roles/src/main/java/org/apache/wicket/authroles/authorization/strategies/role/annotations/AnnotationsRoleAuthorizationStrategy.java @@ -171,7 +171,7 @@ public class AnnotationsRoleAuthorizationStrategy extends AbstractRoleAuthorizat } Roles acceptedRoles = new Roles(authorizeActionAnnotation.roles()); - if (!(isEmpty(acceptedRoles) || hasAny(acceptedRoles))) + if (!hasAny(acceptedRoles)) { return false; }
