From 6995377b6f8d924a1eceb75dffaf019d3e56dbbe Mon Sep 17 00:00:00 2001
From: amirghasempoor <amirghasempoor79@gmail.com>
Date: Wed, 7 May 2025 14:42:22 +0330
Subject: [PATCH] add id to guarded

---
 app/Http/Controllers/Admin/RoleManagementController.php  | 2 ++
 app/Http/Controllers/Admin/UserManagementController.php  | 6 +++---
 app/Http/Requests/Admin/UserManagement/UpdateRequest.php | 3 +--
 app/Models/City.php                                      | 2 +-
 app/Models/Event.php                                     | 2 +-
 app/Models/Permission.php                                | 2 +-
 app/Models/Province.php                                  | 2 +-
 app/Models/Role.php                                      | 2 +-
 app/Models/User.php                                      | 2 +-
 9 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/app/Http/Controllers/Admin/RoleManagementController.php b/app/Http/Controllers/Admin/RoleManagementController.php
index 0ca0947..8f9fec0 100644
--- a/app/Http/Controllers/Admin/RoleManagementController.php
+++ b/app/Http/Controllers/Admin/RoleManagementController.php
@@ -47,6 +47,7 @@ class RoleManagementController extends Controller
             $role = Role::create([
                 'name' => $request->name,
                 'name_fa' => $request->name_fa,
+                'guard_name' => 'web'
             ]);
 
             $role->syncPermissions($request->permissions);
@@ -72,6 +73,7 @@ class RoleManagementController extends Controller
             $role->update([
                 'name' => $request->name,
                 'name_fa' => $request->name_fa,
+                'guard_name' => 'web'
             ]);
 
             $role->syncPermissions($request->permissions);
diff --git a/app/Http/Controllers/Admin/UserManagementController.php b/app/Http/Controllers/Admin/UserManagementController.php
index be1e8c6..e5b4e48 100644
--- a/app/Http/Controllers/Admin/UserManagementController.php
+++ b/app/Http/Controllers/Admin/UserManagementController.php
@@ -10,6 +10,7 @@ use App\Http\Requests\Admin\UserManagement\UpdateRequest;
 use App\Http\Resources\Admin\UserResource;
 use App\Http\Traits\ApiResponse;
 use App\Models\Province;
+use App\Models\Role;
 use App\Models\User;
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
@@ -66,7 +67,7 @@ class UserManagementController extends Controller
                 'telephone_id' => $request->telephone_id,
             ]);
 
-            $user->assignRole($request->role_id);
+            $user->assignRole(Role::query()->find($request->role_id));
         });
 
         return $this->successResponse();
@@ -97,10 +98,9 @@ class UserManagementController extends Controller
                 'province_fa' =>  $province->name,
                 'avatar' => $avatar,
                 'email' => $request->email,
-                'telephone_id' => $request->telephone_id
             ]);
 
-            $user->syncRoles($request->role);
+            $user->syncRoles(Role::query()->find($request->role_id));
         });
 
         return $this->successResponse();
diff --git a/app/Http/Requests/Admin/UserManagement/UpdateRequest.php b/app/Http/Requests/Admin/UserManagement/UpdateRequest.php
index c953eb3..436995c 100644
--- a/app/Http/Requests/Admin/UserManagement/UpdateRequest.php
+++ b/app/Http/Requests/Admin/UserManagement/UpdateRequest.php
@@ -36,10 +36,9 @@ class UpdateRequest extends FormRequest
                 'regex:/^09\d{9}$/'
             ],
             'province_id' => 'required|exists:provinces,id',
-            'role' => 'required|exists:roles,id',
+            'role_id' => 'required|exists:roles,id',
             'avatar' => 'mimes:png,jpg,pdf|max:2048',
             'email' => ['email', Rule::unique('users', 'email')->ignore($this->user->id)],
-            'telephone_id' => ['required', 'string', Rule::unique('users', 'telephone_id')->ignore($this->user->id)]
         ];
     }
 }
diff --git a/app/Models/City.php b/app/Models/City.php
index 05fc89f..21dcf0e 100644
--- a/app/Models/City.php
+++ b/app/Models/City.php
@@ -8,5 +8,5 @@ use Illuminate\Database\Eloquent\Model;
 class City extends Model
 {
     use HasFactory;
-    protected $guarded = [];
+    protected $guarded = ['id'];
 }
diff --git a/app/Models/Event.php b/app/Models/Event.php
index e687502..6177e5a 100644
--- a/app/Models/Event.php
+++ b/app/Models/Event.php
@@ -8,5 +8,5 @@ use Illuminate\Database\Eloquent\Model;
 class Event extends Model
 {
     use HasFactory;
-    protected $guarded = [];
+    protected $guarded = ['id'];
 }
diff --git a/app/Models/Permission.php b/app/Models/Permission.php
index 87455f1..082b63c 100644
--- a/app/Models/Permission.php
+++ b/app/Models/Permission.php
@@ -9,5 +9,5 @@ use Spatie\Permission\Models\Permission as SpatiePermission;
 class Permission extends SpatiePermission
 {
     use HasFactory;
-    protected $guarded = [];
+    protected $guarded = ['id'];
 }
diff --git a/app/Models/Province.php b/app/Models/Province.php
index 9ae60b9..ba3097d 100644
--- a/app/Models/Province.php
+++ b/app/Models/Province.php
@@ -8,5 +8,5 @@ use Illuminate\Database\Eloquent\Model;
 class Province extends Model
 {
     use HasFactory;
-    protected $guarded = [];
+    protected $guarded = ['id'];
 }
diff --git a/app/Models/Role.php b/app/Models/Role.php
index 1371e16..dbd0c8c 100644
--- a/app/Models/Role.php
+++ b/app/Models/Role.php
@@ -9,5 +9,5 @@ use Spatie\Permission\Models\Role as SpatieRole;
 class Role extends SpatieRole
 {
     use HasFactory;
-    protected $guarded = [];
+    protected $guarded = ['id'];
 }
diff --git a/app/Models/User.php b/app/Models/User.php
index f3f3154..344e3f2 100644
--- a/app/Models/User.php
+++ b/app/Models/User.php
@@ -13,7 +13,7 @@ class User extends Authenticatable
 {
     use HasFactory, Notifiable, HasRoles;
 
-    protected $guarded = [];
+    protected $guarded = ['id'];
 
     protected $hidden = ['password', 'pivot'];
     public string $guard_name = 'web';