Changeset 16688 for greenstone3/trunk/src/java/org/greenstone/gsdl3/action/Action.java

Timestamp:

2008-08-08T13:23:59+12:00 (16 years ago)

Author:

davidb

Message:

Changed 'Element process(Element)' in ModuleInterface to 'Node process(Node)'. After some deliberation is was decided this is a more useful (generic) layer of the DOM to pass information around in. Helps with the DocType problem when producing XSL Transformed pages, for example. When this was an Element, it would loose track of its DocType. Supporting method provided in XMLConverter 'Element nodeToElement(Node)' which checks a nodes docType and casts to Element if appropriate, or if a Document, typecasts to that and then extracts the top-level Element. With this fundamental change in ModuleInterface, around 20 files needed to be updated (Actions, Services, etc) that build on top of 'process()' to reflect this change, and use nodeToElement where necessary.

File:

• greenstone3/trunk/src/java/org/greenstone/gsdl3/action/Action.java (modified) (1 diff)

greenstone3/trunk/src/java/org/greenstone/gsdl3/action/Action.java

@@ -62 +62 @@
         return null;
     }
-    Element result = process(message_doc.getDocumentElement());
+    Node result = process(message_doc);
     return this.converter.getString(result);
     }
     
     /** the main process method - must be implemented in subclass */
-    abstract public Element process(Element xml_in);
+    abstract public Node process(Node xml_in);
 
     /** tell the param class what its arguments are