You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commons-dev@ws.apache.org by "Alexander Veit (JIRA)" <ji...@apache.org> on 2006/12/10 23:09:21 UTC

[jira] Created: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
----------------------------------------------------------------------------------

                 Key: WSCOMMONS-140
                 URL: http://issues.apache.org/jira/browse/WSCOMMONS-140
             Project: WS-Commons
          Issue Type: Bug
            Reporter: Alexander Veit


javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.

See also WSCOMMONS-139.

-- 
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: commons-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: commons-dev-help@ws.apache.org


[jira] Commented: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

Posted by "Glen Daniels (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WSCOMMONS-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12485025 ] 

Glen Daniels commented on WSCOMMONS-140:
----------------------------------------

Hm.  OK, I'm working on guarding against this, but it could be argued that new QName(uri, local, null) should mean precisely the same thing as new QName(uri, local)... doesn't the IllegalArgumentException seem a little broken to you?


> org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
> ----------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-140
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-140
>             Project: WS-Commons
>          Issue Type: Bug
>            Reporter: Alexander Veit
>         Assigned To: Glen Daniels
>
> javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.
> See also WSCOMMONS-139.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


[jira] Updated: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

Posted by "Andreas Veithen (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WSCOMMONS-140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andreas Veithen updated WSCOMMONS-140:
--------------------------------------

    Component/s: AXIOM

> org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
> ----------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-140
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-140
>             Project: WS-Commons
>          Issue Type: Bug
>          Components: AXIOM
>            Reporter: Alexander Veit
>            Assignee: Glen Daniels
>
> javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.
> See also WSCOMMONS-139.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

Posted by "Alexander Veit (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WSCOMMONS-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12485323 ] 

Alexander Veit commented on WSCOMMONS-140:
------------------------------------------

> ... doesn't the IllegalArgumentException seem a little broken to you?

The IllegalArgumentException should probably enforce explicit coding and ensure fail-fast behaviour of code that uses the QName class. To me, this seems to be a good practice.

However, in my opinion, QName shouldn't deal with prefixes at all. The semantics of namespace qualified names is solely determined by the namespace URI and the local name*. Prefixes come only into play when we have to process or create a serialized form of XML. In these cases, since namespace to prefix mappings are generally not element-local, it is more natural to handle these mappings in separate data structures.

-- 
Just my 2 cents
Alex

* sometimes we may need the ancestor:: axis in an instance document, too

> org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
> ----------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-140
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-140
>             Project: WS-Commons
>          Issue Type: Bug
>            Reporter: Alexander Veit
>         Assigned To: Glen Daniels
>
> javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.
> See also WSCOMMONS-139.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


[jira] Commented: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

Posted by "Glen Daniels (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WSCOMMONS-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12485329 ] 

Glen Daniels commented on WSCOMMONS-140:
----------------------------------------

> However, in my opinion, QName shouldn't deal with prefixes at all.

No argument from me! :)  Here's a quick blast from the past from 2003....

http://mail-archives.apache.org/mod_mbox/ws-axis-dev/200309.mbox/<0a...@lankabook2>

Alas, we lost that battle.



> org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
> ----------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-140
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-140
>             Project: WS-Commons
>          Issue Type: Bug
>            Reporter: Alexander Veit
>         Assigned To: Glen Daniels
>
> javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.
> See also WSCOMMONS-139.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


[jira] Commented: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

Posted by "Alexander Veit (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WSCOMMONS-140?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12485338 ] 

Alexander Veit commented on WSCOMMONS-140:
------------------------------------------

LOL. I'm sorry for trying to give you a lecture :) I should rather file a feature request at Sun to deprecate prefixes in QName :)

> org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
> ----------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-140
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-140
>             Project: WS-Commons
>          Issue Type: Bug
>            Reporter: Alexander Veit
>         Assigned To: Glen Daniels
>
> javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.
> See also WSCOMMONS-139.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


[jira] Resolved: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

Posted by "Glen Daniels (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WSCOMMONS-140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Glen Daniels resolved WSCOMMONS-140.
------------------------------------

    Resolution: Fixed

Fixed.

> org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
> ----------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-140
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-140
>             Project: WS-Commons
>          Issue Type: Bug
>            Reporter: Alexander Veit
>         Assigned To: Glen Daniels
>
> javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.
> See also WSCOMMONS-139.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


[jira] Updated: (WSCOMMONS-140) org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes

Posted by "Davanum Srinivas (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WSCOMMONS-140?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Davanum Srinivas updated WSCOMMONS-140:
---------------------------------------

    Assignee: Glen Daniels

> org.apache.axiom.om.impl.dom.AttrImpl#getQName() fails for null namespace prefixes
> ----------------------------------------------------------------------------------
>
>                 Key: WSCOMMONS-140
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-140
>             Project: WS-Commons
>          Issue Type: Bug
>            Reporter: Alexander Veit
>         Assigned To: Glen Daniels
>
> javax.xml.namespace.QName implementations normally throw an IllegalArgumentException if a null namespace prefix is given as a construction parameter.
> See also WSCOMMONS-139.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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