ws-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "JC (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (AXIOM-397) OMChildrenNamespaceIterator.isEqual ignores second parameter
Date Tue, 18 Apr 2017 19:06:42 GMT

    [ https://issues.apache.org/jira/browse/AXIOM-397?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15973303#comment-15973303
] 

JC commented on AXIOM-397:
--------------------------

Hi
In a recent GitHub mirror for webservices-axiom (commit id: 0214fd05425499c74f8462535d03279330059f7e),
this problem is still there in this path: axiom-compat/src/main/java/org/apache/axiom/om/impl/traverse/OMChildrenNamespaceIterator.java
{code}
 45     @Override
 46     public boolean isEqual(QName searchQName, QName currentQName) {
 47         return searchQName.getNamespaceURI().equals(searchQName.getNamespaceURI());
 48     }
{code}
I was wondering if this should be fixed or this is just intended. Thanks!

> OMChildrenNamespaceIterator.isEqual ignores second parameter
> ------------------------------------------------------------
>
>                 Key: AXIOM-397
>                 URL: https://issues.apache.org/jira/browse/AXIOM-397
>             Project: Axiom
>          Issue Type: Bug
>          Components: DOOM, LLOM
>    Affects Versions: 1.2.9, 1.2.12
>         Environment: N/A
>            Reporter: Simon Matic Langford
>            Assignee: Andreas Veithen
>             Fix For: 1.2.13
>
>
> Whilst debugging an issue with SOAPHeader.getChildrenWithNamespaceURI() we discovered
that OMChildrenNamespaceIterator.isEqual() ignores the "currentQName" parameter. This renders
it rather useless. We found on 1.2.9, but have verified it's not fixed in 1.2.12 and can't
see an existing issue for it in any of the unreleased versions:
>     public boolean isEqual(QName searchQName, QName currentQName) {
>         return searchQName.getNamespaceURI().equals(searchQName.getNamespaceURI());
>     }
> should read:
>     public boolean isEqual(QName searchQName, QName currentQName) {
>         return searchQName.getNamespaceURI().equals(currentQName.getNamespaceURI());
>     }



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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


Mime
View raw message