You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by bu...@apache.org on 2003/05/16 07:10:36 UTC

DO NOT REPLY [Bug 19977] - MessageElement.setDirty does not propagate up and trigger re-serialization

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=19977>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=19977

MessageElement.setDirty does not propagate up and trigger re-serialization

chris.haddad@cobia.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |axis-dev@ws.apache.org
         AssignedTo|axis-dev@ws.apache.org      |chris.haddad@cobia.net
   Target Milestone|---                         |post-1.1



------- Additional Comments From chris.haddad@cobia.net  2003-05-16 05:10 -------
Proposed fix is to have the MessageElement.setDirty method call up to the 
setDirty method of a registered parent.  

It is proposed that propagating a clear of the dirty bit (false) would be 
incorrect, but setting the dirty bit (true) up the hierarchy is correct.

Here's the diff:

Index: MessageElement.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/message/MessageElement.jav
a,v
retrieving revision 1.152
diff -r1.152 MessageElement.java
312c312,318
<     public void setDirty(boolean dirty) { _isDirty = dirty; }
---
>     public void setDirty(boolean dirty) {
>           _isDirty = dirty;
>           if (dirty == true) {
>             if (parent != null)
>               parent.setDirty(dirty);
>             }
>         }