WithFilters::isRequestedFilter() - Code Metrics - Inspection of "remove deprecated methods" - zingimmick/laravel-query-builder - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 0a943e...cf7103 )

by Zing

created 2020-04-19 09:31 UTC

WithFilters::isRequestedFilter() A

↳ Parent: WithFilters

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
c	1
b	0
f	0
nc	1
nop	1
dl	0
loc	3
ccs	2
cts	2
cp	1
crap	1
rs	10

<?php

namespace Zing\QueryBuilder\Concerns;

use Zing\QueryBuilder\Filter;

trait WithFilters
{
    protected $filters;

    public function enableFilters($filters)
    {
        $filters = is_array($filters) ? $filters : func_get_args();
        $this->filters = $this->formatFilters($filters);
        $this->applyFilters();

        return $this;
    }

    protected function formatFilters($filters)
    {
        return collect($filters)->map(function ($filter) {
            if ($filter instanceof Filter) {
                return $filter;
            }

            return Filter::exact($filter);
        });
    }

    protected function applyFilters()
    {
        $this->filters->each(function (Filter $filter) {
            if ($this->isRequestedFilter($filter)) {
                $filter->filter($this, $this->getFilterValue($filter));

                return;
            }
            if ($filter->hasDefault()) {
                $filter->filter($this, $filter->getDefault());

                return;
            }
        });
    }

    protected function isRequestedFilter(Filter $filter)
    {
        return $this->request->has($filter->getProperty());
    }

    protected function getFilterValue(Filter $filter)
    {
        return $this->request->input($filter->getProperty());
    }
}


1		<?php
2
3		namespace Zing\QueryBuilder\Concerns;
4
5		use Zing\QueryBuilder\Filter;
6
7		trait WithFilters
8		{
9		protected $filters;
10
11	15	public function enableFilters($filters)
12		{
13	15	$filters = is_array($filters) ? $filters : func_get_args();
14	15	$this->filters = $this->formatFilters($filters);
15	15	$this->applyFilters();
16
17	15	return $this;
18		}
19
20	15	protected function formatFilters($filters)
21		{
22		return collect($filters)->map(function ($filter) {
23	15	if ($filter instanceof Filter) {
24	14	return $filter;
25		}
26
27	1	return Filter::exact($filter);
28	15	});
29		}
30
31	15	protected function applyFilters()
32		{
33		$this->filters->each(function (Filter $filter) {
34	15	if ($this->isRequestedFilter($filter)) {
35	14	$filter->filter($this, $this->getFilterValue($filter));
36
37	14	return;
38		}
39	1	if ($filter->hasDefault()) {
40	1	$filter->filter($this, $filter->getDefault());
41
42	1	return;
43		}
44	15	});
45	15	}
46
47	15	protected function isRequestedFilter(Filter $filter)
48		{
49	15	return $this->request->has($filter->getProperty());
50		}
51
52	14	protected function getFilterValue(Filter $filter)
53		{
54	14	return $this->request->input($filter->getProperty());
55		}
56		}
57

zingimmick / laravel-query-builder

Push — master ( 0a943e...cf7103 )

WithFilters::isRequestedFilter() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like