Browse Source

SEC-271: Replaced Java 5 specific code with pre Java 5

Vishal Puri 18 years ago
parent
commit
84a3c87ea4

+ 7 - 5
core/src/main/java/org/acegisecurity/config/LogoutHandlerOrderResolver.java

@@ -68,20 +68,22 @@ public class LogoutHandlerOrderResolver implements BeanFactoryPostProcessor {
 			RootBeanDefinition definition = (RootBeanDefinition) beanFactory.getBeanDefinition(names[i]);
 
 			if (Ordered.class.isAssignableFrom(definition.getBeanClass())) {
-				definition.getPropertyValues().addPropertyValue("order", getOrder(definition.getBeanClass()));
-				list.add(definition);
+				definition.getPropertyValues().addPropertyValue("order", new Integer(getOrder(definition.getBeanClass())));
+			} else {
+				definition.getPropertyValues().addPropertyValue("order", new Integer(Integer.MAX_VALUE));
 			}
+			list.add(definition);
 		}
 		Collections.sort(list, new OrderComparator());
 		return list;
 	}
-
+	
 	private int getOrder(Class clazz) {
 		if (clazz.getName().equals(TokenBasedRememberMeServices.class.getName())) {
-			return 0;
+			return 100;
 		}
 		if (clazz.getName().equals(SecurityContextLogoutHandler.class.getName())) {
-			return 1;
+			return 200;
 		}
 		return Integer.MAX_VALUE;
 	}

+ 2 - 2
core/src/main/java/org/acegisecurity/config/PrincipalRepositoryBeanDefinitionParser.java

@@ -197,10 +197,10 @@ public class PrincipalRepositoryBeanDefinitionParser extends AbstractBeanDefinit
 		String propertyValue = element.getAttribute(ENABLED);
 		if (StringUtils.hasText(propertyValue)) {
 			if (propertyValue.equals("true")) {
-				userAttribute.setEnabled(Boolean.TRUE);
+				userAttribute.setEnabled(true);
 			}
 			else {
-				userAttribute.setEnabled(Boolean.FALSE);
+				userAttribute.setEnabled(false);
 			}
 		}
 	}