fix js menus and handle api menus

app/Http/Controllers/Api/MenusController.php

@@ -3,8 +3,11 @@
 namespace App\Http\Controllers\Api;
 
 use App\Http\Controllers\Controller;
+use App\Http\Requests\MenuRequest;
+use App\Http\Resources\MenuResource;
 use App\Models\Menu;
 use Illuminate\Http\Request;
+use Illuminate\Support\Facades\Log;
 
 class MenusController extends Controller
 {
@@ -13,7 +16,7 @@ class MenusController extends Controller
      */
     public function index(Request $request)
     {
-        $query = Menu::query();
+        $query = Menu::query()->orderBy('id', 'desc');
 
         if($request->has("search") && !empty($request->get("search"))){
             $query = $query->where("name", "like", "%".$request->get("search")."%");
@@ -25,9 +28,15 @@ class MenusController extends Controller
     /**
      * Store a newly created resource in storage.
      */
-    public function store(Request $request)
+    public function store(MenuRequest $request)
     {
-        //
+        try{
+            $menu = Menu::create($request->validated());
+            return response()->json(['message' => 'Menu created successfully', 'data' => new MenuResource($menu)]);
+        }catch(\Exception $e){
+            Log::error($e);
+            return response()->json(['message' => 'Error when creating menu'], 500);
+        }
     }
 
     /**
@@ -35,15 +44,37 @@ class MenusController extends Controller
      */
     public function show(string $id)
     {
-        //
+        try{
+            $menu = Menu::find($id);
+            if($menu){
+                return response()->json(['message' => 'Menu found', 'data' => new MenuResource($menu)]);
+            } else {
+                return response()->json(['message' => 'Menu not found'], 404);
+            }
+        }catch(\Exception $e){
+            Log::error($e);
+            Log::error($e->getMessage());
+            return response()->json(['message' => 'Error when finding menu'], 500);
+        }
     }
 
     /**
      * Update the specified resource in storage.
      */
-    public function update(Request $request, string $id)
+    public function update(MenuRequest $request, string $id)
     {
-        //
+        try{
+            $menu = Menu::findOrFail($id);
+            if($menu){
+                $menu->update($request->validated());
+                return response()->json(['message' => 'Menu updated successfully', 'data' => new MenuResource($menu)]);
+            } else {
+                return response()->json(['message' => 'Menu not found'], 404);
+            }
+        }catch(\Exception $e){
+            Log::error($e);
+            return response()->json(['message' => 'Error when updating menu'], 500);
+        }
     }
 
     /**
@@ -51,6 +82,28 @@ class MenusController extends Controller
      */
     public function destroy(string $id)
     {
-        //
+        try{
+            $menu = Menu::findOrFail($id);
+            if($menu){
+                $this->deleteChildren($menu);
+                $menu->roles()->detach();
+                $menu->delete();
+                return response()->json(['message' => 'Menu deleted successfully']);
+            } else {
+                return response()->json(['message' => 'Menu not found'], 404);
+            }
+        }catch(\Exception $e){
+            Log::error($e);
+            return response()->json(['message' => 'Error when deleting menu'], 500);
+        }
+    }
+
+    private function deleteChildren($menu)
+    {
+        foreach ($menu->children as $child) {
+            $this->deleteChildren($child); // Recursively delete its children
+            $child->roles()->detach(); // Detach roles before deleting
+            $child->delete();
+        }
     }
 }

app/Http/Requests/MenuRequest.php

@@ -3,6 +3,7 @@
 namespace App\Http\Requests;
 
 use Illuminate\Foundation\Http\FormRequest;
+use Illuminate\Validation\Rule;
 
 class MenuRequest extends FormRequest
 {
@@ -21,12 +22,14 @@ class MenuRequest extends FormRequest
      */
     public function rules(): array
     {
+        $menuId = $this->route('menu_id'); // Get the menu ID if updating
+
         return [
-            'name' => ['required','string','max:255'],
-            'url' => ['nullable','string','max:255'],
-            'icon' => ['nullable','string','max:255'],
-            'parent_id' => ['nullable','exists:menus,id'],
-            'sort_order' => ['required','integer'],
+            'name' => ['required', 'string', 'max:255', Rule::unique('menus', 'name')->ignore($menuId)],
+            'url' => ['nullable', 'string', 'max:255'],
+            'icon' => ['nullable', 'string', 'max:255'],
+            'parent_id' => ['nullable', 'exists:menus,id'],
+            'sort_order' => ['required', 'integer'],
         ];
     }
 }

app/Http/Resources/MenuResource.php

@@ -0,0 +1,19 @@
+<?php
+
+namespace App\Http\Resources;
+
+use Illuminate\Http\Request;
+use Illuminate\Http\Resources\Json\JsonResource;
+
+class MenuResource extends JsonResource
+{
+    /**
+     * Transform the resource into an array.
+     *
+     * @return array<string, mixed>
+     */
+    public function toArray(Request $request): array
+    {
+        return parent::toArray($request);
+    }
+}