Tidy: GuidUdi lookups.

KevinJump · Dec 19, 2019 · 951606f · 951606f
1 parent 0b20064
commit 951606f
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 26 deletions.
diff --git a/uSync8.ContentEdition/Mapping/Mappers/MultiUrlMapper.cs b/uSync8.ContentEdition/Mapping/Mappers/MultiUrlMapper.cs
@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Linq;
 using System.Runtime.Serialization;
 
 using Newtonsoft.Json;
@@ -26,17 +27,10 @@ public override IEnumerable<uSyncDependency> GetDependencies(object value, strin
         {
             var links = JsonConvert.DeserializeObject<IEnumerable<LinkDto>>(value.ToString());
 
-            var dependencies = new List<uSyncDependency>();
+            if (links == null || !links.Any()) return Enumerable.Empty<uSyncDependency>();
 
-            foreach (var link in links)
-            {
-                if (link.Udi != null)
-                {
-                    dependencies.Add(CreateDependency(link.Udi, flags));
-                }
-            }
-
-            return dependencies;
+            return links.Where(x => x.Udi != null)
+                .Select(link => CreateDependency(link.Udi, flags));
         }
 
         // taken from umbraco source - this is how it's stored 

diff --git a/uSync8.ContentEdition/Mapping/Mappers/RTEMapper.cs b/uSync8.ContentEdition/Mapping/Mappers/RTEMapper.cs
@@ -50,7 +50,7 @@ public override IEnumerable<uSyncDependency> GetDependencies(object value, strin
 
             foreach (Match m in UdiRegEx.Matches(stringValue))
             {
-                if (Udi.TryParse(m.Value, out Udi udi))
+                if (GuidUdi.TryParse(m.Value, out GuidUdi udi))
                 {
                     if (!dependencies.Any(x => x.Udi == udi))
                         dependencies.Add(CreateDependency(udi, flags));

diff --git a/uSync8.ContentEdition/Mapping/SyncNestedValueMapperBase.cs b/uSync8.ContentEdition/Mapping/SyncNestedValueMapperBase.cs
@@ -77,7 +77,6 @@ protected IEnumerable<uSyncDependency> GetPropertyDependencies(
             }
 
             return dependencies;
-
         }
 
 

diff --git a/uSync8.ContentEdition/Mapping/SyncValueMapperBase.cs b/uSync8.ContentEdition/Mapping/SyncValueMapperBase.cs
@@ -39,31 +39,26 @@ public virtual string GetImportValue(string value, string editorAlias)
 
         protected IEnumerable<uSyncDependency> CreateDependencies(IEnumerable<string> udiStrings, DependencyFlags flags)
         {
-            if (udiStrings == null || !udiStrings.Any()) return Enumerable.Empty<uSyncDependency>();
-
-            var dependencies = new List<uSyncDependency>();
+            if (udiStrings == null || !udiStrings.Any()) yield break;
 
             foreach (var udiString in udiStrings)
             {
                 var dependency = CreateDependency(udiString, flags);
-                if (dependency != null)
-                    dependencies.Add(dependency);
+                if (dependency != null) yield return dependency;
             }
-
-            return dependencies;
         }
 
         protected uSyncDependency CreateDependency(string udiString, DependencyFlags flags)
         {
-            if (Udi.TryParse(udiString, out Udi udi))
+            if (GuidUdi.TryParse(udiString, out GuidUdi udi))
             {
                 return CreateDependency(udi, flags);
             }
 
             return null;
         }
 
-        protected uSyncDependency CreateDependency(Udi udi, DependencyFlags flags)
+        protected uSyncDependency CreateDependency(GuidUdi udi, DependencyFlags flags)
         {
             var entity = GetElement(udi);
 
@@ -77,12 +72,11 @@ protected uSyncDependency CreateDependency(Udi udi, DependencyFlags flags)
             };
         }
 
-        private IEntitySlim GetElement(Udi udi)
+        private IEntitySlim GetElement(GuidUdi udi)
         {
-            if (udi is GuidUdi guidUdi)
-            {
-                return entityService.Get(guidUdi.Guid);
-            }
+            if (udi != null)
+                return entityService.Get(udi.Guid);
+
             return null;
         }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -77,7 +77,6 @@ protected IEnumerable<uSyncDependency> GetPropertyDependencies(
		}

		return dependencies;

		}


Expand Down