Commons SCXML
  1. Commons SCXML
  2. SCXML-58

SCXML not stateless because of State class

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.6
    • Fix Version/s: 0.7
    • Labels:
      None
    • Environment:

      nightly build 20070730

      Description

      The State class is not stateless because of the "done" property. This can cause incorrect firing of the parallel state done transition when sharing an SCXML object between multiple SCXMLExecutors.

      1. statelessTest.patch
        4 kB
        Mike Tillberg
      2. stateless-parallel01.xml
        0.8 kB
        Mike Tillberg

        Activity

        Hide
        Mike Tillberg added a comment -

        Patch to StatelessModelText and accompanying state model to test this problem.

        Show
        Mike Tillberg added a comment - Patch to StatelessModelText and accompanying state model to test this problem.
        Hide
        Rahul Akolkar added a comment -

        Indeed, and thanks for the test case (haven't run it yet, but the issue seems obvious). I intend to look at this in a week's time. If you want to take a stab at it, probably the best solution would be to hand off this book-keeping to the o.a.c.scxml.SCInstance class which manages the state for every executor instance.

        Setting fix version to next release (v0.7).

        Please note that the "Affects Version/s" field should generally be set to the latest release (in this case, v0.6 – I've changed that here).

        Show
        Rahul Akolkar added a comment - Indeed, and thanks for the test case (haven't run it yet, but the issue seems obvious). I intend to look at this in a week's time. If you want to take a stab at it, probably the best solution would be to hand off this book-keeping to the o.a.c.scxml.SCInstance class which manages the state for every executor instance. Setting fix version to next release (v0.7). Please note that the "Affects Version/s" field should generally be set to the latest release (in this case, v0.6 – I've changed that here).
        Hide
        Rahul Akolkar added a comment -

        Fixed in SVN, thanks for the tests, they have been added to the test suite.

        Show
        Rahul Akolkar added a comment - Fixed in SVN, thanks for the tests, they have been added to the test suite.

          People

          • Assignee:
            Unassigned
            Reporter:
            Mike Tillberg
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development