Hi Venkat.

Venkat S Kota:
> I have downloaded the source distribution and found that the attribute 
> node value is ignored while creating a new node in the importNode() 
> implementation in the AbstractDocument class. See the code snippets 
> below.. I presume the bug Cameron mentioned is still existing but I 
> couldn't find any open bugs describing such behaviour. Can someone 
> explain if my understanding is incorrect?

I think you may be right!  I did have a quick look through
AbstractDocument.importNode yesterday but I missed the fact that it
didn't set the new attribute node's value.

The obvious solution is to add one line:

> AbstractDocument.java
> ------------------------
>    public Node importNode(Node importedNode, boolean deep)
>        throws DOMException {
>        ...
>        Node result;
>        switch (importedNode.getNodeType()) {
>        case ELEMENT_NODE:
>        ...
>           
>        case ATTRIBUTE_NODE:
>            result = createAttributeNS(importedNode.getNamespaceURI(),
>                                       importedNode.getNodeName());
             result.setNodeValue(importedNode.getNodeValue());
>            break;

What with the inactivity of the Batik committers at the moment, who
knows when this will be in CVS.

Cameron

-- 
Cameron McCormack
|  Web: http://mcc.id.au/
|  ICQ: 26955922

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to