[prev in list] [next in list] [prev in thread] [next in thread] 

List:       xmlrpc-user
Subject:    [jira] [Resolved] (AXIOM-375) Simplify the OMElement#declareNamespace(String, String) API
From:       "Andreas Veithen (JIRA)" <jira () apache ! org>
Date:       2017-06-04 10:18:04
Message-ID: JIRA.12515007.1311326346000.360863.1496571484155 () Atlassian ! JIRA
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/AXIOM-375?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Andreas Veithen resolved AXIOM-375.
-----------------------------------
    Resolution: Fixed
      Assignee: Andreas Veithen

> Simplify the OMElement#declareNamespace(String, String) API
> -----------------------------------------------------------
> 
> Key: AXIOM-375
> URL: https://issues.apache.org/jira/browse/AXIOM-375
> Project: Axiom
> Issue Type: Improvement
> Components: API
> Reporter: Andreas Veithen
> Assignee: Andreas Veithen
> Priority: Minor
> Fix For: 1.3.0
> 
> 
> OMElement#declareNamespace(String, String) auto-generates a prefix if the prefix is \
> either null or the empty string. There are several issues with this: 1. It is \
> inconsistent with other APIs (in particular OMElement#declareNamespace(OMNamespace) \
> which make a distinction between null (indicating that a prefix should be \
> generated) and the empty string (meaning the default namespace). 2. It is not \
> possible to use OMElement#declareNamespace(String, String) to declare a default \
> namespace. This is especially annoying when the prefix is provided as a variable, \
> in which case one has to write something like this: if (prefix.length() == 0) {
> nextElem.declareDefaultNamespace(uri);
> } else {
> nextElem.declareNamespace(uri, prefix);
> }
> 3. It is obviously not necessary to support two different ways to indicate that a \
> prefix should be generated. One (prefix == null) should be enough. This should be \
> changed in the next major release (1.3). To make the transition easier, the Javadoc \
> in 1.2.x should be updated to indicate that using an empty string is deprecated and \
> the code should generate a warning if the method is invoked in this way.



--
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


[prev in list] [next in list] [prev in thread] [next in thread]