Changeset 37096

Timestamp:

2023-01-04T21:00:34+13:00 (16 months ago)

Author:

davidb

Message:

More careful typecasting for Element as we can now have TEXT_NODE reach here (which can occur when CGI arg 'excerptid-text' is used)

File:

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

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/LibraryServlet.java

@@ -1322 +1322 @@
     data = docNode.getDocumentElement();
       }
-    else
+    else if (nodeType == Node.ELEMENT_NODE)
       {
     data = (Element) dataNode;
       }
 
+    // Normally 'dataNode' is an Element or Document, but it is possible for it to be one of the other Node types
+    // (e.g., Node.TEXT_NODE, which is generated when using excerptid-text=xxxx)
+    // For these simpler cases, there is no URL encoding work to be done, so leaving 'data' as null is sufficient
+    
     if (data != null)
       {