FilterBetween::filter() - Code Metrics - naturalweb/nwlaravel - Measure and Improve Code Quality continuously with Scrutinizer

FilterBetween::filter() B
last analyzed 2021-06-11 16:37 UTC

↳ Parent: FilterBetween

Complexity

Conditions	6
Paths	10

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	16
CRAP Score	6

Importance

Changes

Metric	Value
cc	6
nc	10
nop	3
dl	0
loc	23
ccs	16
cts	16
cp	1
crap	6
rs	8.9297
c	0
b	0
f	0

<?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);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            return true;
        }

        return false;
    }
}


1		<?php
2
3		namespace NwLaravel\Repositories\Criterias\Filters;
4
5		class FilterBetween implements FilterInterface
6		{
7		/**
8		* @var string
9		*/
10		protected $table;
11
12		/**
13		* @var array
14		*/
15		protected $columns;
16
17		/**
18		* @var array
19		*/
20		protected $dates;
21
22		/**
23		* Construct
24		*
25		* @param string $table
26		* @param array $columns
27		* @param array $dates
28		*/
29	15	public function __construct($table, $columns, $dates)
30		{
31	15	$this->table = $table;
32	15	$this->columns = $columns;
33	15	$this->dates = $dates;
34	15	}
35
36		/**
37		* Filter
38		*
39		* @param Query\Builder $query
40		* @param int\|string $key
41		* @param mixed $value
42		*
43		* @return boolean
44		*/
45	14	public function filter($query, $key, $value)
46		{
47	14	$table = $this->table;
48	14	$column = $key;
49	14	if (preg_match('/^([a-zA-Z0-9_\-]+)\.([a-zA-Z0-9_\-]+)/', $key, $matches)) {
50	11	$table = $matches[1];
51	11	$column = $matches[2];
52	11	}
53
54	14	if (preg_match('/^(.+)(_ini\|_fim)$/', $column, $match) && in_array($match[1], $this->columns)) {
55	5	$field = $match[1];
56	5	$operator = ($match[2]=='_ini')? '>=' : '<=';
57	5	if (in_array($field, $this->dates)) {
58	2	$filterDate = new FilterDate($operator);
59	2	return $filterDate->filter($query, $table.'.'.$field, $value);
60		}
61
62	4	$query = $query->where($table.'.'.$field, $operator, $value);
		0 ignored issues – show Unused Code introduced 2017-08-15 12:49 UTC by Report Bug Copy Issue Report `$query` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
63	4	return true;
64		}
65
66	10	return false;
67		}
68		}
69

naturalweb / nwlaravel

FilterBetween::filter() B last analyzed 2021-06-11 16:37 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

FilterBetween::filter() B
last analyzed 2021-06-11 16:37 UTC