Rinvex\Support\Traits\Escaper

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	escape()	0	16	4
A	withValidator()	0	5	1

<?php

declare(strict_types=1);

namespace Rinvex\Support\Traits;

use Illuminate\Support\Arr;

trait Escaper
{
    /**
     * Escape all values.
     *
     * @param array $data
     *
     * @return array
     */
    protected function escape(array $data): array
    {
        $arrayDot = array_filter(Arr::dot($data));

$data is of type array, but the function expects a object<Illuminate\Support\iterable>.

        foreach ($arrayDot as $key => $value) {
            if (is_string($value)) {
                $arrayDot[$key] = e($value);
            }
        }

        foreach ($arrayDot as $key => $value) {
            Arr::set($data, $key, $value);
        }

        return $data;
    }

    /**
     * Configure the validator instance.
     *
     * @param \Illuminate\Validation\Validator $validator
     *
     * @return void
     */
    public function withValidator($validator): void

The parameter $validator is not used and could be removed.

    {
        // Sanitize input data before submission
        $this->replace($this->escape($this->all()));

It seems like all() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like replace() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}