DocumentIterator::convertDocument() - Code Metrics - Inspection of "[WIP] Refactor aggregations result handling" - ongr-io/ElasticsearchBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#555)

by Mantas

created 2016-01-21 14:31 UTC

DocumentIterator::convertDocument() A

↳ Parent: DocumentIterator

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	1

<?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\ValueAggregation;
use ONGR\ElasticsearchBundle\Service\Manager;

/**
 * Class DocumentIterator.
 */
class DocumentIterator extends AbstractResultsIterator
{
    /**
     * @var array
     */
    private $aggregations;


    /**
     * {@inheritdoc}
     */
    public function __construct(array $rawData, Manager $manager, array $scroll = [])
    {
        if (isset($rawData['aggregations'])) {
            $this->aggregations = $rawData['aggregations'];
            unset($rawData['aggregations']);
        }

        parent::__construct($rawData, $manager, $scroll);
    }

    /**
     * Get a specific aggregation by name. It fetches from the top level only.
     *
     * @param string $name
     *
     * @return ValueAggregation|null
     */
    public function getAggregation($name)
    {
        if (!isset($this->aggregations[$name])) {
            return null;
        }

        return new ValueAggregation($this->aggregations[$name]);
    }

    /**
     * {@inheritdoc}
     */
    protected function convertDocument(array $document)
    {
        return $this->getConverter()->convertToDocument($document, $this->getManager());
    }
}


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\ValueAggregation;
15			use ONGR\ElasticsearchBundle\Service\Manager;
16
17			/**
18			* Class DocumentIterator.
19			*/
20			class DocumentIterator extends AbstractResultsIterator
21			{
22			/**
23			* @var array
24			*/
25			private $aggregations;
			0 ignored issues – show Comprehensibility introduced 2016-01-19 14:22 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
26
27			/**
28			* {@inheritdoc}
29			*/
30			public function __construct(array $rawData, Manager $manager, array $scroll = [])
31			{
32			if (isset($rawData['aggregations'])) {
33			$this->aggregations = $rawData['aggregations'];
34			unset($rawData['aggregations']);
35			}
36
37			parent::__construct($rawData, $manager, $scroll);
38			}
39
40			/**
41			* Get a specific aggregation by name. It fetches from the top level only.
42			*
43			* @param string $name
44			*
45			* @return ValueAggregation\|null
46			*/
47			public function getAggregation($name)
48			{
49			if (!isset($this->aggregations[$name])) {
50			return null;
51			}
52
53			return new ValueAggregation($this->aggregations[$name]);
54			}
55
56			/**
57			* {@inheritdoc}
58			*/
59			protected function convertDocument(array $document)
60			{
61			return $this->getConverter()->convertToDocument($document, $this->getManager());
62			}
63			}
64

ongr-io / ElasticsearchBundle

Pull Request — master (#555)

DocumentIterator::convertDocument() A

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like