From 511751d1473a85fc19e01e958e66abbb2ea42e3c Mon Sep 17 00:00:00 2001 From: arkadiuszbiel Date: Thu, 10 Oct 2019 15:50:00 +0200 Subject: [PATCH] refactor based on shannon comments --- src/Umbraco.Examine/IUmbracoIndexConfig.cs | 8 ++++---- src/Umbraco.Examine/Umbraco.Examine.csproj | 2 +- ...racoUmbracoIndexConfig.cs => UmbracoIndexConfig.cs} | 10 +++++----- src/Umbraco.Web/Search/UmbracoIndexesCreator.cs | 6 +++--- 4 files changed, 13 insertions(+), 13 deletions(-) rename src/Umbraco.Examine/{UmbracoUmbracoIndexConfig.cs => UmbracoIndexConfig.cs} (65%) diff --git a/src/Umbraco.Examine/IUmbracoIndexConfig.cs b/src/Umbraco.Examine/IUmbracoIndexConfig.cs index 798c173f5c..02c6c51d0c 100644 --- a/src/Umbraco.Examine/IUmbracoIndexConfig.cs +++ b/src/Umbraco.Examine/IUmbracoIndexConfig.cs @@ -2,11 +2,11 @@ using Examine; namespace Umbraco.Examine { - public abstract class IUmbracoIndexConfig + public interface IUmbracoIndexConfig { - public abstract IContentValueSetValidator GetContentValueSetValidator(); - public abstract IContentValueSetValidator GetPublishedContentValueSetValidator(); - public abstract IValueSetValidator GetMemberValueSetValidator(); + IContentValueSetValidator GetContentValueSetValidator(); + IContentValueSetValidator GetPublishedContentValueSetValidator(); + IValueSetValidator GetMemberValueSetValidator(); } } diff --git a/src/Umbraco.Examine/Umbraco.Examine.csproj b/src/Umbraco.Examine/Umbraco.Examine.csproj index 2bfa810b45..e30d355dfe 100644 --- a/src/Umbraco.Examine/Umbraco.Examine.csproj +++ b/src/Umbraco.Examine/Umbraco.Examine.csproj @@ -68,7 +68,7 @@ - + diff --git a/src/Umbraco.Examine/UmbracoUmbracoIndexConfig.cs b/src/Umbraco.Examine/UmbracoIndexConfig.cs similarity index 65% rename from src/Umbraco.Examine/UmbracoUmbracoIndexConfig.cs rename to src/Umbraco.Examine/UmbracoIndexConfig.cs index 0a9b8607e8..7ad9c638d3 100644 --- a/src/Umbraco.Examine/UmbracoUmbracoIndexConfig.cs +++ b/src/Umbraco.Examine/UmbracoIndexConfig.cs @@ -4,20 +4,20 @@ using Umbraco.Core.Services.Implement; namespace Umbraco.Examine { - public class UmbracoUmbracoIndexConfig : IUmbracoIndexConfig + public class UmbracoIndexConfig : IUmbracoIndexConfig { - public UmbracoUmbracoIndexConfig(IPublicAccessService publicAccessService) + public UmbracoIndexConfig(IPublicAccessService publicAccessService) { PublicAccessService = publicAccessService; } protected IPublicAccessService PublicAccessService { get; } - public override IContentValueSetValidator GetContentValueSetValidator() + public IContentValueSetValidator GetContentValueSetValidator() { return new ContentValueSetValidator(false, true, PublicAccessService); } - public override IContentValueSetValidator GetPublishedContentValueSetValidator() + public IContentValueSetValidator GetPublishedContentValueSetValidator() { return new ContentValueSetValidator(true, false, PublicAccessService); } @@ -26,7 +26,7 @@ namespace Umbraco.Examine /// Returns the for the member indexer /// /// - public override IValueSetValidator GetMemberValueSetValidator() + public IValueSetValidator GetMemberValueSetValidator() { return new MemberValueSetValidator(); } diff --git a/src/Umbraco.Web/Search/UmbracoIndexesCreator.cs b/src/Umbraco.Web/Search/UmbracoIndexesCreator.cs index 73b7343b16..ec536b9d75 100644 --- a/src/Umbraco.Web/Search/UmbracoIndexesCreator.cs +++ b/src/Umbraco.Web/Search/UmbracoIndexesCreator.cs @@ -88,12 +88,12 @@ namespace Umbraco.Web.Search ); return index; } - [Obsolete] + [Obsolete("This method should not be used and will be removed in future versions. GetContentValueSetValidator was moved to IUmbracoIndexConfig")] public virtual IContentValueSetValidator GetContentValueSetValidator() { return UmbracoIndexConfig.GetContentValueSetValidator(); } - [Obsolete] + [Obsolete("This method should not be used and will be removed in future versions. GetPublishedContentValueSetValidator was moved to IUmbracoIndexConfig")] public virtual IContentValueSetValidator GetPublishedContentValueSetValidator() { return UmbracoIndexConfig.GetPublishedContentValueSetValidator(); @@ -103,7 +103,7 @@ namespace Umbraco.Web.Search /// Returns the for the member indexer /// /// - [Obsolete] + [Obsolete("This method should not be used and will be removed in future versions. GetMemberValueSetValidator was moved to IUmbracoIndexConfig")] public virtual IValueSetValidator GetMemberValueSetValidator() { return UmbracoIndexConfig.GetMemberValueSetValidator();