NumberFilter::filter() - Code Metrics - Inspection of "Apply php 7 cs-fixer rules" - sonata-project/SonataDoctrineMongoDBAdminBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 3.x-dev-kit ( 6ed754...67996a )

by Grégoire

created 2019-01-10 21:48 UTC

NumberFilter::filter() A

↳ Parent: NumberFilter

Complexity

Conditions	6
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	6
nc	3
nop	4
dl	0
loc	17
rs	9.0777
c	0
b	0
f	0

<?php

declare(strict_types=1);

/*
 * This file is part of the Sonata Project package.
 *
 * (c) Thomas Rabaix <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sonata\DoctrineMongoDBAdminBundle\Filter;

use Sonata\AdminBundle\Datagrid\ProxyQueryInterface;
use Sonata\AdminBundle\Form\Type\Filter\NumberType;

class NumberFilter extends Filter
{
    /**
     * @param ProxyQueryInterface $queryBuilder
     * @param string              $alias
     * @param string              $field
     * @param string              $data
     */
    public function filter(ProxyQueryInterface $queryBuilder, $alias, $field, $data)
    {
        if (!$data || !\is_array($data) || !array_key_exists('value', $data) || !is_numeric($data['value'])) {
            return;
        }

        $type = $data['type'] ?? false;

        $operator = $this->getOperator($type);

        if (!$operator) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $operator = 'equals';
        }

        $queryBuilder->field($field)->$operator((float) $data['value']);
        $this->active = true;
    }

    /**
     * @return array
     */
    public function getDefaultOptions()
    {
        return [];
    }

    public function getRenderSettings()
    {
        return [NumberType::class, [
            'field_type' => $this->getFieldType(),
            'field_options' => $this->getFieldOptions(),
            'label' => $this->getLabel(),
        ]];
    }

    /**
     * @param $type
     *
     * @return bool
     */
    private function getOperator($type)
    {
        $choices = [
            NumberType::TYPE_EQUAL => 'equals',
            NumberType::TYPE_GREATER_EQUAL => 'gte',
            NumberType::TYPE_GREATER_THAN => 'gt',
            NumberType::TYPE_LESS_EQUAL => 'lte',
            NumberType::TYPE_LESS_THAN => 'lt',
        ];

        return $choices[$type] ?? false;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* This file is part of the Sonata Project package.
7			*
8			* (c) Thomas Rabaix <[email protected]>
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace Sonata\DoctrineMongoDBAdminBundle\Filter;
15
16			use Sonata\AdminBundle\Datagrid\ProxyQueryInterface;
17			use Sonata\AdminBundle\Form\Type\Filter\NumberType;
18
19			class NumberFilter extends Filter
20			{
21			/**
22			* @param ProxyQueryInterface $queryBuilder
23			* @param string $alias
24			* @param string $field
25			* @param string $data
26			*/
27			public function filter(ProxyQueryInterface $queryBuilder, $alias, $field, $data)
28			{
29			if (!$data \|\| !\is_array($data) \|\| !array_key_exists('value', $data) \|\| !is_numeric($data['value'])) {
30			return;
31			}
32
33			$type = $data['type'] ?? false;
34
35			$operator = $this->getOperator($type);
36
37			if (!$operator) {
			0 ignored issues – show Bug Best Practice introduced 2015-12-14 09:14 UTC by Report Bug Copy Issue Report The expression `$operator` of type `string\|false` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== false` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
38			$operator = 'equals';
39			}
40
41			$queryBuilder->field($field)->$operator((float) $data['value']);
42			$this->active = true;
43			}
44
45			/**
46			* @return array
47			*/
48			public function getDefaultOptions()
49			{
50			return [];
51			}
52
53			public function getRenderSettings()
54			{
55			return [NumberType::class, [
56			'field_type' => $this->getFieldType(),
57			'field_options' => $this->getFieldOptions(),
58			'label' => $this->getLabel(),
59			]];
60			}
61
62			/**
63			* @param $type
64			*
65			* @return bool
66			*/
67			private function getOperator($type)
68			{
69			$choices = [
70			NumberType::TYPE_EQUAL => 'equals',
71			NumberType::TYPE_GREATER_EQUAL => 'gte',
72			NumberType::TYPE_GREATER_THAN => 'gt',
73			NumberType::TYPE_LESS_EQUAL => 'lte',
74			NumberType::TYPE_LESS_THAN => 'lt',
75			];
76
77			return $choices[$type] ?? false;
78			}
79			}
80

sonata-project / SonataDoctrineMongoDBAdminBundle

Push — 3.x-dev-kit ( 6ed754...67996a )

NumberFilter::filter() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like