Issues in GeoNear.php - All Issues - Inspection of "Merge branch '1.3.x'" - doctrine/mongodb-odm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 09b86b...ba0798 )

by Andreas

created 2019-04-15 11:30 UTC

Doctrine/ODM/MongoDB/Aggregation/Stage/GeoNear.php (2 issues)

Labels

Severity

Major 2

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

declare(strict_types=1);

namespace Doctrine\ODM\MongoDB\Aggregation\Stage;

use Doctrine\ODM\MongoDB\Aggregation\Builder;
use GeoJson\Geometry\Point;
use function is_array;

/**
 * Fluent interface for adding a $geoNear stage to an aggregation pipeline.
 */
class GeoNear extends Match
{
    /** @var string */
    private $distanceField;

    /** @var float */
    private $distanceMultiplier;

    /** @var string */
    private $includeLocs;

    /** @var float */
    private $maxDistance;

    /** @var float */
    private $minDistance;

    /** @var array */
    private $near;

    /** @var int */
    private $num;

    /** @var bool */
    private $spherical = false;

    /** @var bool */
    private $uniqueDocs;

    /**
     * @param float|array|Point $x
     * @param float             $y
     */
    public function __construct(Builder $builder, $x, $y = null)
    {
        parent::__construct($builder);

        $this->near($x, $y);
    }

    /**
     * {@inheritdoc}
     */
    public function getExpression() : array
    {
        $geoNear = [
            'near' => $this->near,
            'spherical' => $this->spherical,
            'distanceField' => $this->distanceField,
            'query' => $this->query->getQuery(),
            'distanceMultiplier' => $this->distanceMultiplier,
            'includeLocs' => $this->includeLocs,
            'maxDistance' => $this->maxDistance,
            'minDistance' => $this->minDistance,
            'num' => $this->num,
            'uniqueDocs' => $this->uniqueDocs,
        ];

        foreach (['distanceMultiplier', 'includeLocs', 'maxDistance', 'minDistance', 'num', 'uniqueDocs'] as $option) {
            if ($geoNear[$option]) {
                continue;
            }

            unset($geoNear[$option]);
        }

        return ['$geoNear' => $geoNear];
    }

    /**
     * The output field that contains the calculated distance. To specify a field within an embedded document, use dot notation.
     */
    public function distanceField(string $distanceField) : self
    {
        $this->distanceField = $distanceField;

        return $this;
    }

    /**
     * The factor to multiply all distances returned by the query.
     */
    public function distanceMultiplier(float $distanceMultiplier) : self
    {
        $this->distanceMultiplier = $distanceMultiplier;

        return $this;
    }

    /**
     * This specifies the output field that identifies the location used to calculate the distance.
     */
    public function includeLocs(string $includeLocs) : self
    {
        $this->includeLocs = $includeLocs;

        return $this;
    }

    /**
     * The maximum number of documents to return.
     */
    public function limit(int $limit) : self
    {
        return $this->num($limit);
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
    }

    /**
     * The maximum distance from the center point that the documents can be.
     */
    public function maxDistance(float $maxDistance) : self
    {
        $this->maxDistance = $maxDistance;

        return $this;
    }

    /**
     * The minimum distance from the center point that the documents can be.
     */
    public function minDistance(float $minDistance) : self
    {
        $this->minDistance = $minDistance;

        return $this;
    }

    /**
     * The point for which to find the closest documents.
     *
     * A GeoJSON point may be provided as the first and only argument for
     * 2dsphere queries. This single parameter may be a GeoJSON point object or
     * an array corresponding to the point's JSON representation. If GeoJSON is
     * used, the "spherical" option will default to true.
     *
     * @param float|array|Point $x
     * @param float             $y
     */
    public function near($x, $y = null) : self
    {
        if ($x instanceof Point) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            $x = $x->jsonSerialize();
        }

        $this->near      = is_array($x) ? $x : [$x, $y];
        $this->spherical = is_array($x) && isset($x['type']);

        return $this;
    }

    /**
     * The maximum number of documents to return.
     */
    public function num(int $num) : self
    {
        $this->num = $num;

        return $this;
    }

    /**
     * Required if using a 2dsphere index. Determines how MongoDB calculates the distance.
     */
    public function spherical(bool $spherical = true) : self
    {
        $this->spherical = $spherical;

        return $this;
    }

    /**
     * If this value is true, the query returns a matching document once, even if more than one of the document’s location fields match the query.
     */
    public function uniqueDocs(bool $uniqueDocs = true) : self
    {
        $this->uniqueDocs = $uniqueDocs;

        return $this;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Doctrine\ODM\MongoDB\Aggregation\Stage;
6
7		use Doctrine\ODM\MongoDB\Aggregation\Builder;
8		use GeoJson\Geometry\Point;
9		use function is_array;
10
11		/**
12		* Fluent interface for adding a $geoNear stage to an aggregation pipeline.
13		*/
14		class GeoNear extends Match
15		{
16		/** @var string */
17		private $distanceField;
18
19		/** @var float */
20		private $distanceMultiplier;
21
22		/** @var string */
23		private $includeLocs;
24
25		/** @var float */
26		private $maxDistance;
27
28		/** @var float */
29		private $minDistance;
30
31		/** @var array */
32		private $near;
33
34		/** @var int */
35		private $num;
36
37		/** @var bool */
38		private $spherical = false;
39
40		/** @var bool */
41		private $uniqueDocs;
42
43		/**
44		* @param float\|array\|Point $x
45		* @param float $y
46		*/
47	11	public function __construct(Builder $builder, $x, $y = null)
48		{
49	11	parent::__construct($builder);
50
51	11	$this->near($x, $y);
52	11	}
53
54		/**
55		* {@inheritdoc}
56		*/
57	11	public function getExpression() : array
58		{
59		$geoNear = [
60	11	'near' => $this->near,
61	11	'spherical' => $this->spherical,
62	11	'distanceField' => $this->distanceField,
63	11	'query' => $this->query->getQuery(),
64	11	'distanceMultiplier' => $this->distanceMultiplier,
65	11	'includeLocs' => $this->includeLocs,
66	11	'maxDistance' => $this->maxDistance,
67	11	'minDistance' => $this->minDistance,
68	11	'num' => $this->num,
69	11	'uniqueDocs' => $this->uniqueDocs,
70		];
71
72	11	foreach (['distanceMultiplier', 'includeLocs', 'maxDistance', 'minDistance', 'num', 'uniqueDocs'] as $option) {
73	11	if ($geoNear[$option]) {
74	8	continue;
75		}
76
77	11	unset($geoNear[$option]);
78		}
79
80	11	return ['$geoNear' => $geoNear];
81		}
82
83		/**
84		* The output field that contains the calculated distance. To specify a field within an embedded document, use dot notation.
85		*/
86	3	public function distanceField(string $distanceField) : self
87		{
88	3	$this->distanceField = $distanceField;
89
90	3	return $this;
91		}
92
93		/**
94		* The factor to multiply all distances returned by the query.
95		*/
96	1	public function distanceMultiplier(float $distanceMultiplier) : self
97		{
98	1	$this->distanceMultiplier = $distanceMultiplier;
99
100	1	return $this;
101		}
102
103		/**
104		* This specifies the output field that identifies the location used to calculate the distance.
105		*/
106	1	public function includeLocs(string $includeLocs) : self
107		{
108	1	$this->includeLocs = $includeLocs;
109
110	1	return $this;
111		}
112
113		/**
114		* The maximum number of documents to return.
115		*/
116	2	public function limit(int $limit) : self
117		{
118	2	return $this->num($limit);
		0 ignored issues – show Bug Best Practice introduced 2017-12-22 05:31 UTC by Report Bug Copy Issue Report Show Similar Issues like this The return type of `return $this->num($limit);` (`Doctrine\ODM\MongoDB\Aggregation\Stage\GeoNear`) is incompatible with the return type of the parent method `Doctrine\ODM\MongoDB\Aggregation\Stage::limit` of type `Doctrine\ODM\MongoDB\Aggregation\Stage\Limit`. If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design. Let’s take a look at an example: class Author { private $name; public function __construct($name) { $this->name = $name; } public function getName() { return $this->name; } } abstract class Post { public function getAuthor() { return 'Johannes'; } } class BlogPost extends Post { public function getAuthor() { return new Author('Johannes'); } } class ForumPost extends Post { /* ... */ } function my_function(Post $post) { echo strtoupper($post->getAuthor()); } Our function `my_function` expects a `Post` object, and outputs the author of the post. The base class `Post` returns a simple string and outputting a simple string will work just fine. However, the child class `BlogPost` which is a sub-type of `Post` instead decided to return an `object`, and is therefore violating the SOLID principles. If a `BlogPost` were passed to `my_function`, PHP would not complain, but ultimately fail when executing the `strtoupper` call in its body. Loading history...
119		}
120
121		/**
122		* The maximum distance from the center point that the documents can be.
123		*/
124	1	public function maxDistance(float $maxDistance) : self
125		{
126	1	$this->maxDistance = $maxDistance;
127
128	1	return $this;
129		}
130
131		/**
132		* The minimum distance from the center point that the documents can be.
133		*/
134	1	public function minDistance(float $minDistance) : self
135		{
136	1	$this->minDistance = $minDistance;
137
138	1	return $this;
139		}
140
141		/**
142		* The point for which to find the closest documents.
143		*
144		* A GeoJSON point may be provided as the first and only argument for
145		* 2dsphere queries. This single parameter may be a GeoJSON point object or
146		* an array corresponding to the point's JSON representation. If GeoJSON is
147		* used, the "spherical" option will default to true.
148		*
149		* @param float\|array\|Point $x
150		* @param float $y
151		*/
152	11	public function near($x, $y = null) : self
153		{
154	11	if ($x instanceof Point) {
		0 ignored issues – show Bug introduced 2018-01-06 12:29 UTC by Report Bug Copy Issue Report Show Similar Issues like this The class `GeoJson\Geometry\Point` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
155		$x = $x->jsonSerialize();
156		}
157
158	11	$this->near = is_array($x) ? $x : [$x, $y];
159	11	$this->spherical = is_array($x) && isset($x['type']);
160
161	11	return $this;
162		}
163
164		/**
165		* The maximum number of documents to return.
166		*/
167	3	public function num(int $num) : self
168		{
169	3	$this->num = $num;
170
171	3	return $this;
172		}
173
174		/**
175		* Required if using a 2dsphere index. Determines how MongoDB calculates the distance.
176		*/
177		public function spherical(bool $spherical = true) : self
178		{
179		$this->spherical = $spherical;
180
181		return $this;
182		}
183
184		/**
185		* If this value is true, the query returns a matching document once, even if more than one of the document’s location fields match the query.
186		*/
187	1	public function uniqueDocs(bool $uniqueDocs = true) : self
188		{
189	1	$this->uniqueDocs = $uniqueDocs;
190
191	1	return $this;
192		}
193		}
194

doctrine / mongodb-odm

Push — master ( 09b86b...ba0798 )

Doctrine/ODM/MongoDB/Aggregation/Stage/GeoNear.php (2 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like