From 8592b7039521520cc3bf9eb1a43b60f35170bca6 Mon Sep 17 00:00:00 2001 From: enisn Date: Fri, 5 Mar 2021 17:35:43 +0300 Subject: [PATCH] CmsKit - Remove unnecessary Blog Existence check in BlogPostManager --- .../Volo/CmsKit/Blogs/BlogPostManager.cs | 8 -------- 1 file changed, 8 deletions(-) diff --git a/modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Blogs/BlogPostManager.cs b/modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Blogs/BlogPostManager.cs index 9a4f627da7..05241bab48 100644 --- a/modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Blogs/BlogPostManager.cs +++ b/modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Blogs/BlogPostManager.cs @@ -34,14 +34,6 @@ namespace Volo.CmsKit.Blogs Check.NotNullOrEmpty(title, nameof(title)); Check.NotNullOrEmpty(slug, nameof(slug)); - /* TODO: BlogRepository.GetAsync already throws the same exception. - So, if you get Blog you don't have to check its existence. - Actually, you SHOULD not check. What if I've created the Blog, but - not saved to database, and I am creating a post inside it in same UOW. - In this case, you throw an unnecessary exception. - */ - await CheckBlogExistenceAsync(blog.Id); - var blogPost = new BlogPost( GuidGenerator.Create(), blog.Id,