feat: implement PhoneNumber validation rule across multiple request classes to ensure consistent phone number formatting
This commit is contained in:
parent
c23e791c5e
commit
0b4418862d
@ -3,6 +3,7 @@
|
|||||||
namespace App\Http\Requests\Admin\Account;
|
namespace App\Http\Requests\Admin\Account;
|
||||||
|
|
||||||
use App\Enums\Gender;
|
use App\Enums\Gender;
|
||||||
|
use App\Rules\PhoneNumber;
|
||||||
use Illuminate\Foundation\Http\FormRequest;
|
use Illuminate\Foundation\Http\FormRequest;
|
||||||
use Illuminate\Validation\Rule;
|
use Illuminate\Validation\Rule;
|
||||||
|
|
||||||
@ -22,7 +23,7 @@ public function rules(): array
|
|||||||
'email' => ['required', 'email', 'max:100', Rule::unique('users', 'email')->ignore($this->user()?->id)],
|
'email' => ['required', 'email', 'max:100', Rule::unique('users', 'email')->ignore($this->user()?->id)],
|
||||||
'username' => ['required', 'string', 'max:20', 'alpha_dash', Rule::unique('users', 'username')->ignore($this->user()?->id)],
|
'username' => ['required', 'string', 'max:20', 'alpha_dash', Rule::unique('users', 'username')->ignore($this->user()?->id)],
|
||||||
'full_name' => ['required', 'string', 'max:200'],
|
'full_name' => ['required', 'string', 'max:200'],
|
||||||
'phone_number' => ['nullable', 'string', 'max:20', 'regex:/^08\d{8,11}$/'],
|
'phone_number' => ['nullable', 'string', 'max:20', new PhoneNumber],
|
||||||
'gender' => ['nullable', Rule::enum(Gender::class)],
|
'gender' => ['nullable', Rule::enum(Gender::class)],
|
||||||
'birth_date' => ['nullable', 'date', 'before:today'],
|
'birth_date' => ['nullable', 'date', 'before:today'],
|
||||||
'address' => ['nullable', 'string'],
|
'address' => ['nullable', 'string'],
|
||||||
|
|||||||
@ -6,6 +6,7 @@
|
|||||||
use App\Enums\Gender;
|
use App\Enums\Gender;
|
||||||
use App\Enums\Permission;
|
use App\Enums\Permission;
|
||||||
use App\Enums\Role;
|
use App\Enums\Role;
|
||||||
|
use App\Rules\PhoneNumber;
|
||||||
use Illuminate\Foundation\Http\FormRequest;
|
use Illuminate\Foundation\Http\FormRequest;
|
||||||
use Illuminate\Validation\Rule;
|
use Illuminate\Validation\Rule;
|
||||||
|
|
||||||
@ -31,7 +32,7 @@ public function rules(): array
|
|||||||
'email' => ['required', 'email', 'max:100', Rule::unique('users', 'email')->ignore($this->route('user')?->id)],
|
'email' => ['required', 'email', 'max:100', Rule::unique('users', 'email')->ignore($this->route('user')?->id)],
|
||||||
'username' => ['required', 'string', 'max:20', 'alpha_dash', Rule::unique('users', 'username')->ignore($this->route('user')?->id)],
|
'username' => ['required', 'string', 'max:20', 'alpha_dash', Rule::unique('users', 'username')->ignore($this->route('user')?->id)],
|
||||||
'full_name' => ['required', 'string', 'max:200'],
|
'full_name' => ['required', 'string', 'max:200'],
|
||||||
'phone_number' => ['nullable', 'string', 'max:20', 'regex:/^08\d{8,11}$/'],
|
'phone_number' => ['nullable', 'string', 'max:20', new PhoneNumber],
|
||||||
'gender' => ['nullable', Rule::enum(Gender::class)],
|
'gender' => ['nullable', Rule::enum(Gender::class)],
|
||||||
'birth_date' => ['nullable', 'date', 'before:today'],
|
'birth_date' => ['nullable', 'date', 'before:today'],
|
||||||
'address' => ['nullable', 'string'],
|
'address' => ['nullable', 'string'],
|
||||||
|
|||||||
@ -3,6 +3,7 @@
|
|||||||
namespace App\Http\Requests\Admin\Master;
|
namespace App\Http\Requests\Admin\Master;
|
||||||
|
|
||||||
use App\Enums\Permission;
|
use App\Enums\Permission;
|
||||||
|
use App\Rules\PhoneNumber;
|
||||||
use Illuminate\Foundation\Http\FormRequest;
|
use Illuminate\Foundation\Http\FormRequest;
|
||||||
|
|
||||||
class CustomerRequest extends FormRequest
|
class CustomerRequest extends FormRequest
|
||||||
@ -23,7 +24,7 @@ public function rules(): array
|
|||||||
{
|
{
|
||||||
return [
|
return [
|
||||||
'name' => ['required', 'string', 'max:200'],
|
'name' => ['required', 'string', 'max:200'],
|
||||||
'phone_number' => ['nullable', 'string', 'max:20'],
|
'phone_number' => ['nullable', 'string', 'max:20', new PhoneNumber],
|
||||||
'address' => ['nullable', 'string'],
|
'address' => ['nullable', 'string'],
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
|
|||||||
@ -3,6 +3,7 @@
|
|||||||
namespace App\Http\Requests\Admin\Master;
|
namespace App\Http\Requests\Admin\Master;
|
||||||
|
|
||||||
use App\Enums\Permission;
|
use App\Enums\Permission;
|
||||||
|
use App\Rules\PhoneNumber;
|
||||||
use Illuminate\Foundation\Http\FormRequest;
|
use Illuminate\Foundation\Http\FormRequest;
|
||||||
|
|
||||||
class SupplierRequest extends FormRequest
|
class SupplierRequest extends FormRequest
|
||||||
@ -23,7 +24,7 @@ public function rules(): array
|
|||||||
{
|
{
|
||||||
return [
|
return [
|
||||||
'name' => ['required', 'string', 'max:200'],
|
'name' => ['required', 'string', 'max:200'],
|
||||||
'phone_number' => ['nullable', 'string', 'max:20'],
|
'phone_number' => ['nullable', 'string', 'max:20', new PhoneNumber],
|
||||||
'address' => ['nullable', 'string'],
|
'address' => ['nullable', 'string'],
|
||||||
];
|
];
|
||||||
}
|
}
|
||||||
|
|||||||
@ -3,6 +3,7 @@
|
|||||||
namespace App\Http\Requests\Admin\System\Setting;
|
namespace App\Http\Requests\Admin\System\Setting;
|
||||||
|
|
||||||
use App\Enums\Permission;
|
use App\Enums\Permission;
|
||||||
|
use App\Rules\PhoneNumber;
|
||||||
use Illuminate\Foundation\Http\FormRequest;
|
use Illuminate\Foundation\Http\FormRequest;
|
||||||
|
|
||||||
class SystemRequest extends FormRequest
|
class SystemRequest extends FormRequest
|
||||||
@ -21,7 +22,7 @@ public function rules(): array
|
|||||||
'app_name' => ['required', 'string', 'max:100'],
|
'app_name' => ['required', 'string', 'max:100'],
|
||||||
'about_app' => ['required', 'string'],
|
'about_app' => ['required', 'string'],
|
||||||
'email' => ['required', 'email', 'max:100'],
|
'email' => ['required', 'email', 'max:100'],
|
||||||
'phone' => ['required', 'string', 'max:20'],
|
'phone' => ['required', 'string', 'max:20', new PhoneNumber],
|
||||||
'address' => ['required', 'string'],
|
'address' => ['required', 'string'],
|
||||||
'logo' => ['required', 'image', 'max:2048'],
|
'logo' => ['required', 'image', 'max:2048'],
|
||||||
'login_cover' => ['required', 'image', 'max:5120'],
|
'login_cover' => ['required', 'image', 'max:5120'],
|
||||||
|
|||||||
16
app/Rules/PhoneNumber.php
Normal file
16
app/Rules/PhoneNumber.php
Normal file
@ -0,0 +1,16 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
namespace App\Rules;
|
||||||
|
|
||||||
|
use Closure;
|
||||||
|
use Illuminate\Contracts\Validation\ValidationRule;
|
||||||
|
|
||||||
|
class PhoneNumber implements ValidationRule
|
||||||
|
{
|
||||||
|
public function validate(string $attribute, mixed $value, Closure $fail): void
|
||||||
|
{
|
||||||
|
if (! preg_match('/^08\d{8,11}$/', $value)) {
|
||||||
|
$fail('Nomor telepon harus diawali dengan 08 dan terdiri dari 10-13 digit.');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
Loading…
Reference in New Issue
Block a user