Mbright\Validation\Rule\Sanitize\Hex

Complexity

Total Complexity

6

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

78.56%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	__invoke()	0	21	5

<?php

namespace Mbright\Validation\Rule\Sanitize;

class Hex
{
    /** @var int|null */
    protected $max;

    /**
     * @param int|null $max
     */
    public function __construct(int $max = null)
    {
        $this->max = $max;
    }

    /**
     * @param $subject
     * @param $field
     *
     * @return bool
     */
    public function __invoke($subject, string $field): bool
    {
        $value = $subject->$field;

        if (!is_scalar($value)) {
            return false;
        }

        $value = preg_replace('/[^0-9a-f]/i', '', $value);
        if ($value === '') {
            return false;
        }

        if ($this->max && strlen($value) > $this->max) {

The expression $this->max of type integer|null is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

            $value = substr($value, 0, $this->max);
        }

        $subject->$field = $value;

        return true;
    }
}