Uploaded image for project: 'Commons SCXML'
  1. Commons SCXML
  2. SCXML-58

SCXML not stateless because of State class

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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
        mtillberg Mike Tillberg added a comment -

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

        Show
        mtillberg Mike Tillberg added a comment - Patch to StatelessModelText and accompanying state model to test this problem.
        Hide
        rahul@apache.org 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@apache.org 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@apache.org Rahul Akolkar added a comment -

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

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development