Avoid changing root-ness of assets

cognitedata · Sep 9, 2024 · f9821b6 · f9821b6
1 parent 11df628
commit f9821b6
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/Extractor/Pushers/PusherUtils.cs b/Extractor/Pushers/PusherUtils.cs
@@ -181,11 +181,11 @@ private static bool ShouldSetNewMetadata(FullConfig config, Dictionary<string, s
             if (update.Context && !newAsset.ParentId.IsNullNodeId)
             {
                 var parentId = extractor.GetUniqueId(newAsset.ParentId);
-                if (parentId != old.ParentExternalId)
+                // Do not move an asset from root to non-root or the other way around.
+                if (parentId is null == old.ParentExternalId is null
+                    && parentId != old.ParentExternalId)
                 {
-#pragma warning disable CS8604 // Possible null reference argument.
-                    assetUpdate.ParentExternalId = new UpdateNullable<string>(parentId);
-#pragma warning restore CS8604 // Possible null reference argument.
+                    assetUpdate.ParentExternalId = new UpdateNullable<string?>(parentId);
                 }
             }