Fixed upsert bug (closes #1)

Author: linkdotnet

LinkDotNet.Blog.IntegrationTests/Infrastructure/Persistence/Sql/SqlRepositoryTests.cs

@@ -1,6 +1,7 @@
 using System.Linq;
 using System.Threading.Tasks;
 using FluentAssertions;
+using LinkDotNet.Blog.TestUtilities;
 using LinkDotNet.Domain;
 using Microsoft.EntityFrameworkCore;
 using Xunit;
@@ -66,5 +67,21 @@ public async Task ShouldGetAllBlogPosts()
             var tagContent = blogPostFromRepo.Tags.Select(t => t.Content).ToList();
             tagContent.Should().Contain(new[] { "Tag 1", "Tag 2" });
         }
+
+        [Fact]
+        public async Task ShouldBeUpdateable()
+        {
+            var blogPost = new BlogPostBuilder().Build();
+            await DbContext.BlogPosts.AddAsync(blogPost);
+            await DbContext.SaveChangesAsync();
+            var blogPostFromDb = await BlogPostRepository.GetByIdAsync(blogPost.Id);
+            var updater = new BlogPostBuilder().WithTitle("New Title").Build();
+            blogPostFromDb.Update(updater);
+
+            await BlogPostRepository.StoreAsync(blogPostFromDb);
+
+            var blogPostAfterSave = await DbContext.BlogPosts.AsNoTracking().SingleAsync(b => b.Id == blogPostFromDb.Id);
+            blogPostAfterSave.Title.Should().Be("New Title");
+        }
     }
 }

LinkDotNet.Blog.Web/Pages/Admin/CreateNew.razor renamed to LinkDotNet.Blog.Web/Pages/Admin/CreateNewBlogPostPage.razor

@@ -0,0 +1,44 @@
+@page "/update/{blogPostId}"
+@attribute [Authorize]
+@using LinkDotNet.Infrastructure.Persistence
+@using LinkDotNet.Blog.Web.Shared.Admin
+@using LinkDotNet.Domain
+@inherits MarkdownComponentBase
+@inject IRepository _repository
+@inject IToastService _toastService
+
+@if (blogPostFromDb != null)
+{
+    <CreateNewBlogPost
+        Title="Update BlogPost"
+        BlogPost="@blogPostFromDb"
+        OnBlogPostCreated="StoreBlogPostAsync"></CreateNewBlogPost>
+}
+else
+{
+    <p>Loading...</p>
+}
+
+@code {
+    [Parameter]
+    public string BlogPostId { get; set; }
+
+    private BlogPost blogPostFromDb;
+    
+    protected override async Task OnParametersSetAsync()
+    {
+        if (string.IsNullOrEmpty(BlogPostId))
+        {
+            throw new ArgumentNullException(nameof(BlogPostId), "You have to provide an id");
+        }
+
+        blogPostFromDb = await _repository.GetByIdAsync(BlogPostId);
+    }
+
+    private async Task StoreBlogPostAsync(BlogPost blogPost)
+    {
+        blogPostFromDb.Update(blogPost);
+        await _repository.StoreAsync(blogPostFromDb);
+        _toastService.ShowInfo($"Updated BlogPost {blogPost.Title}");
+    }
+}

LinkDotNet.Blog.Web/Pages/Admin/UpdateBlogPostPage.razor

@@ -46,7 +46,15 @@ public async Task<IEnumerable<BlogPost>> GetAllAsync(Expression<Func<BlogPost, b
 
         public async Task StoreAsync(BlogPost blogPost)
         {
-            await blogPostContext.BlogPosts.AddAsync(blogPost);
+            if (string.IsNullOrEmpty(blogPost.Id))
+            {
+                await blogPostContext.BlogPosts.AddAsync(blogPost);
+            }
+            else
+            {
+                blogPostContext.Entry(blogPost).State = EntityState.Modified;
+            }
+
             await blogPostContext.SaveChangesAsync();
         }
     }