Fix Blade stack requests (#375)

* Fix outdated action reference

* Fix some issues in the EditCategory form request

* Fix various issues in the form requests

* Fix an issue in the EditCategory action

* Specify vuedraggable@next in the Blade presets' required packages

* Fix color inputs for the Blade presets

Author: Riari

src/Actions/EditCategory.php

@@ -16,13 +16,15 @@ public function __construct(Category $category, string $title, string $descripti
 
     protected function transact()
     {
-        return $this->category->update([
+        $this->category->update([
             'title' => $this->title,
             'description' => $this->description,
             'color_light_mode' => $this->colorLightMode,
             'color_dark_mode' => $this->colorDarkMode,
             'accepts_threads' => $this->acceptsThreads,
             'is_private' => $this->isPrivate,
         ]);
+
+        return $this->category;
     }
 }

src/Frontend/Presets/BladeBootstrapPreset.php

@@ -29,7 +29,7 @@ public static function getRequiredPackages(): array
             'bootstrap',
             'feather-icons',
             'vue',
-            'vuedraggable',
+            'vuedraggable@next',
         ];
     }
 }

src/Frontend/Presets/BladeTailwindPreset.php

@@ -34,7 +34,7 @@ public static function getRequiredPackages(): array
             'feather-icons',
             'tailwindcss',
             'vue',
-            'vuedraggable',
+            'vuedraggable@next',
         ];
     }
 

src/Http/Requests/EditCategory.php

@@ -11,20 +11,21 @@ class EditCategory extends CreateCategory
 {
     public function fulfill()
     {
+        $defaultCategoryColor = config('forum.frontend.default_category_color');
+
         $input = $this->validated();
         $action = new Action(
             $this->route('category'),
-            $input['title'] ?? null,
-            $input['description'] ?? null,
-            $input['color'] ?? null,
+            $input['title'],
+            $input['description'] ?? "",
+            $input['color_light_mode'] ?? $defaultCategoryColor,
+            $input['color_dark_mode'] ?? $defaultCategoryColor,
             $input['accepts_threads'] ?? null,
             $input['is_private'] ?? null
         );
         $category = $action->execute();
 
-        if (!$category === null) {
-            UserEditedCategory::dispatch($this->user(), $category);
-        }
+        UserEditedCategory::dispatch($this->user(), $category);
 
         return $category;
     }

src/Http/Requests/LockThread.php

@@ -26,9 +26,7 @@ public function fulfill()
         $action = new Action($this->route('thread'));
         $thread = $action->execute();
 
-        if ($thread !== null) {
-            UserLockedThread::dispatch($this->user(), $thread);
-        }
+        UserLockedThread::dispatch($this->user(), $thread);
 
         return $thread;
     }

src/Http/Requests/MoveThread.php

@@ -37,9 +37,7 @@ public function fulfill()
         $action = new Action($thread, $destinationCategory);
         $thread = $action->execute();
 
-        if (!$thread === null) {
-            UserMovedThread::dispatch($this->user(), $thread, $sourceCategory, $destinationCategory);
-        }
+        UserMovedThread::dispatch($this->user(), $thread, $sourceCategory, $destinationCategory);
 
         return $thread;
     }

src/Http/Requests/PinThread.php

@@ -26,9 +26,7 @@ public function fulfill()
         $action = new Action($this->route('thread'));
         $thread = $action->execute();
 
-        if ($thread !== null) {
-            UserPinnedThread::dispatch($this->user(), $thread);
-        }
+        UserPinnedThread::dispatch($this->user(), $thread);
 
         return $thread;
     }

src/Http/Requests/RestorePost.php

@@ -26,9 +26,7 @@ public function fulfill()
         $action = new Action($this->route('post'));
         $post = $action->execute();
 
-        if ($post !== null) {
-            UserRestoredPost::dispatch($this->user(), $post);
-        }
+        UserRestoredPost::dispatch($this->user(), $post);
 
         return $post;
     }

src/Http/Requests/RestoreThread.php

@@ -26,9 +26,7 @@ public function fulfill()
         $action = new Action($this->route('thread'));
         $thread = $action->execute();
 
-        if (!$thread === null) {
-            UserRestoredThread::dispatch($this->user(), $thread);
-        }
+        UserRestoredThread::dispatch($this->user(), $thread);
 
         return $thread;
     }

src/Http/Requests/UnlockThread.php

@@ -14,9 +14,7 @@ public function fulfill()
         $action = new Action($this->route('thread'));
         $thread = $action->execute();
 
-        if ($thread !== null) {
-            UserUnlockedThread::dispatch($this->user(), $thread);
-        }
+        UserUnlockedThread::dispatch($this->user(), $thread);
 
         return $thread;
     }