You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ws.apache.org by "JC (JIRA)" <ji...@apache.org> on 2017/04/18 19:06:42 UTC

[jira] [Commented] (AXIOM-397) OMChildrenNamespaceIterator.isEqual ignores second parameter

    [ 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