You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by st...@apache.org on 2013/08/29 11:55:38 UTC

svn commit: r1518555 - /sling/trunk/tooling/ide/eclipse-ui/src/org/apache/sling/ide/eclipse/ui/nav/model/ModifiableProperties.java

Author: stefanegli
Date: Thu Aug 29 09:55:38 2013
New Revision: 1518555

URL: http://svn.apache.org/r1518555
Log:
SLING-2985 : npe-fix for properties handling

Modified:
    sling/trunk/tooling/ide/eclipse-ui/src/org/apache/sling/ide/eclipse/ui/nav/model/ModifiableProperties.java

Modified: sling/trunk/tooling/ide/eclipse-ui/src/org/apache/sling/ide/eclipse/ui/nav/model/ModifiableProperties.java
URL: http://svn.apache.org/viewvc/sling/trunk/tooling/ide/eclipse-ui/src/org/apache/sling/ide/eclipse/ui/nav/model/ModifiableProperties.java?rev=1518555&r1=1518554&r2=1518555&view=diff
==============================================================================
--- sling/trunk/tooling/ide/eclipse-ui/src/org/apache/sling/ide/eclipse/ui/nav/model/ModifiableProperties.java (original)
+++ sling/trunk/tooling/ide/eclipse-ui/src/org/apache/sling/ide/eclipse/ui/nav/model/ModifiableProperties.java Thu Aug 29 09:55:38 2013
@@ -91,9 +91,11 @@ public class ModifiableProperties implem
 	public void setNode(GenericJcrRootFile genericJcrRootFile, Node domNode) {
 		this.domNode = domNode;
 		NamedNodeMap attributes = domNode.getAttributes();
-		for(int i=0; i<attributes.getLength(); i++) {
-			Node attr = attributes.item(i);
-			properties.put(attr.getNodeName(), attr.getNodeValue());
+		if (attributes!=null) {
+			for(int i=0; i<attributes.getLength(); i++) {
+				Node attr = attributes.item(i);
+				properties.put(attr.getNodeName(), attr.getNodeValue());
+			}
 		}
 		this.genericJcrRootFile = genericJcrRootFile;
 	}