cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 27678] New: - AbstractXMLProducer.setConsumer implementation is incorrect
Date Mon, 15 Mar 2004 18:37:35 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=27678>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=27678

AbstractXMLProducer.setConsumer implementation is incorrect

           Summary: AbstractXMLProducer.setConsumer implementation is
                    incorrect
           Product: Cocoon 2
           Version: Current CVS 2.1
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: core
        AssignedTo: dev@cocoon.apache.org
        ReportedBy: peter.brant@courts.state.wi.us


The implementation of org.apache.cocoon.xml.AbstractXMLProducer.setConsumer does
not match its Javadoc.  Patch is below.  

Current workaround is to include implementation specified in Javadoc in subclass.

Index: AbstractXMLProducer.java
===================================================================
RCS file:
/home/cvspublic/cocoon-2.1/src/java/org/apache/cocoon/xml/AbstractXMLProducer.java,v
retrieving revision 1.1
diff -u -r1.1 AbstractXMLProducer.java
--- AbstractXMLProducer.java	9 Mar 2003 00:09:45 -0000	1.1
+++ AbstractXMLProducer.java	15 Mar 2004 18:24:27 -0000
@@ -84,8 +84,8 @@
      */
     public void setConsumer(XMLConsumer consumer) {
         this.xmlConsumer = consumer;
-        this.contentHandler = consumer;
-        this.lexicalHandler = consumer;
+        setContentHandler(consumer);
+        setLexicalHandler(consumer);
     }
 
     /**

Mime
View raw message