From 37da92d87c6759427efd40b90734a1fe792059fd Mon Sep 17 00:00:00 2001
From: joddyabott <faezehzafarbakhsh@gmail.com>
Date: Mon, 17 Feb 2025 14:49:26 +0330
Subject: [PATCH] fix controller,request and route

---
 .../V3/DamageManagementController.php         | 10 ++-----
 app/Http/Requests/V3/Damage/DeleteRequest.php | 28 -------------------
 app/Http/Requests/V3/Damage/StoreRequest.php  |  1 -
 app/Http/Requests/V3/Damage/UpdateRequest.php |  4 +--
 routes/v3.php                                 |  2 +-
 5 files changed, 6 insertions(+), 39 deletions(-)
 delete mode 100644 app/Http/Requests/V3/Damage/DeleteRequest.php

diff --git a/app/Http/Controllers/V3/DamageManagementController.php b/app/Http/Controllers/V3/DamageManagementController.php
index a00eac3c..27da5b63 100644
--- a/app/Http/Controllers/V3/DamageManagementController.php
+++ b/app/Http/Controllers/V3/DamageManagementController.php
@@ -4,7 +4,6 @@ namespace App\Http\Controllers\V3;
 
 use App\Facades\DataTable\DataTableFacade;
 use App\Http\Controllers\Controller;
-use App\Http\Requests\V3\Damage\DeleteRequest;
 use App\Http\Requests\V3\Damage\StoreRequest;
 use App\Http\Requests\V3\Damage\UpdateRequest;
 use App\Http\Traits\ApiResponse;
@@ -51,7 +50,7 @@ class DamageManagementController extends Controller
             'title' => $request->title,
             'unit' => $request->unit,
             'base_price' => $request->base_price,
-            'status' => $request->status,
+            'status' => 1,
         ]);
 
         return $this->successResponse();
@@ -77,7 +76,6 @@ class DamageManagementController extends Controller
             'title' => $request->title,
             'unit' => $request->unit,
             'base_price' => $request->base_price,
-            'status' => $request->status,
         ]);
 
         return $this->successResponse();
@@ -86,11 +84,9 @@ class DamageManagementController extends Controller
     /**
      * Remove the specified resource from storage.
      */
-    public function destroy(DeleteRequest $request, Damage $damage): JsonResponse
+    public function activate(Damage $damage): JsonResponse
     {
-        $damage->update([
-            'status' => $request->status,
-        ]);
+        $damage->status == 1 ? $damage->update(['status' => 0]) : $damage->update(['status' => 1]);
 
         return $this->successResponse();
     }
diff --git a/app/Http/Requests/V3/Damage/DeleteRequest.php b/app/Http/Requests/V3/Damage/DeleteRequest.php
deleted file mode 100644
index aea281d6..00000000
--- a/app/Http/Requests/V3/Damage/DeleteRequest.php
+++ /dev/null
@@ -1,28 +0,0 @@
-<?php
-
-namespace App\Http\Requests\V3\Damage;
-
-use Illuminate\Foundation\Http\FormRequest;
-
-class DeleteRequest extends FormRequest
-{
-    /**
-     * Determine if the user is authorized to make this request.
-     */
-    public function authorize(): bool
-    {
-        return true;
-    }
-
-    /**
-     * Get the validation rules that apply to the request.
-     *
-     * @return array<string, \Illuminate\Contracts\Validation\ValidationRule|array<mixed>|string>
-     */
-    public function rules(): array
-    {
-        return [
-            'status' => 'required|in:0,1',
-        ];
-    }
-}
diff --git a/app/Http/Requests/V3/Damage/StoreRequest.php b/app/Http/Requests/V3/Damage/StoreRequest.php
index 2e3e88ef..6d8647b0 100644
--- a/app/Http/Requests/V3/Damage/StoreRequest.php
+++ b/app/Http/Requests/V3/Damage/StoreRequest.php
@@ -25,7 +25,6 @@ class StoreRequest extends FormRequest
             'title' =>'required|string|max:255|unique:damages,title',
             'unit' =>'required|string|max:255',
             'base_price' =>'required|integer',
-            'status' =>'required|in:0,1',
         ];
     }
 }
diff --git a/app/Http/Requests/V3/Damage/UpdateRequest.php b/app/Http/Requests/V3/Damage/UpdateRequest.php
index d3d0e038..70b0833a 100644
--- a/app/Http/Requests/V3/Damage/UpdateRequest.php
+++ b/app/Http/Requests/V3/Damage/UpdateRequest.php
@@ -3,6 +3,7 @@
 namespace App\Http\Requests\V3\Damage;
 
 use Illuminate\Foundation\Http\FormRequest;
+use Illuminate\Validation\Rule;
 
 class UpdateRequest extends FormRequest
 {
@@ -22,10 +23,9 @@ class UpdateRequest extends FormRequest
     public function rules(): array
     {
         return [
-            'title' =>'required|string|unique:damages,title',
+            'title' =>['required', 'string', 'max:255',Rule::unique('damages','title')->ignore($this->damage->id)],
             'unit' =>'required|string',
             'base_price' =>'required|numeric',
-            'status' =>'required|integer',
         ];
     }
 }
diff --git a/routes/v3.php b/routes/v3.php
index 9a1abcc3..b21aeb26 100644
--- a/routes/v3.php
+++ b/routes/v3.php
@@ -226,5 +226,5 @@ Route::prefix('damages')
        Route::post('/', 'store')->name('store');
        Route::get('/{damage}', 'show')->name('show');
        Route::post('/{damage}', 'update')->name('update');
-       Route::post('/delete/{damage}', 'destroy')->name('destroy');
+       Route::post('/activate/{damage}', 'activate')->name('activate');
     });