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 ru...@apache.org on 2001/06/12 14:15:05 UTC
cvs commit: xml-axis/java/src/org/apache/axis/utils Admin.java
rubys 01/06/12 05:15:05
Modified: java/src/org/apache/axis/utils Admin.java
Log:
The assumption that localName == tagName for un-namespace qualified
elements does not appear to hold across parser implementations. The
updated code works with Crimson, Xerces 1, and Xerces 2.
Revision Changes Path
1.38 +3 -3 xml-axis/java/src/org/apache/axis/utils/Admin.java
Index: Admin.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/utils/Admin.java,v
retrieving revision 1.37
retrieving revision 1.38
diff -u -r1.37 -r1.38
--- Admin.java 2001/06/11 10:51:51 1.37
+++ Admin.java 2001/06/12 12:15:05 1.38
@@ -114,7 +114,7 @@
Node node = list.item(i);
if ( node.getNodeType() != Node.ELEMENT_NODE ) continue ;
Element elem = (Element) node ;
- if ( !"option".equals(elem.getLocalName()) ) continue ;
+ if ( !"option".equals(elem.getTagName()) ) continue ;
String name = elem.getAttribute( "name" );
String value = elem.getAttribute( "value" );
@@ -156,7 +156,7 @@
try {
init();
AxisClassLoader cl = AxisClassLoader.getClassLoader();
- String action = root.getLocalName();
+ String action = root.getTagName();
if ( !action.equals("deploy") && !action.equals("undeploy") &&
!action.equals("list") && !action.equals("quit") )
@@ -241,7 +241,7 @@
if ( node.getNodeType() != Node.ELEMENT_NODE ) continue ;
Element elem = (Element) node ;
- String type = elem.getLocalName();
+ String type = elem.getTagName();
String name = elem.getAttribute( "name" );
if ( name != null && name.equals("") ) name = null ;
Re: cvs commit: xml-axis/java/src/org/apache/axis/utils Admin.java
Posted by Glen Daniels <gd...@macromedia.com>.
> The assumption that localName == tagName for un-namespace qualified
> elements does not appear to hold across parser implementations. The
> updated code works with Crimson, Xerces 1, and Xerces 2.
Interesting! What were you getting before?
--G