[Yanel-commits] rev 33459 - public/yanel/trunk/src/contributions/resources/data-repo-sitetree/src/java/org/wyona/yanel/impl/resources

[michi at wyona.com]

Author: michi
Date: 2008-03-16 23:39:46 +0100 (Sun, 16 Mar 2008)
New Revision: 33459

Modified:
   public/yanel/trunk/src/contributions/resources/data-repo-sitetree/src/java/org/wyona/yanel/impl/resources/DataRepoSitetreeResource.java
Log:
formatting and error message fixed

Modified: public/yanel/trunk/src/contributions/resources/data-repo-sitetree/src/java/org/wyona/yanel/impl/resources/DataRepoSitetreeResource.java
===================================================================
--- public/yanel/trunk/src/contributions/resources/data-repo-sitetree/src/java/org/wyona/yanel/impl/resources/DataRepoSitetreeResource.java	2008-03-16 22:38:18 UTC (rev 33458)
+++ public/yanel/trunk/src/contributions/resources/data-repo-sitetree/src/java/org/wyona/yanel/impl/resources/DataRepoSitetreeResource.java	2008-03-16 22:39:46 UTC (rev 33459)
@@ -124,25 +124,27 @@
         */
 
         if (node != null) {
-        if (node.isCollection()) {
-            sb.append("<collection path=\"" + path + "\" name=\" " + node.getName() + "\">");
-            Node[] children = node.getChildren();
-            for (int i = 0; i < children.length; i++) {
-                if (children[i].isCollection()) {
-                    sb.append(getNodeAsXML(children[i].getPath()));
-                } else if (children[i].isResource()) {
-                    sb.append("<resource path=\"" + children[i].getPath() + "\" name=\"" + children[i].getName() + "\"/>");
-                } else {
-                    sb.append("<neither-resource-nor-collection path=\"" + children[i].getPath() + "\" name=\"" + children[i].getName() + "\"/>");
+            if (node.isCollection()) {
+                sb.append("<collection path=\"" + path + "\" name=\" " + node.getName() + "\">");
+                Node[] children = node.getChildren();
+                for (int i = 0; i < children.length; i++) {
+                    if (children[i].isCollection()) {
+                        sb.append(getNodeAsXML(children[i].getPath()));
+                    } else if (children[i].isResource()) {
+                        sb.append("<resource path=\"" + children[i].getPath() + "\" name=\"" + children[i].getName() + "\"/>");
+                    } else {
+                        sb.append("<neither-resource-nor-collection path=\"" + children[i].getPath() + "\" name=\"" + children[i].getName() + "\"/>");
+                    }
                 }
+                sb.append("</collection>");
+            } else {
+                sb.append("<resource path=\"" + path + "\" name=\"" + node.getName() + "\"/>");
             }
-            sb.append("</collection>");
         } else {
-            sb.append("<resource path=\"" + path + "\" name=\"" + node.getName() + "\"/>");
+            String errorMessage = "node is null for path: " + path;
+            sb.append("<exception>" + errorMessage + "</exception>");
+            log.error(errorMessage);
         }
-        } else {
-            sb.append("<exception>node is null for path: " + path + "</exception>");
-        }
         return sb.toString();
     }
 }