Mailing-List: contact issues-help@cxf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cxf.apache.org
Message-ID: <1306138923.1257338972938.JavaMail.jira@brutus>
Date: Wed, 4 Nov 2009 12:49:32 +0000 (UTC)
From: "Hasan Hosgel (JIRA)" <jira@apache.org>
To: issues@cxf.apache.org
Subject: [jira] Created: (CXF-2520) wrong handling in setNamespaceMap for
 AbstractDataBinding
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

wrong handling in setNamespaceMap for AbstractDataBinding
---------------------------------------------------------

                 Key: CXF-2520
                 URL: https://issues.apache.org/jira/browse/CXF-2520
             Project: CXF
          Issue Type: Bug
          Components: JAXB Databinding
    Affects Versions: 2.2.4
            Reporter: Hasan Hosgel


The Class org.apache.cxf.databinding.AbstractDataBinding has a Bug in the checking of duplicate NamespacePrefixes during the setting of the namespaceMap.

Extraction of class:
    public void setNamespaceMap(Map<String, String> namespaceMap) {
        // make some checks. This is a map from namespace to prefix, but we want unique prefixes.
        if (namespaceMap != null) {
            Set<String> prefixesSoFar = new HashSet<String>();
            for (Map.Entry<String, String> mapping : namespaceMap.entrySet()) {
                if (prefixesSoFar.contains(mapping.getValue())) {
                    throw new IllegalArgumentException("Duplicate prefix " + mapping.getValue());
                }
            }
        }
        this.namespaceMap = namespaceMap;
    }

There is no adding method for prefixesSoFar-Set.

My solving idea is:
  /**
   * @param namespaceMap The namespaceMap to set.
   */
  public void setNamespaceMap(Map<String, String> namespaceMap) {
    // make some checks. This is a map from namespace to prefix, but we want unique prefixes.
    if ((namespaceMap == null) || namespaceMap.isEmpty()) {
      throw new IllegalArgumentException("adding a null or empty namespaceMap is not allowed");
    } else {
      Set<String> prefixesSoFar = new HashSet<String>();
      for (Map.Entry<String, String> mapping : namespaceMap.entrySet()) {
        if (prefixesSoFar.contains(mapping.getValue())) {
          throw new IllegalArgumentException("Duplicate prefix " + mapping.getValue());
        }
        prefixesSoFar.add(mapping.getValue());
      }
      mapper = new NamespaceMapper(namespaceMap);
    }
  }

This solution is more strict, because it does not allow to add a null or an empty Map.

The same Bug goes for checkNameSpaceMap. 
The both method are smelling, because it is copy paste. It can be solved in a smater way :).


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