Fix coderabbit warnings

Altinn · Jan 16, 2025 · 7ca7686 · 7ca7686
1 parent 9c6dbb9
commit 7ca7686
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/backend/src/Designer/EventHandlers/ComponentIdChanged/ComponentIdChangedLayoutsHandler.cs b/backend/src/Designer/EventHandlers/ComponentIdChanged/ComponentIdChangedLayoutsHandler.cs
@@ -15,8 +15,7 @@ public class ComponentIdChangedLayoutsHandler : INotificationHandler<ComponentId
     private readonly IFileSyncHandlerExecutor _fileSyncHandlerExecutor;
 
     public ComponentIdChangedLayoutsHandler(IAltinnGitRepositoryFactory altinnGitRepositoryFactory,
-        IFileSyncHandlerExecutor fileSyncHandlerExecutor,
-        IAppDevelopmentService appDevelopmentService)
+        IFileSyncHandlerExecutor fileSyncHandlerExecutor)
     {
         _altinnGitRepositoryFactory = altinnGitRepositoryFactory;
         _fileSyncHandlerExecutor = fileSyncHandlerExecutor;

diff --git a/backend/src/Designer/Services/Implementation/AppDevelopmentService.cs b/backend/src/Designer/Services/Implementation/AppDevelopmentService.cs
@@ -649,6 +649,11 @@ private async Task<bool> UpdateLayoutReferences(AltinnAppGitRepository altinnApp
                             }
 
                             string componentId = component["id"]?.GetValue<string>();
+                            if (string.IsNullOrEmpty(componentId))
+                            {
+                                continue;
+                            }
+
                             bool isComponentDeleted = deletedComponentIdsFromCurrentLayoutSet.Contains(componentId);
 
                             if (isComponentDeleted)