Mailing-List: contact axis-dev-help@ws.apache.org; run by ezmlm
Precedence: bulk
Reply-To: axis-dev@ws.apache.org
Message-ID: <9661226.1150880431326.JavaMail.jira@brutus>
Date: Wed, 21 Jun 2006 09:00:31 +0000 (GMT+00:00)
From: =?utf-8?Q?Ulrich_K=C3=BCster_=28JIRA=29?= <axis-dev@ws.apache.org>
To: axis-dev@ws.apache.org
Subject: [jira] Created: (AXIS-2507) org.apache.axis.message.Text needs
 cloneNode-implementation OR
 org.apache.axis.message.MessageElement.getElementsNS needs to be fixed
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

org.apache.axis.message.Text needs cloneNode-implementation OR org.apache.a=
xis.message.MessageElement.getElementsNS needs to be fixed
---------------------------------------------------------------------------=
----------------------------------------------------------

         Key: AXIS-2507
         URL: http://issues.apache.org/jira/browse/AXIS-2507
     Project: Apache Axis
        Type: Bug

  Components: Basic Architecture =20
    Versions: 1.4   =20
    Reporter: Ulrich K=C3=BCster


When you clone a Node using cloneNode(true) and later invoke getElementsByT=
agNameNS a class cast exception is thrown in getElementNS.

The reason for this is that getElementNS casts to Element if the object und=
er inspection is not an instance of Text. However, org.apache.axis.message.=
Text does not override NodeImpl.cloneNode(boolean deep). Thus, when I clone=
d my Node all Text nodes got replaced by NodeImpl nodes as the NodeImpl.clo=
neNode method was called (and this method returns instances of type NodeImp=
l).
Thus my clone contained nodes which were neither of type Element nore of ty=
pe Text but of type NodeImpl. Thus the class cast exception in getElementNS=
 (which assumes a node is of type Element if it's not of type Text).

I suggest so resolve this issue by implementing cloneNode in Text in a simi=
lar fashion it is implemented in NodeImpl:

    /**
     * Returns a duplicate of this node, i.e., serves as a generic copy
     * constructor for nodes. The duplicate node has no parent; (
     * <code>parentNode</code> is <code>null</code>.).
     * <br>Cloning an <code>Element</code> copies all attributes and their
     * values, including those generated by the XML processor to represent
     * defaulted attributes, but this method does not copy any text it
     * contains unless it is a deep clone, since the text is contained in a
     * child <code>Text</code> node. Cloning an <code>Attribute</code>
     * directly, as opposed to be cloned as part of an <code>Element</code>
     * cloning operation, returns a specified attribute (
     * <code>specified</code> is <code>true</code>). Cloning any other type
     * of node simply returns a copy of this node.
     * <br>Note that cloning an immutable subtree results in a mutable copy=
,
     * but the children of an <code>EntityReference</code> clone are readon=
ly
     * . In addition, clones of unspecified <code>Attr</code> nodes are
     * specified. And, cloning <code>Document</code>,
     * <code>DocumentType</code>, <code>Entity</code>, and
     * <code>Notation</code> nodes is implementation dependent.
     *=20
     * @param deep If <code>true</code>, recursively clone the subtree unde=
r
     *             the specified node; if <code>false</code>, clone only th=
e node
     *             itself (and its attributes, if it is an <code>Element</c=
ode>).
     * @return The duplicate node.
     */
    public Node cloneNode(boolean deep) {
        return new Text(textRep);
    }


--=20
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org