Changeset 23489 for main/trunk/greenstone3/src/java/org/greenstone/gsdl3/util/GSXML.java

Timestamp:

2010-12-14T17:15:30+13:00 (13 years ago)

Author:

sjb48

Message:

Working on saving a format statement to the config file. This looks like it should work but changes in FormatAction have caused major issues with Document/Node violations. This is because saving required the format statement as a Document rather than a string (which did work). Added a check to GSXML for xmlNodeToString to check that attrs is not null (attrs will be null unless the node is an element).

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/util/GSXML.java (modified) (1 diff)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/util/GSXML.java

@@ -921 +921 @@
     sb.append(e.getNodeName());
     NamedNodeMap attrs = e.getAttributes();
-    for (int i = 0; i < attrs.getLength(); i++) {
-      Node attr = attrs.item(i);
-      sb.append(' ');
-      sb.append(attr.getNodeName());
-      sb.append("=\"");
-      sb.append(attr.getNodeValue());
-      sb.append('"');
+    if(attrs != null)
+    {
+        for (int i = 0; i < attrs.getLength(); i++) {
+            Node attr = attrs.item(i);
+            sb.append(' ');
+            sb.append(attr.getNodeName());
+            sb.append("=\"");
+            sb.append(attr.getNodeValue());
+            sb.append('"');
+        }
     }