Moving the call to the PartNodeResolver into the WorkInstructionUpdater when updating work instructions for code clarity.

Author: EyeBrawler

MESS/MESS.Services/CRUD/WorkInstructions/WorkInstructionService.cs

@@ -826,8 +826,6 @@ public async Task<bool> UpdateWorkInstructionAsync(WorkInstructionFormDTO dto)
                     return false;
 
                 await _workInstructionUpdater.ApplyAsync(dto, existing, context);
-                
-                await _partNodeResolver.ResolvePendingNodesAsync(context, existing.Nodes);
 
                 await context.SaveChangesAsync();
                 await transaction.CommitAsync();

MESS/MESS.Services/CRUD/WorkInstructions/WorkInstructionUpdater.cs

@@ -24,17 +24,21 @@ namespace MESS.Services.CRUD.WorkInstructions;
 public class WorkInstructionUpdater : IWorkInstructionUpdater
 {
     private readonly IProductResolver _productResolver;
+    private readonly IPartNodeResolver _partNodeResolver;
     private readonly IPartDefinitionResolver _partDefinitionResolver;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="WorkInstructionUpdater"/> class.
     /// </summary>
     /// <param name="productResolver">The service used for resolving products from product names.</param>
     /// <param name="partDefinitionResolver">The service used for resolving the part definition that a work instruction produces.</param>
-    public WorkInstructionUpdater(IProductResolver productResolver, IPartDefinitionResolver partDefinitionResolver)
+    /// <param name="partNodeResolver">The service used for resolving part nodes in a work instruction.</param>
+    public WorkInstructionUpdater(IProductResolver productResolver, IPartDefinitionResolver partDefinitionResolver,
+        IPartNodeResolver partNodeResolver)
     {
         _productResolver = productResolver;
         _partDefinitionResolver = partDefinitionResolver;
+        _partNodeResolver = partNodeResolver;
     }
 
     /// <summary>
@@ -88,6 +92,8 @@ public async Task ApplyAsync(
         await SyncProductsAsync(dto, entity, context);
 
         SyncNodes(dto.Nodes, entity, context);
+        
+        await _partNodeResolver.ResolvePendingNodesAsync(context, entity.Nodes);
     }
 
     private void ApplyScalars(WorkInstructionFormDTO dto, WorkInstruction entity)