Uploaded image for project: 'Codehaus Cargo'
  1. CARGO-1209

Merging listeners if the root (<web-app>) of web.xml contains the xmlns attribute is buggy

    Details

    • Type: Bug
    • Status: Closed (View workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects versions: 1.4.2
    • Fix versions: 1.4.3
    • Components: Module API
    • Labels:
      None

      Description

      See: http://cargo.996258.n3.nabble.com/Maven-plugin-merging-listeners-in-web-xml-is-buggy-td17307.html

      In version 1.4.2 of the maven plugin there seems to be a bug when merging web.xml files containing multiple <listenerelements if the root (<web-app>) of web.xml contains the xmlns attrubute.

      To reproduce the bug:
      Create a junit test based on org.codehaus.cargo.module.webapp.merge.WebXmlListenerMergerTest and just modify the file1 and file2 Strings in "testMergeListeners()" so that the <web-appelements contain the xmlns attribute (e.g. xmlns="http://java.sun.com/xml/ns/javaee"). The test should fail.

      Fix:
      After some debugging I came to the conclusion that the tags array in org.codehaus.cargo.module.webapp.WebXml22Type is incomplete and adding new WebXmlTag(this, "listener-class"), to it fixes the bug for me.

      Marcin Piela

        Attachments

          Activity

            People

            • Assignee:
              savasalitokmen Savas Ali Tokmen (Inactive)
              Reporter:
              alitokmen S. Ali Tokmen
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: