ChildrenSubscriber::handleHydrate() - Code Metrics - sulu-io/sulu-document-manager - Measure and Improve Code Quality continuously with Scrutinizer

ChildrenSubscriber::handleHydrate() A
last analyzed 2015-12-18 14:57 UTC

↳ Parent: ChildrenSubscriber

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
c	2
b	1
f	0
dl	0
loc	11
rs	9.4286
cc	2
eloc	6
nc	2
nop	1

<?php

/*
 * This file is part of Sulu.
 *
 * (c) MASSIVE ART WebServices GmbH
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Sulu\Component\DocumentManager\Subscriber\Behavior\Mapping;

use Sulu\Component\DocumentManager\Behavior\Mapping\ChildrenBehavior;
use Sulu\Component\DocumentManager\Event\HydrateEvent;
use Sulu\Component\DocumentManager\Events;
use Sulu\Component\DocumentManager\ProxyFactory;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

/**
 * Set the children on the document.
 */
class ChildrenSubscriber implements EventSubscriberInterface
{
    /**
     * @var ProxyFactory
     */
    private $proxyFactory;

    /**
     * @param ProxyFactory $proxyFactory
     */
    public function __construct(ProxyFactory $proxyFactory)
    {
        $this->proxyFactory = $proxyFactory;
    }

    /**
     * {@inheritdoc}
     */
    public static function getSubscribedEvents()
    {
        return [
            Events::HYDRATE => 'handleHydrate',
        ];
    }

    /**
     * @param HydrateEvent $event
     */
    public function handleHydrate(HydrateEvent $event)
    {
        $document = $event->getDocument();

        if (!$document instanceof ChildrenBehavior) {
            return;
        }

        $accessor = $event->getAccessor();
        $accessor->set('children', $this->proxyFactory->createChildrenCollection($document, $event->getOptions()));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }
}


1			<?php
2
3			/*
4			* This file is part of Sulu.
5			*
6			* (c) MASSIVE ART WebServices GmbH
7			*
8			* This source file is subject to the MIT license that is bundled
9			* with this source code in the file LICENSE.
10			*/
11
12			namespace Sulu\Component\DocumentManager\Subscriber\Behavior\Mapping;
13
14			use Sulu\Component\DocumentManager\Behavior\Mapping\ChildrenBehavior;
15			use Sulu\Component\DocumentManager\Event\HydrateEvent;
16			use Sulu\Component\DocumentManager\Events;
17			use Sulu\Component\DocumentManager\ProxyFactory;
18			use Symfony\Component\EventDispatcher\EventSubscriberInterface;
19
20			/**
21			* Set the children on the document.
22			*/
23			class ChildrenSubscriber implements EventSubscriberInterface
24			{
25			/**
26			* @var ProxyFactory
27			*/
28			private $proxyFactory;
29
30			/**
31			* @param ProxyFactory $proxyFactory
32			*/
33			public function __construct(ProxyFactory $proxyFactory)
34			{
35			$this->proxyFactory = $proxyFactory;
36			}
37
38			/**
39			* {@inheritdoc}
40			*/
41			public static function getSubscribedEvents()
42			{
43			return [
44			Events::HYDRATE => 'handleHydrate',
45			];
46			}
47
48			/**
49			* @param HydrateEvent $event
50			*/
51			public function handleHydrate(HydrateEvent $event)
52			{
53			$document = $event->getDocument();
54
55			if (!$document instanceof ChildrenBehavior) {
56			return;
57			}
58
59			$accessor = $event->getAccessor();
60			$accessor->set('children', $this->proxyFactory->createChildrenCollection($document, $event->getOptions()));
			0 ignored issues – show Documentation introduced 2015-12-01 15:26 UTC by Report Bug Copy Issue Report `$event->getOptions()` is of type `object<Symfony\Component...solver\OptionsResolver>`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
61			}
62			}
63

sulu-io / sulu-document-manager

ChildrenSubscriber::handleHydrate() A last analyzed 2015-12-18 14:57 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ChildrenSubscriber::handleHydrate() A
last analyzed 2015-12-18 14:57 UTC