Helldar\Roles\Support\Database\Search

Complexity

Total Complexity

16

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Test Coverage

Coverage

94.12%

Importance

Changes	3
Bugs	2	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	filterNum()	0	4	1
A	map()	0	5	1
A	getId()	0	5	2
A	filterSlugs()	0	4	1
A	toArray()	0	9	3
A	by()	0	19	5
A	isArray()	0	3	1
A	isArrayable()	0	3	2

<?php

namespace Helldar\Roles\Support\Database;

use Helldar\Roles\Models\BaseModel;
use Illuminate\Contracts\Support\Arrayable;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Support\Arr;
use Illuminate\Support\Collection;

class Search
{
    /**
     * @param  \Illuminate\Database\Eloquent\Builder|\Illuminate\Database\Eloquent\Relations\Relation  $builder
     * @param  \Helldar\Roles\Models\BaseModel|int|string  $values
     *
     * @return \Illuminate\Database\Eloquent\Builder|\Illuminate\Database\Eloquent\Relations\Relation
     */
    public function by($builder, $values)
    {
        $value = $this->map($values);

        $nums  = $this->filterNum($value);
        $slugs = $this->filterSlugs($value);

        if ($nums && ! $slugs) {

The expression $slugs of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

The expression $nums of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            return $builder->whereIn('id', $nums);

The expression return $builder->whereIn('id', $nums) also could return the type Illuminate\Database\Query\Builder which is incompatible with the documented return type Illuminate\Database\Eloquent\Builder|Illuminate\Database\Eloquent\Relations\Relation.

        }

        if (! $nums && $slugs) {

The expression $slugs of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

The expression $nums of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return $builder->whereIn('slug', $slugs);

The expression return $builder->whereIn('slug', $slugs) also could return the type Illuminate\Database\Query\Builder which is incompatible with the documented return type Illuminate\Database\Eloquent\Builder|Illuminate\Database\Eloquent\Relations\Relation.

        }

        return $builder->where(function (Builder $builder) use ($nums, $slugs) {
            $builder
                ->whereIn('id', $nums)
                ->orWhereIn('slug', $slugs);
        });
    }

    /**
     * @param  \Helldar\Roles\Models\BaseModel|int|string  $value
     *
     * @return int|string
     */
    public function getId($value)
    {
        return $value instanceof BaseModel
            ? $value->id
            : $value;
    }

    /**
     * @param  array|Arrayable|Collection|int|string  $values
     *
     * @return array
     */
    protected function map($values): array
    {
        return array_map(function ($value) {
            return $this->getId($value);
        }, $this->toArray($values));
    }

    protected function filterNum(array $values): array
    {
        return array_filter($values, function ($value) {
            return is_numeric($value);
        });
    }

    protected function filterSlugs(array $values): array
    {
        return array_filter($values, function ($value) {
            return ! is_numeric($value);
        });
    }

    protected function toArray($values): array
    {
        if ($this->isArrayable($values)) {
            $values = $values->toArray();
        } elseif (! $this->isArray($values)) {
            $values = Arr::wrap($values);
        }

        return Arr::flatten($values);
    }

    protected function isArray($values): bool
    {
        return is_array($values);
    }

    protected function isArrayable($values): bool
    {
        return $values instanceof Arrayable || $values instanceof Collection;
    }
}