Uploaded image for project: 'Commons JXPath'
  1. Commons JXPath
  2. JXPATH-123

Regression: "fix" made to NamespaceResolver broke existing testcases

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.2 Final
    • 1.3
    • None

    Description

      Relatively shortly after introducing NamespaceResolver, Dmitri had changed it to wipe out the reverse mapping cache every time a new mapping was added. The tests, however, relied upon the last added mapping to a given namespace being the preferred prefix. The description for his change was that he was fixing a "bug", but since he didn't elaborate on what kind of bug I can't see how maintaining the cache is any more harmful than not maintaining it, and reverting this change clears up the test failures that for some reason are only seen on JDK 1.3.

      Attachments

        Activity

          People

            Unassigned Unassigned
            mbenson Matthew Jason Benson
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: