Uploaded image for project: 'Axiom'
  1. Axiom
  2. AXIOM-376

Axiom should report conflicting namespace declarations

    Details

    • Type: Improvement
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 1.2.12
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Consider the following code:

      OMFactory factory = OMAbstractFactory.getOMFactory();
      OMNamespace ns = factory.createOMNamespace("urn:ns1", "p");
      OMElement parent = factory.createOMElement("parent", ns);
      OMElement child = factory.createOMElement("child", ns, parent);
      child.declareNamespace("urn:ns2", "p");
      System.out.println("XML = " + parent);
      System.out.println("URI = " + child.getNamespaceURI());
      

      Both elements are created in the urn:ns1 namespace. An attempt is made to add a namespace declaration that binds the prefix to a different namespace. No error occurs and the output is as follows:

      XML = <p:parent xmlns:p="urn:ns1"><p:child xmlns:p="urn:ns2" /></p:parent>
      URI = urn:ns1
      

      This means that in the serialized document, the namespace of the child element is different than in the object model. This situation (which is expected to be accidental in most cases) will lead to subtle issues later.

      Axiom should make sure that the namespaces of element and attribute information items are always preserved and trigger an exception if this constraint is violated by a conflicting namespace declaration.

      These checks can be implemented at two levels:

      • In the serializer. This is the most robust solution, but makes it more difficult to identify the code that is responsible for the conflicting declaration (because it is only detected later).
      • In the declareNamespace and declareDefaultNamespace methods.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                veithen Andreas Veithen
                Reporter:
                veithen Andreas Veithen
              • Votes:
                1 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: