Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG]Cannot break a Flowchart in While #5823

Open
iio888 opened this issue Jul 24, 2024 · 1 comment · May be fixed by #5899
Open

[BUG]Cannot break a Flowchart in While #5823

iio888 opened this issue Jul 24, 2024 · 1 comment · May be fixed by #5899
Labels
bug Something isn't working

Comments

@iio888
Copy link

iio888 commented Jul 24, 2024

Description

When a While’s Body is a flowchart , and there is a Break as a Then activity in a If, I found that I cannot break the While.

While w = new() {
    Body = new Flowchart() {
        Activities = {
         new WriteLine(“Do1”),
            new If(e => true) {
            Then = new Break(),
            },
        new WriteLine(“Do2”),
        }
       // Connected like Sequence
    }
};

Also, without the while , I cannot terminate the flowchart when the Then activity is End.

Flowchart chart = new() {
    Activities = {
        new WriteLine(“Do1”),
        new If(e => true) {
            Then = new End(),
        },
        new WriteLine(“Do2”)
    }
    // Connected like Sequence
}

Expected Behavior

  • For first example, it should output “Do1” as expected.
  • For second example, it should output “Do1” as expected.

Actual Behavior

  • For first example, it outputted ”Do1” “Do2” in actual.
  • For second example, it outputted “Do1” “Do2” in actual.

Environment

  • Elsa Package Version: 3.1.3.
  • Operating System: Windows 10
  • Browser and Version: No browser.

Troubleshooting Attempts

It seems Flowchart doesn’t receive BreakSignal and judge whether IsBreak in OnChildCompletedAsync method of Flowchart.
I have tried to substitute Flowchart with Sequence and it goes well.

@iio888 iio888 added the bug Something isn't working label Jul 24, 2024
@iio888
Copy link
Author

iio888 commented Aug 1, 2024

Is there any quick fix to resolve it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant