ResultIterator::__construct() - Code Metrics - Inspection of "Feature/setup" - sulu/activity-log-elasticsearch - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1)

unknown

created 2017-05-24 11:36 UTC

ResultIterator::__construct() B

↳ Parent: ResultIterator

Complexity

Conditions	5
Paths	6

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	16
rs	8.8571
c	0
b	0
f	0
cc	5
eloc	8
nc	6
nop	3

<?php

namespace Sulu\Bundle\ElasticsearchActivityLogBundle\Storage;

use ONGR\ElasticsearchBundle\Result\DocumentIterator;
use Sulu\Bundle\ElasticsearchActivityLogBundle\Document\ActivityLoggerViewDocument;
use Sulu\Component\Security\Authentication\UserInterface;

/**
 * Take result iterator, parents and creator - and combines them.
 */
class ResultIterator extends \IteratorIterator
{
    /**
     * @var ActivityLoggerViewDocument[]
     */
    private $parents = [];

    /**
     * @var UserInterface[]
     */
    private $creators = [];

    /**
     * @param DocumentIterator $iterator
     * @param DocumentIterator $parents
     * @param array $creators
     */
    public function __construct(DocumentIterator $iterator, DocumentIterator $parents = null, array $creators = null)
    {
        parent::__construct($iterator);

        if (null !== $parents) {
            foreach ($parents as $parent) {
                $this->parents[$parent->getUuid()] = $parent;
            }
        }

        if (null !== $creators) {
            foreach ($creators as $creator) {
                $this->creators[$creator->getId()] = $creator;
            }
        }
    }

    /**
     * Set creator and parent to document.
     *
     * @return ActivityLoggerViewDocument
     */
    public function current()
    {
        $result = parent::current();
        if (!$result) {
            return $result;
        }

        $result->setParent(
            array_key_exists($result->getParentUuid(), $this->parents) ? $this->parents[$result->getParentUuid()] : null
        );

        $result->setCreator(
            array_key_exists($result->getCreatorId(), $this->creators) ? $this->creators[$result->getCreatorId()] : null
        );

        return $result;
    }
}


1			<?php
2
3			namespace Sulu\Bundle\ElasticsearchActivityLogBundle\Storage;
4
5			use ONGR\ElasticsearchBundle\Result\DocumentIterator;
6			use Sulu\Bundle\ElasticsearchActivityLogBundle\Document\ActivityLoggerViewDocument;
7			use Sulu\Component\Security\Authentication\UserInterface;
8
9			/**
10			* Take result iterator, parents and creator - and combines them.
11			*/
12			class ResultIterator extends \IteratorIterator
13			{
14			/**
15			* @var ActivityLoggerViewDocument[]
16			*/
17			private $parents = [];
18
19			/**
20			* @var UserInterface[]
21			*/
22			private $creators = [];
23
24			/**
25			* @param DocumentIterator $iterator
26			* @param DocumentIterator $parents
27			* @param array $creators
28			*/
29			public function __construct(DocumentIterator $iterator, DocumentIterator $parents = null, array $creators = null)
30			{
31			parent::__construct($iterator);
32
33			if (null !== $parents) {
34			foreach ($parents as $parent) {
35			$this->parents[$parent->getUuid()] = $parent;
36			}
37			}
38
39			if (null !== $creators) {
40			foreach ($creators as $creator) {
41			$this->creators[$creator->getId()] = $creator;
42			}
43			}
44			}
45
46			/**
47			* Set creator and parent to document.
48			*
49			* @return ActivityLoggerViewDocument
50			*/
51			public function current()
52			{
53			$result = parent::current();
54			if (!$result) {
55			return $result;
56			}
57
58			$result->setParent(
59			array_key_exists($result->getParentUuid(), $this->parents) ? $this->parents[$result->getParentUuid()] : null
60			);
61
62			$result->setCreator(
63			array_key_exists($result->getCreatorId(), $this->creators) ? $this->creators[$result->getCreatorId()] : null
64			);
65
66			return $result;
67			}
68			}
69

sulu / activity-log-elasticsearch

Pull Request — master (#1)

ResultIterator::__construct() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like