Mail service to work with newsletters

Author: rxtur

src/Core/Api/PostsController.cs

@@ -8,6 +8,7 @@
 using Microsoft.AspNetCore.Mvc;
 using System;
 using System.Collections.Generic;
+using System.Linq;
 using System.Threading.Tasks;
 
 namespace Core.Api
@@ -17,10 +18,12 @@ namespace Core.Api
     public class PostsController : ControllerBase
     {
         IDataService _data;
+        IEmailService _email;
 
-        public PostsController(IDataService data)
+        public PostsController(IDataService data, IEmailService email)
         {
             _data = data;
+            _email = email;
         }
 
         /// <summary>
@@ -202,6 +205,10 @@ public async Task<ActionResult> Publish(int id, string flag)
                 }
                 await Task.CompletedTask;
 
+                if(flag == "P")
+                {
+                    await SendNewsletters(post);
+                }
                 return Ok(Resources.Updated);
             }
             catch (Exception ex)
@@ -252,8 +259,21 @@ public async Task<ActionResult<PostItem>> Post(PostItem post)
         {
             try
             {
+                bool alreadyPublished = false;
+                if (post.Id > 0)
+                {
+                    var existing = _data.BlogPosts.Single(p => p.Id == post.Id);
+                    alreadyPublished = existing.Published > DateTime.MinValue;
+                }
                 post.Slug = await GetSlug(post.Id, post.Title);
                 var saved = await _data.BlogPosts.SaveItem(post);
+
+                if(post.IsPublished && !alreadyPublished)
+                {
+                    var savedPost = _data.BlogPosts.Single(p => p.Id == saved.Id);
+                    await SendNewsletters(savedPost);
+                }
+
                 return Created($"admin/posts/edit?id={saved.Id}", saved);
             }
             catch (Exception ex)
@@ -321,6 +341,20 @@ async Task<string> GetSlug(int id, string title)
             return await Task.FromResult(slug);
         }
 
+        async Task SendNewsletters(BlogPost post)
+        {
+            var pager = new Pager(1, 1000);
+            IEnumerable<Newsletter> newsletters;
+            newsletters = await _data.Newsletters.GetList(e => e.Id > 0, pager);
+
+            var emails = newsletters.Select(i => i.Email).ToList();
+            if (emails != null && emails.Count > 0)
+            {
+                var siteUrl = $"{Request.Scheme}://{Request.Host}{Request.PathBase}";
+                await _email.SendNewsletters(post, emails, siteUrl);
+            }
+        }
+
         int GetUserId(string author)
         {
             int id = 0;

src/Core/Core.csproj

@@ -2,7 +2,7 @@
 
   <PropertyGroup>
     <TargetFramework>netcoreapp2.2</TargetFramework>
-    <Version>2.4.0.7</Version>
+    <Version>2.4.0.8</Version>
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">

src/Core/Services/SendGridService.cs

@@ -10,7 +10,7 @@ namespace Core.Services
 {
     public interface IEmailService
     {
-        Task SendNewsletters(PostItem postItem, List<string> emails, string siteUrl);
+        Task SendNewsletters(BlogPost postItem, List<string> emails, string siteUrl);
         Task SendEmail(string to, string subject, string content);
     }
 
@@ -29,9 +29,10 @@ public SendGridService(IDataService db, IConfiguration config, ILogger<SendGridS
             _storage = storage;
         }
 
-        public async Task SendNewsletters(PostItem post, List<string> emails, string siteUrl)
+        public async Task SendNewsletters(BlogPost post, List<string> emails, string siteUrl)
         {
             var blog = await _db.CustomFields.GetBlogSettings();
+            var author = _db.Authors.Single(a => a.Id == post.AuthorId);
             foreach (var email in emails)
             {
                 var subject = post.Title;
@@ -47,7 +48,7 @@ public async Task SendNewsletters(PostItem post, List<string> emails, string sit
                     post.Slug, // 6
                     post.Published, // 7 
                     post.Cover, // 8
-                    post.Author, // 9
+                    author.DisplayName, // 9
                     siteUrl); // 10
 
                 await SendEmail(email, subject, htmlContent);
@@ -62,7 +63,7 @@ public async Task SendEmail(string to, string subject, string content)
             {
                 var apiKey = section.GetValue<string>("SendGridApiKey");
 
-                if (!string.IsNullOrEmpty(apiKey))
+                if (!string.IsNullOrEmpty(apiKey) && apiKey != "YOUR-SENDGRID-API-KEY")
                 {
                     try
                     {