Merge pull request #31 from nvisage-gf/develop

Workaround for null issues when indexing
skttl · Oct 24, 2015 · c45989e · c45989e
2 parents 5aacebb + 9e4b481
commit c45989e
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/Src/Our.Umbraco.DocTypeGridEditor/Helpers/DocTypeGridEditorHelper.cs b/Src/Our.Umbraco.DocTypeGridEditor/Helpers/DocTypeGridEditorHelper.cs
@@ -27,9 +27,19 @@ public static IPublishedContent ConvertValueToContent(string id, string docTypeA
             if (string.IsNullOrWhiteSpace(docTypeAlias))
                 return null;
 
+           if (UmbracoContext.Current != null)
+           {
             return (IPublishedContent)ApplicationContext.Current.ApplicationCache.RequestCache.GetCacheItem(
                 "DocTypeGridEditorHelper.ConvertValueToContent_" + id + "_" + docTypeAlias, () =>
                 {
+                   return ConvertValue(id, docTypeAlias, dataJson);
+                 });
+            }
+                return (IPublishedContent) ConvertValue(id, docTypeAlias, dataJson);
+        }
+
+        private static IPublishedContent ConvertValue(string id, string docTypeAlias, string dataJson)
+        {
                     using (var timer =  DisposableTimer.DebugDuration<DocTypeGridEditorHelper>(string.Format("ConvertValueToContent ({0}, {1})", id, docTypeAlias)))
                     {
                         Guid docTypeGuid;
@@ -83,15 +93,15 @@ public static IPublishedContent ConvertValueToContent(string id, string docTypeA
                         }
 
                         // Get the current request node we are embedded in
-                        var pcr = UmbracoContext.Current.PublishedContentRequest;
+                        var pcr = UmbracoContext.Current == null ? null : UmbracoContext.Current.PublishedContentRequest;
                         var containerNode = pcr != null && pcr.HasPublishedContent ? pcr.PublishedContent : null;
 
                         return new DetachedPublishedContent(nameObj == null ? null : nameObj.ToString(), 
                             publishedContentType,
                             properties.ToArray(),
                             containerNode);
                     }
-                });
+
         }
     }
-}
+}