From cdac88613cd34737dda307b457ce9ab5ce9377e6 Mon Sep 17 00:00:00 2001 From: allanmcarvalho Date: Wed, 27 Sep 2023 19:56:07 +0000 Subject: [PATCH] Fix styling --- resources/lang/en/validation.php | 8 ++++---- resources/lang/pt_BR/validation.php | 9 ++++----- src/Rules/pt_BR/Document.php | 8 +++++--- src/Rules/pt_BR/Phone.php | 17 +++++++++-------- 4 files changed, 22 insertions(+), 20 deletions(-) diff --git a/resources/lang/en/validation.php b/resources/lang/en/validation.php index 9520746..39750f3 100644 --- a/resources/lang/en/validation.php +++ b/resources/lang/en/validation.php @@ -16,16 +16,16 @@ 'document' => [ 'cnpj' => [ 'invalid' => 'The :attribute isn\'t a valid CNPJ.', - 'size' => 'The :attribute must have 14 digits to be a valid CNPJ.' + 'size' => 'The :attribute must have 14 digits to be a valid CNPJ.', ], 'cpf' => [ 'invalid' => 'The :attribute isn\'t a valid CPF.', - 'size' => 'The :attribute must have 11 digits to be a valid CPF.' + 'size' => 'The :attribute must have 11 digits to be a valid CPF.', ], 'generic' => [ 'invalid' => 'The :attribute isn\'t a valid CNPJ or CPF.', - 'size' => 'The :attribute must have 11 or 14 digits to be a valid CNPJ or CPF.' - ] + 'size' => 'The :attribute must have 11 or 14 digits to be a valid CNPJ or CPF.', + ], ], ]; diff --git a/resources/lang/pt_BR/validation.php b/resources/lang/pt_BR/validation.php index 373fff1..050f597 100644 --- a/resources/lang/pt_BR/validation.php +++ b/resources/lang/pt_BR/validation.php @@ -12,16 +12,16 @@ 'document' => [ 'cnpj' => [ 'invalid' => 'O campo :attribute não é um CNPJ válido.', - 'size' => 'O campo :attribute deve ter 14 dígitos para ser um CNPJ válido.' + 'size' => 'O campo :attribute deve ter 14 dígitos para ser um CNPJ válido.', ], 'cpf' => [ 'invalid' => 'O campo :attribute não é um CPF válido.', - 'size' => 'O campo :attribute deve ter 11 dígitos para ser um CPF válido.' + 'size' => 'O campo :attribute deve ter 11 dígitos para ser um CPF válido.', ], 'generic' => [ 'invalid' => 'O campo :attribute não é um CNPJ ou CPF válido.', - 'size' => 'O campo :attribute deve ter 11 dígitos para CPF ou 14 dígitos para CNPJ.' - ] + 'size' => 'O campo :attribute deve ter 11 dígitos para CPF ou 14 dígitos para CNPJ.', + ], ], 'phone' => [ 'generic' => 'O campo :attribute não é um telefone válido.', @@ -32,5 +32,4 @@ 'public_services' => 'O campo :attribute não é um telefone de serviços públicos válido.', ], - ]; diff --git a/src/Rules/pt_BR/Document.php b/src/Rules/pt_BR/Document.php index d3b666f..9a818c0 100644 --- a/src/Rules/pt_BR/Document.php +++ b/src/Rules/pt_BR/Document.php @@ -7,12 +7,11 @@ readonly class Document implements ValidationRule { - public function __construct( public bool $allowCNPJ = true, public bool $allowCPF = true, ) { - if ( ! $this->allowCNPJ && ! $this->allowCPF) { + if (! $this->allowCNPJ && ! $this->allowCPF) { throw new \InvalidArgumentException('You must allow at least one document type.'); } } @@ -39,8 +38,9 @@ public static function generic(): self */ public function validate(string $attribute, mixed $value, Closure $fail): void { - if (!is_string($value)) { + if (! is_string($value)) { $fail(__('validation.string')); + return; } $value = preg_replace('/[^0-9]/i', '', $value); @@ -86,6 +86,7 @@ protected function validateCnpj(string $value): bool if ($value[13] != ($rest < 2 ? 0 : 11 - $rest)) { return false; } + return true; } @@ -100,6 +101,7 @@ protected function validateCpf(string $value): bool return false; } } + return true; } } diff --git a/src/Rules/pt_BR/Phone.php b/src/Rules/pt_BR/Phone.php index b82c3c4..7065fec 100644 --- a/src/Rules/pt_BR/Phone.php +++ b/src/Rules/pt_BR/Phone.php @@ -7,7 +7,6 @@ readonly class Phone implements ValidationRule { - public function __construct( public bool $allowCellphone = true, public bool $allowLocalFare = true, @@ -15,7 +14,7 @@ public function __construct( public bool $allowPhone = true, public bool $allowPublicServices = true, ) { - if ( ! $allowCellphone && ! $allowLocalFare && ! $allowNonRegional && ! $allowPhone && ! $allowPublicServices) { + if (! $allowCellphone && ! $allowLocalFare && ! $allowNonRegional && ! $allowPhone && ! $allowPublicServices) { throw new \InvalidArgumentException('At least one of the options must be true.'); } } @@ -57,8 +56,9 @@ public static function publicServices(): self */ public function validate(string $attribute, mixed $value, Closure $fail): void { - if (!is_string($value)) { + if (! is_string($value)) { $fail(__('validation.string')); + return; } $value = preg_replace('/[^0-9]/i', '', $value); @@ -119,21 +119,22 @@ protected function validatePublicServices(string $value): string|true protected function validateGeneric(string $value): string|true { - if (preg_match('/^[1-9][0-9]9$/', substr($value, 0, 3)) === 1) { + if (preg_match('/^[1-9][0-9]9$/', substr($value, 0, 3)) === 1) { return $this->validateCellphone($value); } - if (preg_match('/^400$/', substr($value, 0, 3)) === 1) { + if (preg_match('/^400$/', substr($value, 0, 3)) === 1) { return $this->validateLocalFare($value); } - if (preg_match('/^0[3589]00$/', substr($value, 0, 4)) === 1) { + if (preg_match('/^0[3589]00$/', substr($value, 0, 4)) === 1) { return $this->validateNonRegional($value); } - if (preg_match('/^[1-9][0-9][1-5][0-9]$/', substr($value, 0, 4)) === 1) { + if (preg_match('/^[1-9][0-9][1-5][0-9]$/', substr($value, 0, 4)) === 1) { return $this->validatePhone($value); } - if (preg_match('/^1[0-9]{2}$/', substr($value, 0, 3)) === 1) { + if (preg_match('/^1[0-9]{2}$/', substr($value, 0, 3)) === 1) { return $this->validatePublicServices($value); } + return __('laravuewind::validation.phone.generic'); } }