ValidLandlineNumber::passes() - Code Metrics - Inspection of "Merge pull request #108 from vahidkaargar/1.x" - milwad-dev/laravel-validate - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — 1.x ( ced68d...fcb885 )

by Milwad

created 2025-04-20 15:10 UTC

ValidLandlineNumber::passes() A

↳ Parent: ValidLandlineNumber

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	4
c	1
b	0
f	0
nc	2
nop	2
dl	0
loc	9
rs	10

<?php

namespace Milwad\LaravelValidate\Rules;

use Illuminate\Contracts\Validation\Rule;
use Milwad\LaravelValidate\Utils\CountryLandlineCallback;

class ValidLandlineNumber implements Rule
{
    public function __construct(protected ?string $code = null) {}

    /**
     * Check phone number is valid.
     */
    public function passes($attribute, $value): bool
    {
        if (is_string($this->code)) {
            $passes = CountryLandlineCallback::callLandlineValidator($this->code, $value);

            return collect($passes)->some(fn ($passe) => $passe);
        }

        return preg_match('/^(?:\+|00)(?:[1-9]\d{0,2})(?:[ .\-]?\(?\d+\)?)+$/', $value);

    }

    /**
     * Get the validation error message.
     */
    public function message(): string
    {
        return __('validate.landline-number');
    }
}


1			<?php
2
3			namespace Milwad\LaravelValidate\Rules;
4
5			use Illuminate\Contracts\Validation\Rule;
6			use Milwad\LaravelValidate\Utils\CountryLandlineCallback;
7
8			class ValidLandlineNumber implements Rule
9			{
10			public function __construct(protected ?string $code = null) {}
11
12			/**
13			* Check phone number is valid.
14			*/
15			public function passes($attribute, $value): bool
16			{
17			if (is_string($this->code)) {
18			$passes = CountryLandlineCallback::callLandlineValidator($this->code, $value);
19
20			return collect($passes)->some(fn ($passe) => $passe);
21			}
22
23			return preg_match('/^(?:\+\|00)(?:[1-9]\d{0,2})(?:[ .\-]?\(?\d+\)?)+$/', $value);
			0 ignored issues – show Bug Best Practice introduced 2025-04-17 13:43 UTC by Report Bug Copy Issue Report The expression `return preg_match('/^(?:...\(?\d+\)?)+$/', $value)` returns the type `integer` which is incompatible with the type-hinted return `boolean`. Loading history...
24			}
25
26			/**
27			* Get the validation error message.
28			*/
29			public function message(): string
30			{
31			return __('validate.landline-number');
32			}
33			}
34

milwad-dev / laravel-validate

Push — 1.x ( ced68d...fcb885 )

ValidLandlineNumber::passes() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like