From d04637b1efea7a4e1b4eb95e82007c052d62a071 Mon Sep 17 00:00:00 2001
From: faezehzafarbakhsh <faezehzafarbakhsh@gmail.com>
Date: Mon, 18 May 2026 14:10:45 +0330
Subject: [PATCH] fix code

---
 .../Admin/Controllers/ProfileController.php   |  1 -
 .../Admin/Requests/Profile/EditRequest.php    | 45 ++-----------------
 .../User/Controllers/ProfileController.php    |  8 +---
 .../app/User/Requests/Profile/EditRequest.php | 38 ++++++++--------
 4 files changed, 22 insertions(+), 70 deletions(-)

diff --git a/backend/app/Admin/Controllers/ProfileController.php b/backend/app/Admin/Controllers/ProfileController.php
index dde39f98..23a80f7e 100755
--- a/backend/app/Admin/Controllers/ProfileController.php
+++ b/backend/app/Admin/Controllers/ProfileController.php
@@ -23,7 +23,6 @@ class ProfileController extends Controller
     public function edit(EditRequest $request): JsonResponse
     {
         $expert = $request->user('expert');
-//        $expert->update($request->validated());
 
         $expert->update([
             'username' => $request->username,
diff --git a/backend/app/Admin/Requests/Profile/EditRequest.php b/backend/app/Admin/Requests/Profile/EditRequest.php
index 2e56b73f..ccc056b4 100755
--- a/backend/app/Admin/Requests/Profile/EditRequest.php
+++ b/backend/app/Admin/Requests/Profile/EditRequest.php
@@ -26,53 +26,14 @@ class EditRequest extends FormRequest
     {
         $user = $this->user();
         return [
-            'first_name' => [
-                Rule::prohibitedIf($user->kyc_status == 1), 'string',
-            ],
-
-            'last_name' => [
-                Rule::prohibitedIf($user->kyc_status == 1), 'string',
-            ],
-
-            'national_id' => [
-                Rule::prohibitedIf($user->kyc_status == 1), 'string',
-            ],
             'username' => 'string|max:255',
-
+            'first_name' => 'required|string',
+            'last_name' => 'required|string',
+            'national_id' => 'required',
             'email' => 'string|email|max:255',
             'phone_number' => 'string',
             'province_id' => 'string',
             'city_id' => 'string',
         ];
     }
-
-    public function after(): array
-    {
-        return [
-            function (Validator $validator) {
-                $user = $this->user();
-
-                if (! $user) {
-                    return;
-                }
-
-                if ($user->kyc_status == 1) {
-                    $lockedFields = [
-                        'first_name',
-                        'last_name',
-                        'national_id',
-                    ];
-
-                    foreach ($lockedFields as $field) {
-                        if ($this->filled($field)) {
-                            $validator->errors()->add(
-                                $field,
-                                'After KYC approval, you cannot change this field.'
-                            );
-                        }
-                    }
-                }
-            },
-        ];
-    }
 }
diff --git a/backend/app/User/Controllers/ProfileController.php b/backend/app/User/Controllers/ProfileController.php
index 9afaa04a..3d89e5dc 100755
--- a/backend/app/User/Controllers/ProfileController.php
+++ b/backend/app/User/Controllers/ProfileController.php
@@ -24,13 +24,7 @@ class ProfileController extends Controller
     {
         $user = Auth::user();
 
-        $data = $request->validated();
-
-        if ($user->kyc_status === 2) {
-            $data->safe()->except(['first_name', 'last_name' ,'national_id']);
-        }
-
-        $user->update($data);
+        $user->update($request->validated());
 
         return $this->successResponse();
     }
diff --git a/backend/app/User/Requests/Profile/EditRequest.php b/backend/app/User/Requests/Profile/EditRequest.php
index 4b0e1eb4..3f7468bd 100755
--- a/backend/app/User/Requests/Profile/EditRequest.php
+++ b/backend/app/User/Requests/Profile/EditRequest.php
@@ -25,31 +25,17 @@ class EditRequest extends FormRequest
     public function rules(): array
     {
         $user = $this->user();
-        $isKycApproved = (int) $user->kyc_status === 2;
-        
+
         return [
             'first_name' => [
-                Rule::prohibitedIf($isKycApproved),
-                'required',
-                'string',
-                'max:255',
+                Rule::prohibitedIf($user->kyc_status == 1), 'string',
             ],
-
             'last_name' => [
-                Rule::prohibitedIf($isKycApproved),
-                'required',
-                'string',
-                'max:255',
+                Rule::prohibitedIf($user->kyc_status == 1), 'string',
             ],
-
             'national_id' => [
-                Rule::prohibitedIf($isKycApproved),
-                'required',
-                'string',
-                'max:20',
-                Rule::unique('users', 'national_id')->ignore($user->id),
+                Rule::prohibitedIf($user->kyc_status == 1), 'string',
             ],
-
             'address' => 'required',
             'postal_code' => 'required',
             'phone_number' => 'required',
@@ -62,9 +48,21 @@ class EditRequest extends FormRequest
         return [
             function (Validator $validator) {
                 $user = $this->user();
+                if ($user->kyc_status == 1) {
+                    $lockedFields = [
+                        'first_name',
+                        'last_name',
+                        'national_id',
+                    ];
 
-                if ( $user->kyc_status === 1) {
-                    if
+                    foreach ($lockedFields as $field) {
+                        if ($this->filled($field)) {
+                            $validator->errors()->add(
+                                $field,
+                                'you cannot change this field.'
+                            );
+                        }
+                    }
                 }
             },
         ];