Axiom
  1. Axiom
  2. AXIOM-172

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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 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 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
        Andreas Veithen added a comment -

        This should be fixed by r1054444.

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development