From c9c869356001e55dcd30a3c3f7076ba0d01bb3b4 Mon Sep 17 00:00:00 2001
From: Shannon <sdeminick@gmail.com>
Date: Wed, 1 Apr 2020 13:13:15 +1100
Subject: [PATCH] removes one null check, adds notes

---
 src/Umbraco.Web/PublishedCache/NuCache/ContentStore.cs    | 8 ++++++++
 src/Umbraco.Web/PublishedCache/NuCache/Snap/LinkedNode.cs | 6 ++----
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/Umbraco.Web/PublishedCache/NuCache/ContentStore.cs b/src/Umbraco.Web/PublishedCache/NuCache/ContentStore.cs
index e5b66e259c..9298201ecf 100644
--- a/src/Umbraco.Web/PublishedCache/NuCache/ContentStore.cs
+++ b/src/Umbraco.Web/PublishedCache/NuCache/ContentStore.cs
@@ -502,6 +502,14 @@ namespace Umbraco.Web.PublishedCache.NuCache
             }
         }
 
+        /// <summary>
+        /// Validate the <see cref="ContentNodeKit"/> and try to create a parent <see cref="LinkedNode{ContentNode}"/>
+        /// </summary>
+        /// <param name="kit"></param>
+        /// <param name="parent"></param>
+        /// <returns>
+        /// Returns false if the parent was not found or if the kit validation failed
+        /// </returns>
         private bool BuildKit(ContentNodeKit kit, out LinkedNode<ContentNode> parent)
         {
             // make sure parent exists
diff --git a/src/Umbraco.Web/PublishedCache/NuCache/Snap/LinkedNode.cs b/src/Umbraco.Web/PublishedCache/NuCache/Snap/LinkedNode.cs
index 8794978852..94f83ac4e5 100644
--- a/src/Umbraco.Web/PublishedCache/NuCache/Snap/LinkedNode.cs
+++ b/src/Umbraco.Web/PublishedCache/NuCache/Snap/LinkedNode.cs
@@ -1,6 +1,4 @@
-using System;
-
-namespace Umbraco.Web.PublishedCache.NuCache.Snap
+namespace Umbraco.Web.PublishedCache.NuCache.Snap
 {
     //NOTE: This cannot be struct because it references itself
 
@@ -13,7 +11,7 @@ namespace Umbraco.Web.PublishedCache.NuCache.Snap
     {
         public LinkedNode(TValue value, long gen, LinkedNode<TValue> next = null)
         {
-            Value = value ?? throw new ArgumentNullException(nameof(value));
+            Value = value; // This is allowed to be null, we actually explicitly set this to null in ClearLocked
             Gen = gen;
             Next = next;
         }