--- projects/cms/source/util/uk/org/iscream/cms/util/XMLPacketMaker.java	2000/11/22 08:40:53	1.1
+++ projects/cms/source/util/uk/org/iscream/cms/util/XMLPacketMaker.java	2004/08/01 10:41:08	1.20
@@ -1,50 +1,117 @@
-import java.io.*;
-import java.util.ArrayList;
+/* 
+ * i-scream central monitoring system
+ * http://www.i-scream.org
+ * Copyright (C) 2000-2002 i-scream
+ * 
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; either version 2
+ * of the License, or (at your option) any later version.
+ * 
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ * 
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
+ */
 
+//---PACKAGE DECLARATION---
+package uk.org.iscream.cms.util;
+
+//---IMPORTS---
+import java.io.*;
 import org.xml.sax.*;
+import javax.xml.parsers.*;
 
-import javax.xml.parsers.SAXParserFactory;
-import javax.xml.parsers.ParserConfigurationException;
-import javax.xml.parsers.SAXParser;
+/**
+ * XMLPacketMaker - Creates an XMLPacket object.
+ *
+ * @author  $Author: tdb $
+ * @version $Id: XMLPacketMaker.java,v 1.20 2004/08/01 10:41:08 tdb Exp $
+ */
+public class XMLPacketMaker {
 
-// Paul Mutton, pjm2@ukc.ac.uk
+//---FINAL ATTRIBUTES---
 
-// XMLPacketMaker - Creates an XMLPacket object.
-public class XMLPacketMaker extends HandlerBase {
+    /**
+     * The current CVS revision of this class
+     */
+    public final String REVISION = "$Revision: 1.20 $";
 
-    // No-args constructor.  Generally not used.
-    public XMLPacketMaker () {
-        this.xml = "<packet></packet>";
-    }
+    /** 
+     * A reference to the system saxParser factory
+     */ 
+    private final SAXParserFactory _factory = SAXParserFactory.newInstance();
+     
+//---STATIC METHODS---
 
-    // Constructor for accepting XML input.
-    public XMLPacketMaker (String xml) {
-        this.xml = xml;
-    }
+//---CONSTRUCTORS---
 
-    public XMLPacket createXMLPacket() {
+//---PUBLIC METHODS---
 
+    /**
+     * Method to create an XML packet from the data this
+     * class was constructed with.
+     *
+     * @param xml the XML String to parse
+     * @return an XMLPacket representing the XML String given
+     * @throws InvalidXMLException if the XML cannot be parsed
+     */
+    public XMLPacket createXMLPacket(String xml) throws InvalidXMLException {
+
         // Create the XMLPacket to store values in.
         XMLPacket packet = new XMLPacket();
-
-        // Use the default (non-validating) parser
-        SAXParserFactory factory = SAXParserFactory.newInstance();
+        
         try {
-
             // Parse the input
             InputSource inputSource = new InputSource(new StringReader(xml));
-            SAXParser saxParser = factory.newSAXParser();
-            saxParser.parse(inputSource, new XMLStringParser(packet));
-
+            _factory.newSAXParser().parse(inputSource, new XMLStringParser(packet));
         }
         catch (Exception e) {
-            System.out.println("XMLPacketMaker - I just received an XML packet that did not contain valid XML.");
-            //t.printStackTrace();
-            return null;
+            // couldn't parse the XML for some reason
+            throw new InvalidXMLException("Could not parse the XML: "+xml);
         }
         
+        // parsed successfully, return the packet
         return packet;
     }
+
+    /**
+     * Overrides the {@link java.lang.Object#toString() Object.toString()}
+     * method to provide clean logging (every class should have this).
+     *
+     * This uses the uk.org.iscream.cms.server.util.NameFormat class
+     * to format the toString()
+     *
+     * @return the name of this class and its CVS revision
+     */
+    public String toString() {
+        return FormatName.getName(
+            _name,
+            getClass().getName(),
+            REVISION);
+    }
+
+//---PRIVATE METHODS---
+
+//---ACCESSOR/MUTATOR METHODS---
+
+//---ATTRIBUTES---
     
-    String xml;
-}
\ No newline at end of file
+    /**
+     * This is the friendly identifier of the
+     * component this class is running in.
+     * eg, a Filter may be called "filter1",
+     * If this class does not have an owning
+     * component,  a name from the configuration
+     * can be placed here.  This name could also
+     * be changed to null for utility classes.
+     */
+    private String _name = null;
+
+//---STATIC ATTRIBUTES---
+
+}