From 04dbbb6b2753a3d0e7ea02fe402ed0ac5d57056d Mon Sep 17 00:00:00 2001
From: rxtur <rtur.net@gmail.com>
Date: Sun, 4 Apr 2021 00:10:44 -0500
Subject: [PATCH] Admin Panel Layout #266 - Categories

---
 .../Pages/Blog/CategoriesView.razor           | 273 +++++++-----------
 src/Blogifier.Core/Blogifier.Core.csproj      |   2 +-
 .../Providers/CategoryProvider.cs             | 115 +++++---
 src/Blogifier.Shared/Models/CategoryItem.cs   |  23 +-
 .../Controllers/CategoryController.cs         |  39 ++-
 5 files changed, 228 insertions(+), 224 deletions(-)

diff --git a/src/Blogifier.Admin/Pages/Blog/CategoriesView.razor b/src/Blogifier.Admin/Pages/Blog/CategoriesView.razor
index 166a11186..8c978d023 100644
--- a/src/Blogifier.Admin/Pages/Blog/CategoriesView.razor
+++ b/src/Blogifier.Admin/Pages/Blog/CategoriesView.razor
@@ -6,8 +6,6 @@
 @inject IJSRuntime _jsruntime
 @inject IToaster _toaster
 @inject NavigationManager _navigation
-@inject BlogStateProvider _stateprovider
-@implements IDisposable
 
 <PageTitle Title="@_localizer["Categories"]" />
 
@@ -19,9 +17,9 @@
             <input type="checkbox" @onchange="EventArgs => { CheckAll(EventArgs.Value); }" class="list-check-input form-check-input">
         </label>
 
-        <a class="btn btn-blogifier me-3 -add" href="/admin/pages/editor">@_localizer["New Category"]</a>
+        @*<a class="btn btn-blogifier me-3 -add" href="/admin/pages/editor">@_localizer["New Category"]</a>*@
 
-        @if (Posts != null && Posts.Count > 0)
+        @if (Categories != null && Categories.Count > 0)
         {
             <button type="button" @onclick="(async ()=> await RunAction(GroupAction.Delete))" class="btn btn-link text-danger" title="@_localizer["delete"]">
                 <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" class="bi bi-x-circle" viewBox="0 0 16 16">
@@ -41,29 +39,26 @@
 
     <div class="list-search collapse" id="collapseSearch">
         <input @bind="SearchTerm" @onkeyup="SearchKeyPress" class="list-search-input" type="search">
-        <button @onclick="(async () => await SearchPosts())" class="list-search-button" type="button">
+        <button @onclick="(async () => await SearchCategories())" class="list-search-button" type="button">
             @_localizer["search"]
         </button>
     </div>
 
-    @if (Posts != null && Posts.Count > 0)
+    @if (Categories != null && Categories.Count > 0)
     {
         <ul class="list mb-5" aria-label="posts">
-            <Virtualize Items="Posts" Context="post">
-
-                @{
-                    string pubDate = post.Published > DateTime.MinValue ? post.Published.ToFriendlyShortDateString() : "Draft";
-                    string pubStatus = post.Published > DateTime.MinValue ? "published" : "draft";
-                }
-
+            <Virtualize Items="Categories" Context="category">
                 <li class="list-item">
                     <label class="list-check form-check">
-                        <input type="checkbox" @bind="post.Selected" class="list-check-input form-check-input">
+                        <input type="checkbox" @bind="category.Selected" class="list-check-input form-check-input">
                     </label>
-                    <a class="list-title -link" href="/admin/pages/editor/@post.Slug">
-                        @post.Title
+                    <a class="list-title -link" href="" @onclick="async ()=> await ShowEdit(category.Id)" @onclick:preventDefault>
+                        @category.Category
                     </a>
-                    <a class="list-btn" href="posts/@post.Slug" target="_blank" data-toggle="tooltip" title="Link">
+                    <span class="list-text -nowrap ms-auto">
+                        @category.PostCount
+                    </span>
+                    <a class="list-btn" href="categories/@category.Category" target="_blank" data-toggle="tooltip" title="Link">
                         <svg xmlns="http://www.w3.org/2000/svg" width="17" height="17" fill="currentColor" class="bi bi-link-45deg" viewBox="0 0 16 16">
                             <path d="M4.715 6.542L3.343 7.914a3 3 0 1 0 4.243 4.243l1.828-1.829A3 3 0 0 0 8.586 5.5L8 6.086a1.001 1.001 0 0 0-.154.199 2 2 0 0 1 .861 3.337L6.88 11.45a2 2 0 1 1-2.83-2.83l.793-.792a4.018 4.018 0 0 1-.128-1.287z" />
                             <path d="M6.586 4.672A3 3 0 0 0 7.414 9.5l.775-.776a2 2 0 0 1-.896-3.346L9.12 3.55a2 2 0 0 1 2.83 2.83l-.793.792c.112.42.155.855.128 1.287l1.372-1.372a3 3 0 0 0-4.243-4.243L6.586 4.672z" />
@@ -85,167 +80,117 @@
     }
 
     <!-- New or Edit -->
+    @if(IsEdit && CurrentCategory != null)
+    {
     <div>
-        <h1 class="section-title">@_localizer["New Category"]</h1>
+        <h1 class="section-title">@_localizer["Edit Category"]</h1>
         <div class="section-content -half">
             <div class="form-item">
                 <label class="form-label">@_localizer["title"]</label>
-                <input type="text" class="form-control" name="title" />
+                <input type="text" class="form-control" name="title" @bind="CurrentCategory.Content" />
             </div>
-            <div class="form-item">
+            @*<div class="form-item">
                 <label class="form-label">@_localizer["description"]</label>
                 <textarea class="form-control" cols="30" rows="5" name="description"></textarea>
-            </div>
+            </div>*@
             <div class="form-item">
-                <button type="submit" class="btn btn-blogifier px-5 me-3">@_localizer["save"]</button>
-                <button type="button" class="btn btn-default px-4">@_localizer["cancel"]</button>
+                <button type="submit" class="btn btn-blogifier px-5 me-3" @onclick="SaveEdit">@_localizer["save"]</button>
+                <button type="button" class="btn btn-default px-4" @onclick="CancelEdit">@_localizer["cancel"]</button>
             </div>
         </div>
     </div>
+    }
     <!--/ -->
 
 </div>
+<ToasterComponent @ref="Toaster" />
 
 @code {
-    protected List<Post> Posts { get; set; }
-    protected Author Author { get; set; }
-
-    protected string SearchTerm { get; set; }
-
-    protected string FilterLabel { get; set; }
-    protected PublishedStatus FilterValue { get; set; }
-
-    protected string PostTypeLabel { get; set; }
-    protected string PostTypeButton { get; set; }
-
-    protected override async Task OnInitializedAsync()
-    {
-        Author = await _http.GetFromJsonAsync<Author>("api/author/getcurrent");
-
-        FilterLabel = _localizer["all"];
-        PostTypeLabel = _localizer["posts"];
-        PostTypeButton = _localizer["post"];
-
-        _stateprovider.OnChange += StateHasChanged;
-        _stateprovider.PostType = PostType.Post;
-
-        await Load();
-    }
-
-    protected async Task Load()
-    {
-        Posts = await _http.GetFromJsonAsync<List<Post>>($"api/post/list/{FilterValue}/{PostType.Page}");
-    }
-
-    public void CheckAll(object checkValue)
-    {
-        bool isChecked = (bool)checkValue;
-        Posts.ForEach(p => p.Selected = isChecked);
-        StateHasChanged();
-    }
-
-    public async Task RunAction(GroupAction action)
-    {
-        string confirm = _localizer["confirm-delete"];
-        bool confirmed = false;
-
-        if (action == GroupAction.Delete)
-        {
-            confirmed = await _jsruntime.InvokeAsync<bool>("confirm", confirm);
-            if (!confirmed)
-                return;
-        }
-
-        foreach (var post in Posts)
-        {
-            if (post.Selected)
-            {
-                switch (action)
-                {
-                    case GroupAction.Publish:
-                        await _http.PutAsJsonAsync($"api/post/publish/{post.Id}", true);
-                        break;
-                    case GroupAction.Unpublish:
-                        await _http.PutAsJsonAsync($"api/post/publish/{post.Id}", false);
-                        break;
-                    case GroupAction.Delete:
-                        await _http.DeleteAsync($"api/post/{post.Id}");
-                        break;
-                }
-            }
-        }
-        await Load();
-    }
-
-    protected async Task SearchKeyPress(KeyboardEventArgs e)
-    {
-        if (e.Key == "Enter")
-            await SearchPosts();
-    }
-
-    protected async Task SearchPosts()
-    {
-        if (string.IsNullOrEmpty(SearchTerm))
-            SearchTerm = "*";
-
-        Posts = await _http.GetFromJsonAsync<List<Post>>($"api/post/list/search/{SearchTerm}");
-        SearchTerm = "";
-    }
-
-    protected async Task GetPages()
-    {
-        PostTypeLabel = _localizer["pages"];
-        PostTypeButton = _localizer["page"];
-        _stateprovider.SetPostType(PostType.Page);
-        await Load();
-    }
-
-    public async Task Filter(PublishedStatus filter)
-    {
-        FilterValue = filter;
-        switch (filter)
-        {
-            case PublishedStatus.Published:
-                FilterLabel = _localizer["published"];
-                break;
-            case PublishedStatus.Drafts:
-                FilterLabel = _localizer["draft", true];
-                break;
-            default:
-                FilterLabel = _localizer["all"];
-                break;
-        }
-        await Load();
-    }
-
-    public async Task Publish(Post post)
-    {
-        Toast(await _http.PutAsJsonAsync($"api/post/publish/{post.Id}", (post.Published == DateTime.MinValue)));
-        await Load();
-    }
-
-    protected override async Task OnAfterRenderAsync(bool firstRender)
-    {
-        if (firstRender)
-        {
-            await _jsruntime.InvokeVoidAsync("commonJsFunctions.setTooltip");
-        }
-    }
-
-    protected void Toast(HttpResponseMessage msg)
-    {
-        if (msg.IsSuccessStatusCode)
-            _toaster.Success(_localizer["completed"]);
-        else
-            _toaster.Error(_localizer["generic-error"]);
-    }
-
-    protected void Open()
-    {
-    }
-
-    public void Dispose()
-    {
-        _stateprovider.OnChange -= StateHasChanged;
-    }
+   protected ToasterComponent Toaster;
+   protected List<CategoryItem> Categories { get; set; }
+   protected Category CurrentCategory { get; set; }
+   protected string SearchTerm { get; set; }
+   protected bool IsEdit = false;
+
+   protected override async Task OnInitializedAsync()
+   {
+       await Load();
+   }
+
+   protected async Task Load()
+   {
+       Categories = await _http.GetFromJsonAsync<List<CategoryItem>>($"api/category");
+   }
+
+   protected async Task ShowEdit(int categoryId)
+   {
+       CurrentCategory = await _http.GetFromJsonAsync<Category>($"api/category/byId/{categoryId}");
+       IsEdit = true;
+   }
+
+   protected void CancelEdit()
+   {
+       CurrentCategory = null;
+       IsEdit = false;
+   }
+
+   protected async Task SaveEdit()
+   {
+       Toaster.Toast(await _http.PutAsJsonAsync<Category>("api/category", CurrentCategory));
+       CurrentCategory = null;
+       IsEdit = false;
+       await Load();
+   }
+
+   public void CheckAll(object checkValue)
+   {
+       bool isChecked = (bool)checkValue;
+       Categories.ForEach(p => p.Selected = isChecked);
+       StateHasChanged();
+   }
+
+   public async Task RunAction(GroupAction action)
+   {
+       string confirm = _localizer["confirm-delete"];
+       bool confirmed = false;
+
+       if (action == GroupAction.Delete)
+       {
+           confirmed = await _jsruntime.InvokeAsync<bool>("confirm", confirm);
+           if (!confirmed)
+               return;
+       }
+
+       foreach (var category in Categories)
+       {
+           if (category.Selected)
+           {
+               await _http.DeleteAsync($"api/category/{category.Id}");
+           }
+       }
+       await Load();
+   }
+
+   protected async Task SearchKeyPress(KeyboardEventArgs e)
+   {
+       if (e.Key == "Enter")
+           await SearchCategories();
+   }
+
+   protected async Task SearchCategories()
+   {
+       if (string.IsNullOrEmpty(SearchTerm))
+           SearchTerm = "*";
+
+       Categories = await _http.GetFromJsonAsync<List<CategoryItem>>($"api/category/{SearchTerm}");
+       SearchTerm = "";
+   }
+
+   protected override async Task OnAfterRenderAsync(bool firstRender)
+   {
+       if (firstRender)
+       {
+           await _jsruntime.InvokeVoidAsync("commonJsFunctions.setTooltip");
+       }
+   }
 }
diff --git a/src/Blogifier.Core/Blogifier.Core.csproj b/src/Blogifier.Core/Blogifier.Core.csproj
index 3ba3999d0..b16773e08 100644
--- a/src/Blogifier.Core/Blogifier.Core.csproj
+++ b/src/Blogifier.Core/Blogifier.Core.csproj
@@ -2,7 +2,7 @@
 
 	<PropertyGroup>
 		<TargetFramework>net5.0</TargetFramework>
-		<Version>2.9.0.9</Version>
+		<Version>2.9.1.0</Version>
 		<Authors>blogifierdotnet</Authors>
 		<Description>Blogifier is an open-source publishing platform Written in .NET 5.0 and Blazor WebAssembly.</Description>
 		<Copyright>Blogifier.Net</Copyright>
diff --git a/src/Blogifier.Core/Providers/CategoryProvider.cs b/src/Blogifier.Core/Providers/CategoryProvider.cs
index ae375c53a..e9fda6047 100644
--- a/src/Blogifier.Core/Providers/CategoryProvider.cs
+++ b/src/Blogifier.Core/Providers/CategoryProvider.cs
@@ -10,15 +10,20 @@ namespace Blogifier.Core.Providers
 {
 	public interface ICategoryProvider
 	{
-		Task<IEnumerable<CategoryItem>> Categories();
-		Task<ICollection<Category>> GetPostCategories(int postId);
+        Task<List<CategoryItem>> Categories();
+        Task<List<CategoryItem>> SearchCategories(string term);
+
+        Task<Category> GetCategory(int categoryId);
+        Task<ICollection<Category>> GetPostCategories(int postId);
+
+        Task<bool> SaveCategory(Category category);
         Task<Category> SaveCategory(string tag);
 
         Task<bool> AddPostCategory(int postId, string tag);
         Task<bool> SavePostCategories(int postId, List<Category> categories);
 
-        Task<bool> RemoveCategory(int postId, int categoryId);
-	}
+        Task<bool> RemoveCategory(int categoryId);
+    }
 
 	public class CategoryProvider : ICategoryProvider
 	{
@@ -29,17 +34,24 @@ public CategoryProvider(AppDbContext db)
 			_db = db;
 		}
 
-		public async Task<IEnumerable<CategoryItem>> Categories()
-		{
-			var cats = new List<CategoryItem>();
+        public async Task<List<CategoryItem>> Categories()
+        {
+            var cats = new List<CategoryItem>();
 
-			if (_db.Posts != null && _db.Posts.Count() > 0)
-			{
-				foreach (var pc in _db.PostCategories.Include(pc => pc.Category).AsNoTracking())
-				{
+            if (_db.Posts != null && _db.Posts.Count() > 0)
+            {
+                foreach (var pc in _db.PostCategories.Include(pc => pc.Category).AsNoTracking())
+                {
                     if (!cats.Exists(c => c.Category.ToLower() == pc.Category.Content.ToLower()))
                     {
-                        cats.Add(new CategoryItem { Category = pc.Category.Content.ToLower(), PostCount = 1 });
+                        cats.Add(new CategoryItem
+                        {
+                            Selected = false,
+                            Id = pc.CategoryId,
+                            Category = pc.Category.Content.ToLower(),
+                            PostCount = 1,
+                            DateCreated = pc.Category.DateCreated
+                        });
                     }
                     else
                     {
@@ -49,10 +61,27 @@ public async Task<IEnumerable<CategoryItem>> Categories()
                     }
                 }
             }
-			return await Task.FromResult(cats);
-		}
+            return await Task.FromResult(cats);
+        }
+
+        public async Task<List<CategoryItem>> SearchCategories(string term)
+        {
+            var cats = await Categories();
+
+            if (term == "*")
+                return cats;
 
-		public async Task<ICollection<Category>> GetPostCategories(int postId)
+            return cats.Where(c => c.Category.ToLower().Contains(term.ToLower())).ToList();
+        }
+
+        public async Task<Category> GetCategory(int categoryId)
+        {
+            return await _db.Categories.AsNoTracking()
+                .Where(c => c.Id == categoryId)
+                .FirstOrDefaultAsync();
+        }
+
+        public async Task<ICollection<Category>> GetPostCategories(int postId)
 		{
             return await _db.PostCategories.AsNoTracking()
                 .Where(pc => pc.PostId == postId)
@@ -60,6 +89,24 @@ public async Task<ICollection<Category>> GetPostCategories(int postId)
                 .ToListAsync();
 		}
 
+        public async Task<bool> SaveCategory(Category category)
+        {
+            Category existing = await _db.Categories.AsNoTracking()
+                .Where(c => c.Content.ToLower() == category.Content.ToLower()).FirstOrDefaultAsync();
+
+            if (existing != null)
+                return false; // already exists category with the same title
+
+            Category dbCategory = await _db.Categories.Where(c => c.Id == category.Id).FirstOrDefaultAsync();
+            if (dbCategory == null)
+                return false;
+
+            dbCategory.Content = category.Content;
+            dbCategory.DateUpdated = DateTime.UtcNow;
+
+            return await _db.SaveChangesAsync() > 0;
+        }
+
 		public async Task<Category> SaveCategory(string tag)
 		{
 			Category category = await _db.Categories
@@ -119,10 +166,7 @@ public async Task<bool> SavePostCategories(int postId, List<Category> categories
             List<PostCategory> existingPostCategories = await _db.PostCategories.AsNoTracking()
                 .Where(pc => pc.PostId == postId).ToListAsync();
 
-            foreach (var pc in existingPostCategories)
-            {
-                await RemoveCategory(postId, pc.CategoryId);
-            }
+            _db.PostCategories.RemoveRange(existingPostCategories);
 
             await _db.SaveChangesAsync();
 
@@ -134,33 +178,20 @@ public async Task<bool> SavePostCategories(int postId, List<Category> categories
             return await _db.SaveChangesAsync() > 0;
         }
 
-		public async Task<bool> RemoveCategory(int postId, int categoryId)
-		{
-            PostCategory postCategory = await _db.PostCategories
+        public async Task<bool> RemoveCategory(int categoryId)
+        {
+            List<PostCategory> postCategories = await _db.PostCategories
                 .AsNoTracking()
                 .Where(pc => pc.CategoryId == categoryId)
-                .Where(pc => pc.PostId == postId)
-                .FirstOrDefaultAsync();
-
-            if(postCategory != null)
-            {
-                _db.PostCategories.Remove(postCategory);
-
-                int postCount = await _db.PostCategories.AsNoTracking()
-                    .Where(pc => pc.CategoryId == categoryId).CountAsync();
+                .ToListAsync();
+            _db.PostCategories.RemoveRange(postCategories);
 
-                if(postCount == 1)
-                {
-                    Category category = _db.Categories
+            Category category = _db.Categories
                         .Where(c => c.Id == categoryId)
                         .FirstOrDefault();
-                    _db.Categories.Remove(category);
-                }
+            _db.Categories.Remove(category);
 
-                return await _db.SaveChangesAsync() > 0;
-            }
-
-            return true;
-		}
-	}
+            return await _db.SaveChangesAsync() > 0;
+        }
+    }
 }
diff --git a/src/Blogifier.Shared/Models/CategoryItem.cs b/src/Blogifier.Shared/Models/CategoryItem.cs
index 186c1dde9..3500c8c3d 100644
--- a/src/Blogifier.Shared/Models/CategoryItem.cs
+++ b/src/Blogifier.Shared/Models/CategoryItem.cs
@@ -1,15 +1,18 @@
-using System;
+using System;
 
 namespace Blogifier.Shared
 {
-	public class CategoryItem : IComparable<CategoryItem>
-   {
-      public string Category { get; set; }
-      public int PostCount { get; set; }
+    public class CategoryItem : IComparable<CategoryItem>
+    {
+        public bool Selected { get; set; }
+        public int Id { get; set; }
+        public string Category { get; set; }
+        public int PostCount { get; set; }
+        public DateTime DateCreated { get; set; }
 
-      public int CompareTo(CategoryItem other)
-      {
-         return Category.ToLower().CompareTo(other.Category.ToLower());
-      }
-   }
+        public int CompareTo(CategoryItem other)
+        {
+            return Category.ToLower().CompareTo(other.Category.ToLower());
+        }
+    }
 }
diff --git a/src/Blogifier/Controllers/CategoryController.cs b/src/Blogifier/Controllers/CategoryController.cs
index 2af2ee778..d4504da9c 100644
--- a/src/Blogifier/Controllers/CategoryController.cs
+++ b/src/Blogifier/Controllers/CategoryController.cs
@@ -24,13 +24,38 @@ public async Task<ICollection<Category>> GetPostCategories(int postId)
 			return await _categoryProvider.GetPostCategories(postId);
 		}
 
-		[Authorize]
+        [HttpGet("byId/{categoryId:int}")]
+        public async Task<Category> GetCategory(int categoryId)
+        {
+            return await _categoryProvider.GetCategory(categoryId);
+        }
+
+        [HttpGet]
+        public async Task<List<CategoryItem>> GetCategories()
+        {
+            return await _categoryProvider.Categories();
+        }
+
+        [HttpGet("{term}")]
+        public async Task<List<CategoryItem>> SearchCategories(string term = "*")
+        {
+            return await _categoryProvider.SearchCategories(term);
+        }
+
+        [Authorize]
 		[HttpPost("{postId:int}/{tag}")]
 		public async Task<ActionResult<bool>> AddPostCategory(int postId, string tag)
 		{
 			return await _categoryProvider.AddPostCategory(postId, tag);
 		}
 
+        [Authorize]
+        [HttpPut]
+        public async Task<ActionResult<bool>> SaveCategory(Category category)
+        {
+            return await _categoryProvider.SaveCategory(category);
+        }
+
         [Authorize]
         [HttpPut("{postId:int}")]
         public async Task<ActionResult<bool>> SavePostCategories(int postId, List<Category> categories)
@@ -39,10 +64,10 @@ public async Task<ActionResult<bool>> SavePostCategories(int postId, List<Catego
         }
 
         [Authorize]
-		[HttpDelete("{postId:int}/{categoryId:int}")]
-		public async Task<ActionResult<bool>> RemoveCategory(int postId, int categoryId)
-		{
-			return await _categoryProvider.RemoveCategory(postId, categoryId);
-		}
-	}
+        [HttpDelete("{categoryId:int}")]
+        public async Task<ActionResult<bool>> RemoveCategory(int categoryId)
+        {
+            return await _categoryProvider.RemoveCategory(categoryId);
+        }
+    }
 }