You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by pr...@apache.org on 2001/02/09 12:49:43 UTC

cvs commit: xml-cocoon/src/org/apache/cocoon/xml/dom DOMBuilder.java

prussell    01/02/09 03:49:42

  Modified:    src/org/apache/cocoon/xml/dom Tag: xml-cocoon2
                        DOMBuilder.java
  Log:
  Disabling the recreation of the Document node if a DTD isn't specified as
  it is causing DOM006 Document hierachy errors. I'll try and refactor the
  DOMBuilder sometime soon, as I have a feeling it may be overcomplicating
  the issue.
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.1.2.10  +6 -1      xml-cocoon/src/org/apache/cocoon/xml/dom/Attic/DOMBuilder.java
  
  Index: DOMBuilder.java
  ===================================================================
  RCS file: /home/cvs/xml-cocoon/src/org/apache/cocoon/xml/dom/Attic/DOMBuilder.java,v
  retrieving revision 1.1.2.9
  retrieving revision 1.1.2.10
  diff -u -r1.1.2.9 -r1.1.2.10
  --- DOMBuilder.java	2001/01/23 17:59:39	1.1.2.9
  +++ DOMBuilder.java	2001/02/09 11:49:40	1.1.2.10
  @@ -40,7 +40,7 @@
    *
    * @author <a href="mailto:fumagalli@exoffice.com">Pierpaolo Fumagalli</a>
    *         (Apache Software Foundation, Exoffice Technologies)
  - * @version CVS $Revision: 1.1.2.9 $ $Date: 2001/01/23 17:59:39 $
  + * @version CVS $Revision: 1.1.2.10 $ $Date: 2001/02/09 11:49:40 $
    */
   public class DOMBuilder implements XMLConsumer, Loggable {
       protected Logger log;
  @@ -243,6 +243,10 @@
                                             "'"+location());
               // Recreate the document since no DTD was specified
               } else {
  +				/* DISABLED pending us tracking down what's causing document 
  +				 * hierachy errors when we do this. This may well break people's
  +				 * code, but hopefully that will provide some enlightenment
  +				 * anyhow.
                   // Recreate the document element
                   Document doc=this.factory.newDocument(n.getQName());
                   // Copy the old document root PIs
  @@ -253,6 +257,7 @@
                   }
                   // Declare the new document as the new real document
                   this.document=doc;
  +				*/
                   this.current=this.document;
               }
               // Change the state before continuing