From cb6215b2353e848cea054e52e37d1fe6c4756631 Mon Sep 17 00:00:00 2001 From: Robert von Burg Date: Wed, 9 Nov 2011 15:03:50 +0100 Subject: [PATCH] [Bugfix] fixed a bug where if the tag was not defined in the PrivilegeModel.xml file then a NullPointer was thrown. It is now possible to leave out this tag, in which case false is assumed --- .../eitchnet/privilege/handler/XmlPersistenceHandler.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/ch/eitchnet/privilege/handler/XmlPersistenceHandler.java b/src/ch/eitchnet/privilege/handler/XmlPersistenceHandler.java index b5a962f4d..6910ad735 100644 --- a/src/ch/eitchnet/privilege/handler/XmlPersistenceHandler.java +++ b/src/ch/eitchnet/privilege/handler/XmlPersistenceHandler.java @@ -491,8 +491,11 @@ public class XmlPersistenceHandler implements PersistenceHandler { String privilegeName = privilegeElement.attributeValue(XmlConstants.XML_ATTR_NAME); String privilegePolicy = privilegeElement.attributeValue(XmlConstants.XML_ATTR_POLICY); - String allAllowedS = privilegeElement.element(XmlConstants.XML_ALL_ALLOWED).getTextTrim(); - boolean allAllowed = Boolean.valueOf(allAllowedS).booleanValue(); + Element allAllowedE = privilegeElement.element(XmlConstants.XML_ALL_ALLOWED); + boolean allAllowed = false; + if (allAllowedE != null) { + allAllowed = Boolean.valueOf(allAllowedE.getTextTrim()).booleanValue(); + } @SuppressWarnings("unchecked") List denyElements = privilegeElement.elements(XmlConstants.XML_DENY);