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

Regression: "fix" made to NamespaceResolver broke existing testcases

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.2 Final
    • Fix Version/s: 1.3
    • Labels:
      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

            • Assignee:
              Unassigned
              Reporter:
              mbenson Matt Benson
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: