Removed commented out line.

This commit is contained in:
Luke Taylor 2008-07-31 20:45:25 +00:00
parent ef44bd91f2
commit 1d96283876

View File

@ -63,10 +63,10 @@ public class MethodDefinitionSourceAdvisor extends AbstractPointcutAdvisor imple
* @deprecated use the decoupled approach instead * @deprecated use the decoupled approach instead
*/ */
public MethodDefinitionSourceAdvisor(MethodSecurityInterceptor advice) { public MethodDefinitionSourceAdvisor(MethodSecurityInterceptor advice) {
Assert.notNull(advice.getObjectDefinitionSource(), "Cannot construct a MethodDefinitionSourceAdvisor using a " + Assert.notNull(advice.getObjectDefinitionSource(), "Cannot construct a MethodDefinitionSourceAdvisor using a " +
"MethodSecurityInterceptor that has no ObjectDefinitionSource configured"); "MethodSecurityInterceptor that has no ObjectDefinitionSource configured");
this.interceptor = advice; this.interceptor = advice;
this.attributeSource = advice.getObjectDefinitionSource(); this.attributeSource = advice.getObjectDefinitionSource();
} }
@ -83,35 +83,34 @@ public class MethodDefinitionSourceAdvisor extends AbstractPointcutAdvisor imple
* @param attributeSource the attribute source (should be the same as the one used on the interceptor) * @param attributeSource the attribute source (should be the same as the one used on the interceptor)
*/ */
public MethodDefinitionSourceAdvisor(String adviceBeanName, MethodDefinitionSource attributeSource) { public MethodDefinitionSourceAdvisor(String adviceBeanName, MethodDefinitionSource attributeSource) {
Assert.notNull(adviceBeanName, "The adviceBeanName cannot be null"); Assert.notNull(adviceBeanName, "The adviceBeanName cannot be null");
Assert.notNull(attributeSource, "The attributeSource cannot be null"); Assert.notNull(attributeSource, "The attributeSource cannot be null");
this.adviceBeanName = adviceBeanName; this.adviceBeanName = adviceBeanName;
this.attributeSource = attributeSource; this.attributeSource = attributeSource;
} }
//~ Methods ======================================================================================================== //~ Methods ========================================================================================================
public Pointcut getPointcut() { public Pointcut getPointcut() {
return pointcut; return pointcut;
} }
public Advice getAdvice() { public Advice getAdvice() {
synchronized (this.adviceMonitor) { synchronized (this.adviceMonitor) {
if (interceptor == null) { if (interceptor == null) {
Assert.notNull(adviceBeanName, "'adviceBeanName' must be set for use with bean factory lookup."); Assert.notNull(adviceBeanName, "'adviceBeanName' must be set for use with bean factory lookup.");
Assert.state(beanFactory != null, "BeanFactory must be set to resolve 'adviceBeanName'"); Assert.state(beanFactory != null, "BeanFactory must be set to resolve 'adviceBeanName'");
interceptor = (MethodSecurityInterceptor) interceptor = (MethodSecurityInterceptor)
beanFactory.getBean(this.adviceBeanName, MethodSecurityInterceptor.class); beanFactory.getBean(this.adviceBeanName, MethodSecurityInterceptor.class);
// attributeSource = interceptor.getObjectDefinitionSource(); }
} return interceptor;
return interceptor; }
} }
}
public void setBeanFactory(BeanFactory beanFactory) throws BeansException { public void setBeanFactory(BeanFactory beanFactory) throws BeansException {
this.beanFactory = beanFactory; this.beanFactory = beanFactory;
} }
//~ Inner Classes ================================================================================================== //~ Inner Classes ==================================================================================================
@ -153,7 +152,7 @@ public class MethodDefinitionSourceAdvisor extends AbstractPointcutAdvisor imple
} }
public Object getThis() { public Object getThis() {
return this.targetClass; return this.targetClass;
} }
public Object proceed() throws Throwable { public Object proceed() throws Throwable {