From af0a03a3e084547391fabe7f8ff10905b45798ec Mon Sep 17 00:00:00 2001 From: Stephan Date: Thu, 15 Mar 2018 09:51:12 +0100 Subject: [PATCH] U4-10756 - cleanup --- .../PublishedCache/XmlPublishedCache/PublishedContentCache.cs | 2 -- .../PublishedCache/XmlPublishedCache/PublishedMediaCache.cs | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs b/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs index d69b7104f6..c073963754 100644 --- a/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs +++ b/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedContentCache.cs @@ -351,8 +351,6 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache // populate the idkmap by indexing the content cache // assuming that the content cache cannot be corrupted - // PROBLEM: still, the idk map on-demand will do a DB lookup, is it bad? - // OR should we register PROVIDERS for the map FFS what a mess private void EnsureIdkMap(UmbracoContext umbracoContext) { lock (_idkMapLocker) diff --git a/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedMediaCache.cs b/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedMediaCache.cs index 978ca6b454..3bc0ecb7cd 100644 --- a/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedMediaCache.cs +++ b/src/Umbraco.Web/PublishedCache/XmlPublishedCache/PublishedMediaCache.cs @@ -74,7 +74,7 @@ namespace Umbraco.Web.PublishedCache.XmlPublishedCache public virtual IPublishedContent GetById(UmbracoContext umbracoContext, bool preview, Guid nodeKey) { - // fixme optimize with Lucene and stuff? + // TODO optimize with Examine? var mapAttempt = ApplicationContext.Current.Services.IdkMap.GetIdForKey(nodeKey, UmbracoObjectTypes.Media); return mapAttempt ? GetById(umbracoContext, preview, mapAttempt.Result) : null; }