DocumentIterator::getAggregations() - Code Metrics - ongr-io/ElasticsearchBundle - Measure and Improve Code Quality continuously with Scrutinizer

DocumentIterator::getAggregations() A
last analyzed 2020-03-31 10:30 UTC

↳ Parent: DocumentIterator

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.9332
c	0
b	0
f	0
cc	2
nc	2
nop	0

<?php

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

namespace ONGR\ElasticsearchBundle\Result;

use ONGR\ElasticsearchBundle\Result\Aggregation\AggregationValue;

class DocumentIterator extends AbstractResultsIterator
{
    public function getAggregations()
    {
        $aggregations = [];

        foreach (parent::getAggregations() as $key => $aggregation) {
            $aggregations[$key] = $this->getAggregation($key);
        }

        return $aggregations;
    }

    public function getAggregation($name)
    {
        $aggregations = parent::getAggregations();
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
        if (!array_key_exists($name, $aggregations)) {
            return null;
        }

        return new AggregationValue($aggregations[$name]);
    }

    protected function convertDocument(array $raw)
    {
        $data = $raw['_source'] ?? $raw['_fields'] ?? null;
        $data['_id'] = $raw['_id'] ?? null;

        return $this->getConverter()->convertArrayToDocument($this->getIndex()->getNamespace(), $data);
    }
}


1			<?php
2
3			/*
4			* This file is part of the ONGR package.
5			*
6			* (c) NFQ Technologies UAB <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace ONGR\ElasticsearchBundle\Result;
13
14			use ONGR\ElasticsearchBundle\Result\Aggregation\AggregationValue;
15
16			class DocumentIterator extends AbstractResultsIterator
17			{
18			public function getAggregations()
19			{
20			$aggregations = [];
21
22			foreach (parent::getAggregations() as $key => $aggregation) {
23			$aggregations[$key] = $this->getAggregation($key);
24			}
25
26			return $aggregations;
27			}
28
29			public function getAggregation($name)
30			{
31			$aggregations = parent::getAggregations();
			0 ignored issues – show Comprehensibility Bug introduced 2016-09-09 09:00 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`getAggregations()` instead of `getAggregation()`). Are you sure this is correct? If so, you might want to change this to `$this->getAggregations()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
32			if (!array_key_exists($name, $aggregations)) {
33			return null;
34			}
35
36			return new AggregationValue($aggregations[$name]);
37			}
38
39			protected function convertDocument(array $raw)
40			{
41			$data = $raw['_source'] ?? $raw['_fields'] ?? null;
42			$data['_id'] = $raw['_id'] ?? null;
43
44			return $this->getConverter()->convertArrayToDocument($this->getIndex()->getNamespace(), $data);
45			}
46			}
47

ongr-io / ElasticsearchBundle

DocumentIterator::getAggregations() A last analyzed 2020-03-31 10:30 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

DocumentIterator::getAggregations() A
last analyzed 2020-03-31 10:30 UTC