Fixes for search of members + Removed fixme in member that apparently are not necessary after tags refactoring

src/Umbraco.Core/Cache/HttpRequestAppCache.cs

@@ -31,7 +31,7 @@ namespace Umbraco.Core.Cache
         /// </summary>
         /// <remarks>
         /// <para>Will use HttpContext.Current.</para>
-        /// fixme/task: use IHttpContextAccessor NOT HttpContext.Current
+        /// TODO - https://github.com/umbraco/Umbraco-CMS/issues/4239 - use IHttpContextAccessor NOT HttpContext.Current
         /// </remarks>
         public HttpRequestAppCache()
         { }

src/Umbraco.Core/Configuration/UmbracoVersion.cs

@@ -81,7 +81,7 @@ namespace Umbraco.Core.Configuration
             {
                 try
                 {
-                    // fixme/task - stop having version in web.config appSettings
+                    // TODO - https://github.com/umbraco/Umbraco-CMS/issues/4238 - stop having version in web.config appSettings
                     var value = ConfigurationManager.AppSettings["umbracoConfigurationStatus"];
                     return value.IsNullOrWhiteSpace() ? null : SemVersion.TryParse(value, out var semver) ? semver : null;
                 }

src/Umbraco.Core/Persistence/Repositories/Implement/DataTypeRepository.cs

@@ -26,7 +26,7 @@ namespace Umbraco.Core.Persistence.Repositories.Implement
     {
         private readonly Lazy<PropertyEditorCollection> _editors;
 
-        // fixme/task - get rid of Lazy injection and fix circular dependencies
+        // TODO - https://github.com/umbraco/Umbraco-CMS/issues/4237 - get rid of Lazy injection and fix circular dependencies
         public DataTypeRepository(IScopeAccessor scopeAccessor, AppCaches cache, Lazy<PropertyEditorCollection> editors, ILogger logger)
             : base(scopeAccessor, cache, logger)
         {

src/Umbraco.Core/Persistence/Repositories/Implement/DocumentRepository.cs

@@ -260,7 +260,7 @@ namespace Umbraco.Core.Persistence.Repositories.Implement
 
         protected override void PersistNewItem(IContent entity)
         {
-            // fixme/task - sort out IContent vs Content
+            // TODO: https://github.com/umbraco/Umbraco-CMS/issues/4234 - sort out IContent vs Content
             // however, it's not just so we have access to AddingEntity
             // there are tons of things at the end of the methods, that can only work with a true Content
             // and basically, the repository requires a Content, not an IContent

src/Umbraco.Web/PublishedContentExtensions.cs

@@ -954,7 +954,7 @@ namespace Umbraco.Web
         /// </remarks>
         public static IEnumerable<IPublishedContent> Children(this IPublishedContent content, string culture = null)
         {
-            if (content == null) throw new ArgumentNullException(nameof(content)); // fixme/task wtf is this?
+            if (content == null) throw new ArgumentNullException(nameof(content));
 
             return content.Children.WhereIsInvariantOrHasCulture(culture);
         }