LP-10 registration request proof_of address required added

fintechbd · Nov 17, 2023 · 4fcc284 · 4fcc284
1 parent 71b97bc
commit 4fcc284
Show file tree

Hide file tree

Showing 5 changed files with 68 additions and 65 deletions.
diff --git a/config/auth.php b/config/auth.php
@@ -185,12 +185,12 @@
         'photo' => ['string', 'nullable'],
         'documents' => ['array', 'required', 'min:1'],
         'documents.*.type' => ['string', 'required'],
-        'documents.*.front' => ['string', 'required'],
-        'documents.*.back' => ['string', 'nullable'],
+        'documents.*.back' => ['string', 'required_without:documents.*.front'],
+        'documents.*.front' => ['string', 'required_without:documents.*.back'],
         'proof_of_address' => ['array', 'required', 'min:1'],
         'proof_of_address.*.type' => ['string', 'required'],
-        'proof_of_address.*.front' => ['string', 'required'],
-        'proof_of_address.*.back' => ['string', 'nullable'],
+        'proof_of_address.*.back' => ['string', 'required_without:proof_of_address.*.front'],
+        'proof_of_address.*.front' => ['string', 'required_without:proof_of_address.*.back'],
         'date_of_birth' => ['date', 'nullable'],
         'permanent_address' => ['string', 'nullable'],
         'city_id' => ['integer', 'nullable'],

diff --git a/src/Http/Controllers/RegisterController.php b/src/Http/Controllers/RegisterController.php
@@ -32,7 +32,7 @@ public function __invoke(RegistrationRequest $request): JsonResponse
 
             $profile = Auth::profile()->create($user->getKey(), $request->except($userFields));
 
-            //            event(new Registered($user));
+            event(new Registered($user));
 
             return $this->created('Registration Successful.');
 

diff --git a/src/Http/Requests/RegistrationRequest.php b/src/Http/Requests/RegistrationRequest.php
@@ -23,50 +23,50 @@ public function authorize(): bool
     public function rules(): array
     {
         $rules = config('fintech.auth.register_rules', [
-            //user
-            'name' => ['required', 'string', 'min:2', 'max:255'],
-            'mobile' => ['required', 'string', 'min:10'],
-            'email' => ['required', 'string', 'email:rfc,dns', 'min:2', 'max:255'],
-            'pin' => ['required', 'string', 'min:4', 'max:16'],
-            'app_version' => ['nullable', 'string'],
-            'fcm_token' => ['nullable', 'string'],
-            'language' => ['nullable', 'string'],
-            'currency' => ['nullable', 'string'],
+                //user
+                'name' => ['required', 'string', 'min:2', 'max:255'],
+                'mobile' => ['required', 'string', 'min:10'],
+                'email' => ['required', 'string', 'email:rfc,dns', 'min:2', 'max:255'],
+                'pin' => ['required', 'string', 'min:4', 'max:16'],
+                'app_version' => ['nullable', 'string'],
+                'fcm_token' => ['nullable', 'string'],
+                'language' => ['nullable', 'string'],
+                'currency' => ['nullable', 'string'],
 
-            //profile
-            'father_name' => ['string', 'nullable'],
-            'mother_name' => ['string', 'nullable'],
-            'gender' => ['string', 'nullable'],
-            'marital_status' => ['string', 'nullable'],
-            'occupation' => ['string', 'nullable'],
-            'source_of_income' => ['string', 'nullable'],
-            'id_type' => ['string', 'nullable'],
-            'id_no' => ['string', 'nullable'],
-            'id_issue_country' => ['string', 'nullable'],
-            'id_expired_at' => ['string', 'nullable'],
-            'id_issue_at' => ['string', 'nullable'],
-            'photo' => ['string', 'nullable'],
-            'documents' => ['array', 'required', 'min:1'],
-            'documents.*.type' => ['string', 'required'],
-            'documents.*.front' => ['string', 'required'],
-            'documents.*.back' => ['string', 'nullable'],
-            'proof_of_address' => ['array', 'required', 'min:1'],
-            'proof_of_address.*.type' => ['string', 'required'],
-            'proof_of_address.*.front' => ['string', 'required'],
-            'proof_of_address.*.back' => ['string', 'nullable'],
-            'date_of_birth' => ['date', 'nullable'],
-            'permanent_address' => ['string', 'nullable'],
-            'city_id' => ['integer', 'nullable'],
-            'state_id' => ['integer', 'nullable'],
-            'country_id' => ['integer', 'nullable'],
-            'post_code' => ['string', 'nullable'],
-            'present_address' => ['string', 'nullable'],
-            'present_city_id' => ['integer', 'nullable'],
-            'present_state_id' => ['integer', 'nullable'],
-            'present_country_id' => ['integer', 'nullable'],
-            'present_post_code' => ['string', 'nullable'],
-            'nationality' => ['string', 'nullable'],
-        ]);
+                //profile
+                'father_name' => ['string', 'nullable'],
+                'mother_name' => ['string', 'nullable'],
+                'gender' => ['string', 'nullable'],
+                'marital_status' => ['string', 'nullable'],
+                'occupation' => ['string', 'nullable'],
+                'source_of_income' => ['string', 'nullable'],
+                'id_type' => ['string', 'nullable'],
+                'id_no' => ['string', 'nullable'],
+                'id_issue_country' => ['string', 'nullable'],
+                'id_expired_at' => ['string', 'nullable'],
+                'id_issue_at' => ['string', 'nullable'],
+                'photo' => ['string', 'nullable'],
+                'documents' => ['array', 'required', 'min:1'],
+                'documents.*.type' => ['string', 'required'],
+                'documents.*.back' => ['string', 'required_without:documents.*.front'],
+                'documents.*.front' => ['string', 'required_without:documents.*.back'],
+                'proof_of_address' => ['array', 'required', 'min:1'],
+                'proof_of_address.*.type' => ['string', 'required'],
+                'proof_of_address.*.front' => ['string', 'required_without:proof_of_address.*.back'],
+                'proof_of_address.*.back' => ['string', 'required_without:proof_of_address.*.front'],
+                'date_of_birth' => ['date', 'nullable'],
+                'permanent_address' => ['string', 'nullable'],
+                'city_id' => ['integer', 'nullable'],
+                'state_id' => ['integer', 'nullable'],
+                'country_id' => ['integer', 'nullable'],
+                'post_code' => ['string', 'nullable'],
+                'present_address' => ['string', 'nullable'],
+                'present_city_id' => ['integer', 'nullable'],
+                'present_state_id' => ['integer', 'nullable'],
+                'present_country_id' => ['integer', 'nullable'],
+                'present_post_code' => ['string', 'nullable'],
+                'nationality' => ['string', 'nullable'],
+            ]);
 
         $rules[config('fintech.auth.auth_field', 'login_id')] = config('fintech.auth.auth_field_rules', ['required', 'string', 'min:6', 'max:255']);
 

diff --git a/src/Http/Requests/StoreUserRequest.php b/src/Http/Requests/StoreUserRequest.php
@@ -22,7 +22,7 @@ public function authorize(): bool
      */
     public function rules(): array
     {
-        $rules = [
+        $rules = config('fintech.auth.register_rules', [
             //user
             'name' => ['required', 'string', 'min:2', 'max:255'],
             'mobile' => ['required', 'string', 'min:10'],
@@ -48,12 +48,12 @@ public function rules(): array
             'photo' => ['string', 'nullable'],
             'documents' => ['array', 'required', 'min:1'],
             'documents.*.type' => ['string', 'required'],
-            'documents.*.front' => ['string', 'required'],
-            'documents.*.back' => ['string', 'nullable'],
+            'documents.*.back' => ['string', 'required_without:documents.*.front'],
+            'documents.*.front' => ['string', 'required_without:documents.*.back'],
             'proof_of_address' => ['array', 'required', 'min:1'],
             'proof_of_address.*.type' => ['string', 'required'],
-            'proof_of_address.*.front' => ['string', 'required'],
-            'proof_of_address.*.back' => ['string', 'nullable'],
+            'proof_of_address.*.front' => ['string', 'required_without:proof_of_address.*.back'],
+            'proof_of_address.*.back' => ['string', 'required_without:proof_of_address.*.front'],
             'date_of_birth' => ['date', 'nullable'],
             'permanent_address' => ['string', 'nullable'],
             'city_id' => ['integer', 'nullable'],
@@ -66,7 +66,7 @@ public function rules(): array
             'present_country_id' => ['integer', 'nullable'],
             'present_post_code' => ['string', 'nullable'],
             'nationality' => ['string', 'nullable'],
-        ];
+        ]);
 
         $rules[config('fintech.auth.auth_field', 'login_id')] = config('fintech.auth.auth_field_rules', ['required', 'string', 'min:6', 'max:255']);
 

diff --git a/src/Http/Requests/UpdateUserRequest.php b/src/Http/Requests/UpdateUserRequest.php
@@ -22,12 +22,12 @@ public function authorize(): bool
      */
     public function rules(): array
     {
-        $rules = [
+        $rules = config('fintech.auth.register_rules', [
             //user
             'name' => ['required', 'string', 'min:2', 'max:255'],
             'mobile' => ['required', 'string', 'min:10'],
             'email' => ['required', 'string', 'email:rfc,dns', 'min:2', 'max:255'],
-            'pin' => ['nullable', 'string', 'min:4', 'max:16'],
+            'pin' => ['required', 'string', 'min:4', 'max:16'],
             'app_version' => ['nullable', 'string'],
             'fcm_token' => ['nullable', 'string'],
             'language' => ['nullable', 'string'],
@@ -46,14 +46,14 @@ public function rules(): array
             'id_expired_at' => ['string', 'nullable'],
             'id_issue_at' => ['string', 'nullable'],
             'photo' => ['string', 'nullable'],
-            'documents' => ['array', 'nullable', 'min:1'],
+            'documents' => ['array', 'required', 'min:1'],
             'documents.*.type' => ['string', 'required'],
-            'documents.*.front' => ['string', 'required'],
-            'documents.*.back' => ['string', 'nullable'],
-            'proof_of_address' => ['array', 'nullable', 'min:1'],
+            'documents.*.back' => ['string', 'required_without:documents.*.front'],
+            'documents.*.front' => ['string', 'required_without:documents.*.back'],
+            'proof_of_address' => ['array', 'required', 'min:1'],
             'proof_of_address.*.type' => ['string', 'required'],
-            'proof_of_address.*.front' => ['string', 'required'],
-            'proof_of_address.*.back' => ['string', 'nullable'],
+            'proof_of_address.*.back' => ['string', 'required_without:proof_of_address.*.front'],
+            'proof_of_address.*.front' => ['string', 'required_without:proof_of_address.*.back'],
             'date_of_birth' => ['date', 'nullable'],
             'permanent_address' => ['string', 'nullable'],
             'city_id' => ['integer', 'nullable'],
@@ -66,13 +66,16 @@ public function rules(): array
             'present_country_id' => ['integer', 'nullable'],
             'present_post_code' => ['string', 'nullable'],
             'nationality' => ['string', 'nullable'],
-            'roles' => ['array', 'required'],
-            'roles.*' => ['integer', 'required'],
-        ];
+        ]);
 
         $rules[config('fintech.auth.auth_field', 'login_id')] = config('fintech.auth.auth_field_rules', ['required', 'string', 'min:6', 'max:255']);
 
         $rules[config('fintech.auth.password_field', 'password')] = ['nullable', ...config('fintech.auth.password_field_rules', ['string', Password::default()])];
+
+        $rules['roles'] = ['array', 'required'];
+
+        $rules['roles.*'] = ['integer', 'required'];
+
         return $rules;
     }