From 83744f1f17e08ce928524fc9cfe4c6cafe04dfee Mon Sep 17 00:00:00 2001 From: Sipke Schoorstra Date: Sat, 22 Feb 2025 20:50:36 +0100 Subject: [PATCH] Fix workflow execution context flag after activity execution Previously, the `WorkflowExecutionContext.IsExecuting` flag was incorrectly set to `true` after an activity executed, causing potential logic inconsistencies. This change ensures the flag is correctly set to `false`, maintaining the expected workflow state. --- .../Middleware/Activities/DefaultActivityInvokerMiddleware.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/modules/Elsa.Workflows.Core/Middleware/Activities/DefaultActivityInvokerMiddleware.cs b/src/modules/Elsa.Workflows.Core/Middleware/Activities/DefaultActivityInvokerMiddleware.cs index 71dedf79a..0912b677e 100644 --- a/src/modules/Elsa.Workflows.Core/Middleware/Activities/DefaultActivityInvokerMiddleware.cs +++ b/src/modules/Elsa.Workflows.Core/Middleware/Activities/DefaultActivityInvokerMiddleware.cs @@ -90,7 +90,7 @@ public class DefaultActivityInvokerMiddleware(ActivityMiddlewareDelegate next, I // Mark activity as executed. context.IsExecuting = false; - context.WorkflowExecutionContext.IsExecuting = true; + context.WorkflowExecutionContext.IsExecuting = false; // Conditionally commit the workflow state. if (ShouldCommit(context, ActivityLifetimeEvent.ActivityExecuted))