[jira] Commented: (AXIS-1307) SOAPElement and Text should be mutually exclusive

[ji...@apache.org]

The following comment has been added to this issue:

     Author: Davanum Srinivas
    Created: Tue, 13 Apr 2004 7:12 AM
       Body:
can you please submit a patch?
---------------------------------------------------------------------
View this comment:
  http://issues.apache.org/jira/browse/AXIS-1307?page=comments#action_34993

---------------------------------------------------------------------
View the issue:
  http://issues.apache.org/jira/browse/AXIS-1307

Here is an overview of the issue:
---------------------------------------------------------------------
        Key: AXIS-1307
    Summary: SOAPElement and Text should be mutually exclusive
       Type: Improvement

     Status: Unassigned
   Priority: Major

    Project: Axis
 Components: 
             Basic Architecture
   Versions:
             current (nightly)

   Assignee: 
   Reporter: Brian Ewins

    Created: Tue, 13 Apr 2004 4:49 AM
    Updated: Tue, 13 Apr 2004 7:12 AM

Description:
In SAAJ, javax.xml.soap.Text does not extend javax.xml.soap.SOAPElement; having it do so would be counterintuitive when compared to the DOM equivalents. However, in axis, org.apache.axis.message.Text extends org.apache.axis.message.MessageElement, which implements SOAPElement.

This means that to find element nodes which /actually are/ elements you have to do an extra test... 

if (!(obj instanceof Text)) {
...
}

A possible solution: move the "implements SOAPElement" from MessageElement to a new class org.apache.axis.message.SOAPElement extends MessageElement, with an empty class body. Since the other SOAPElement implementations in the package already "implement SOAPElement" by virtue of implementing SOAPBodyElement, Detail, etc, this should be the only change required.



---------------------------------------------------------------------
JIRA INFORMATION:
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

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira