Zing\QueryBuilder\Filters\DateFilter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	withPropertyConstraint()	0	22	4

<?php

declare(strict_types=1);

namespace Zing\QueryBuilder\Filters;

use DateTimeInterface;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Support\Facades\DB;

class DateFilter extends ExactFilter
{
    /**
     * @param mixed $value
     * @param \Illuminate\Database\Query\Expression|string $property
     */
    protected function withPropertyConstraint(Builder $query, $value, $property): Builder
    {
        if (is_array($value)) {
            $value = array_map(
                function ($value) {
                    if ($value instanceof DateTimeInterface) {
                        return $value->format('Y-m-d');
                    }

                    return $value;
                },
                $value
            );

            return $query->whereIn(DB::raw(sprintf('date(%s)', $property)), $value);

The expression return $query->whereIn(Illuminate\Support\Facades\DB::raw(sprintf('date(%s)', $property)), $value) could return the type Illuminate\Database\Query\Builder which is incompatible with the type-hinted return Illuminate\Database\Eloquent\Builder. Consider adding an additional type-check to rule them out.

        }

        if ($value instanceof DateTimeInterface) {
            $value = $value->format('Y-m-d');
        }

        return $query->whereDate($property, $value);

The expression return $query->whereDate($property, $value) could return the type Illuminate\Database\Query\Builder which is incompatible with the type-hinted return Illuminate\Database\Eloquent\Builder. Consider adding an additional type-check to rule them out.

    }
}