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

OMChildrenQNameIterator doesn't work correctly if hasNext() is not called before next()

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.2.8
    • Fix Version/s: 1.2.11
    • Component/s: DOOM, LLOM
    • Labels:
      None

      Description

      OMChildrenQNameIterator only works correctly if hasNext() is called before each invocation of next(). While this is usually the case, it is not at all required!

        Activity

        Hide
        hudson Hudson added a comment -

        Integrated in ws-axiom-trunk #667 (See https://builds.apache.org/job/ws-axiom-trunk/667/)

        • Removed the old test cases for AXIOM-78 which failed to provide evidence for the issue.
        • Added a regression test for AXIOM-78 and AXIOM-172.

        veithen :
        Files :

        • /webservices/commons/trunk/modules/axiom/modules/axiom-tests/src/test/java/org/apache/axiom/om/IteratorTest.java
        • /webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/OMTestSuiteBuilder.java
        • /webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/element/TestGetChildrenWithNameNextWithoutHasNext.java
        Show
        hudson Hudson added a comment - Integrated in ws-axiom-trunk #667 (See https://builds.apache.org/job/ws-axiom-trunk/667/ ) Removed the old test cases for AXIOM-78 which failed to provide evidence for the issue. Added a regression test for AXIOM-78 and AXIOM-172 . veithen : Files : /webservices/commons/trunk/modules/axiom/modules/axiom-tests/src/test/java/org/apache/axiom/om/IteratorTest.java /webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/OMTestSuiteBuilder.java /webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/element/TestGetChildrenWithNameNextWithoutHasNext.java
        Hide
        veithen Andreas Veithen added a comment -

        This should be fixed by r1054444.

        Show
        veithen Andreas Veithen added a comment - This should be fixed by r1054444.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development