SemianiakaVY/elsa-core
1
0
Fork 0
Code Issues Pull requests Actions 1 Packages Projects Releases Wiki Activity

Fix double scheduling of activities

Browse source
This commit is contained in:
Sipke Schoorstra 2021-06-30 12:27:30 +02:00
parent 5b39515bce
commit 189883ee7f
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/core/Elsa.Abstractions/ActivityResults/OutcomeResult.cs
View file

@ -30,8 +30,10 @@ namespace Elsa.ActivityResults
var nextConnections = GetNextConnections(workflowExecutionContext, activityExecutionContext.ActivityBlueprint.Id, outcomes).ToList();
// See if we got a "default" connection (from the current activity to the next activity via the default "Done" outcome).
if (!outcomes.Contains(OutcomeNames.Done) && !nextConnections.Any())
nextConnections = GetNextConnections(workflowExecutionContext, activityExecutionContext.ActivityBlueprint.Id, new[] { OutcomeNames.Done }).ToList();
// Unless the current activity is a "scope" activity, in which case it will schedule a Done outcome itself.
// TODO: This is another sign that we need to refactor activity scopes into a natural contained activity structure.
if (!outcomes.Contains(OutcomeNames.Done) && !nextConnections.Any() && activityExecutionContext.ActivityId != activityExecutionContext.CurrentScope?.ActivityId)
nextConnections = GetNextConnections(workflowExecutionContext, activityExecutionContext.ActivityId, new[] { OutcomeNames.Done }).ToList();
var nextActivities =
(