The link can be null / undefined, which is fine and no reason to to blow up

nul800sebastiaan · nul800sebastiaan · commit 14ebe71cfb82 · 2017-09-29T15:35:52.000+02:00
diff --git a/src/Umbraco.Deploy.Contrib.Connectors/ValueConnectors/RelatedLinks2ValueConnector.cs b/src/Umbraco.Deploy.Contrib.Connectors/ValueConnectors/RelatedLinks2ValueConnector.cs
@@ -50,7 +50,8 @@ public string GetValue(Property property, ICollection<ArtifactDependency> depend
             foreach (var internalLink in relatedLinks.Where(x => x.IsInternal))
             {
                 //As the page was picked we need to add it to collection of depenencies for this deployment
-                dependencies.Add(new ArtifactDependency(internalLink.Link, false, ArtifactDependencyMode.Exist));
+                if (internalLink.Link != null)
+                    dependencies.Add(new ArtifactDependency(internalLink.Link, false, ArtifactDependencyMode.Exist));
             }
 
             //As this property editor already stores UDIs and not ints

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,8 @@ public string GetValue(Property property, ICollection<ArtifactDependency> depend`
`50`	`50`	`foreach (var internalLink in relatedLinks.Where(x => x.IsInternal))`
`51`	`51`	`{`
`52`	`52`	`//As the page was picked we need to add it to collection of depenencies for this deployment`
`53`		`- dependencies.Add(new ArtifactDependency(internalLink.Link, false, ArtifactDependencyMode.Exist));`
	`53`	`+ if (internalLink.Link != null)`
	`54`	`+ dependencies.Add(new ArtifactDependency(internalLink.Link, false, ArtifactDependencyMode.Exist));`
`54`	`55`	`}`
`55`	`56`
`56`	`57`	`//As this property editor already stores UDIs and not ints`