From b57bf92e731107290278886380df18618fed2511 Mon Sep 17 00:00:00 2001 From: leekelleher Date: Wed, 8 Sep 2021 07:13:57 +0100 Subject: [PATCH] DataList - UmbracoContent, UmbracoContext refactor Small tweak to save calling `_umbracoContextAccessor.GetRequiredUmbracoContext()` twice. --- .../DataList/DataSources/UmbracoContentDataListSource.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Community.Contentment/DataEditors/DataList/DataSources/UmbracoContentDataListSource.cs b/src/Umbraco.Community.Contentment/DataEditors/DataList/DataSources/UmbracoContentDataListSource.cs index 3b2be36a..2e50bd7e 100644 --- a/src/Umbraco.Community.Contentment/DataEditors/DataList/DataSources/UmbracoContentDataListSource.cs +++ b/src/Umbraco.Community.Contentment/DataEditors/DataList/DataSources/UmbracoContentDataListSource.cs @@ -65,11 +65,11 @@ public IEnumerable GetItems(Dictionary config) var preview = true; var parentNode = config.GetValueAs("parentNode", string.Empty); var startNode = default(IPublishedContent); + var umbracoContext = _umbracoContextAccessor.GetRequiredUmbracoContext(); if (parentNode.InvariantStartsWith("umb://document/") == false) { var nodeContextId = default(int?); - var umbracoContext = _umbracoContextAccessor.GetRequiredUmbracoContext(); // NOTE: First we check for "id" (if on a content page), then "parentId" (if editing an element). if (int.TryParse(_requestAccessor.GetQueryStringValue("id"), out var currentId) == true) @@ -98,7 +98,7 @@ public IEnumerable GetItems(Dictionary config) } else if (UdiParser.TryParse(parentNode, out GuidUdi udi) == true && udi.Guid != Guid.Empty) { - startNode = _umbracoContextAccessor.GetRequiredUmbracoContext().Content.GetById(preview, udi.Guid); + startNode = umbracoContext.Content.GetById(preview, udi.Guid); } if (startNode != null)