DO NOT REPLY [Bug 14792] New: - HTMLTableRowElement.cloneNode() incorrectly copies _cells

[bu...@apache.org]

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=14792>.
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=14792

HTMLTableRowElement.cloneNode() incorrectly copies _cells

           Summary: HTMLTableRowElement.cloneNode() incorrectly copies
                    _cells
           Product: Xerces2-J
           Version: 2.2.0
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: DOM
        AssignedTo: xerces-j-dev@xml.apache.org
        ReportedBy: bsutton@noojee.com.au


If you attempt to clone (deep) an existing row the _cells private member 
variable is also cloned. The problem is that (provided _cells has been 
initialised by a call to getCells()) _cells points to the original row. Any 
future calls to getCells() on the new row will return the cell information 
from the original row. 

Suggested fix:
During the call to cloneNode() _cells should be set to null. The next call to 
getCells() will then correctly initialise it.

Implement HTMLTableRowElement.cloneNode(boolean deep) as follows:

public Node HTMLTableRowElement.cloneNode(boolean deep)
{
   Node newNode = super.cloneNode(deep); 
   newNode._cells = null;
   return newNode;
}

---------------------------------------------------------------------
To unsubscribe, e-mail: xerces-j-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xerces-j-dev-help@xml.apache.org