From 700de0d3885eac9f1c9cc5b9a978f95b0d9c7eee Mon Sep 17 00:00:00 2001 From: Luke Taylor Date: Thu, 25 Oct 2007 15:07:15 +0000 Subject: [PATCH] Tidying. --- .../vote/AbstractAccessDecisionManager.java | 20 ++++++++----------- 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/core/src/main/java/org/springframework/security/vote/AbstractAccessDecisionManager.java b/core/src/main/java/org/springframework/security/vote/AbstractAccessDecisionManager.java index 1e42b0bcf2..cdb75a5647 100644 --- a/core/src/main/java/org/springframework/security/vote/AbstractAccessDecisionManager.java +++ b/core/src/main/java/org/springframework/security/vote/AbstractAccessDecisionManager.java @@ -38,8 +38,7 @@ import org.springframework.util.Assert; */ public abstract class AbstractAccessDecisionManager implements AccessDecisionManager, InitializingBean, MessageSourceAware { - // ~ Instance fields - // ================================================================================================ + //~ Instance fields ================================================================================================ private List decisionVoters; @@ -47,8 +46,7 @@ public abstract class AbstractAccessDecisionManager implements AccessDecisionMan private boolean allowIfAllAbstainDecisions = false; - // ~ Methods - // ======================================================================================================== + //~ Methods ======================================================================================================== public void afterPropertiesSet() throws Exception { Assert.notEmpty(this.decisionVoters, "A list of AccessDecisionVoters is required"); @@ -81,8 +79,8 @@ public abstract class AbstractAccessDecisionManager implements AccessDecisionMan while (iter.hasNext()) { Object currentObject = iter.next(); - Assert.isInstanceOf(AccessDecisionVoter.class, currentObject, "AccessDecisionVoter " + currentObject.getClass().getName() - + " must implement AccessDecisionVoter"); + Assert.isInstanceOf(AccessDecisionVoter.class, currentObject, "AccessDecisionVoter " + + currentObject.getClass().getName() + " must implement AccessDecisionVoter"); } this.decisionVoters = newList; @@ -107,15 +105,13 @@ public abstract class AbstractAccessDecisionManager implements AccessDecisionMan } /** - * Iterates through all AccessDecisionVoters and ensures - * each can support the presented class. + * Iterates through all AccessDecisionVoters and ensures each can support the presented class. *

- * If one or more voters cannot support the presented class, - * false is returned. + * If one or more voters cannot support the presented class, false is returned. *

* - * @param clazz DOCUMENT ME! - * @return DOCUMENT ME! + * @param clazz the type of secured object being presented + * @return true if this type is supported */ public boolean supports(Class clazz) { Iterator iter = this.decisionVoters.iterator();