NwLaravel\Repositories\Criterias\Filters\FilterBetween

Complexity

Total Complexity

7

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
B	filter()	0	23	6

<?php

namespace NwLaravel\Repositories\Criterias\Filters;

class FilterBetween implements FilterInterface
{
    /**
     * @var string
     */
    protected $table;

    /**
     * @var array
     */
    protected $columns;

    /**
     * @var array
     */
    protected $dates;

    /**
     * Construct
     *
     * @param string $table
     * @param array  $columns
     * @param array  $dates
     */
    public function __construct($table, $columns, $dates)
    {
        $this->table = $table;
        $this->columns = $columns;
        $this->dates = $dates;
    }

    /**
     * Filter
     *
     * @param Query\Builder $query
     * @param int|string    $key
     * @param mixed         $value
     *
     * @return boolean
     */
    public function filter($query, $key, $value)
    {
        $table = $this->table;
        $column = $key;
        if (preg_match('/^([a-zA-Z0-9_\-]+)\.([a-zA-Z0-9_\-]+)/', $key, $matches)) {
            $table = $matches[1];
            $column = $matches[2];
        }

        if (preg_match('/^(.+)(_ini|_fim)$/', $column, $match) && in_array($match[1], $this->columns)) {
            $field = $match[1];
            $operator = ($match[2]=='_ini')? '>=' : '<=';
            if (in_array($field, $this->dates)) {
                $filterDate = new FilterDate($operator);
                return $filterDate->filter($query, $table.'.'.$field, $value);
            }
            
            $query = $query->where($table.'.'.$field, $operator, $value);

$query is not used, you could remove the assignment.

            return true;
        }

        return false;
    }
}