PackageSubscriber::onPostDeserialize() - Code Metrics - Inspection of "[BridgeBundle] Invalid argument supplied for forea..." - SuperdeskWebPublisher/SWPBridgeBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7a9694...a77116 )

by Rafał

created 2016-09-09 14:14 UTC

PackageSubscriber::onPostDeserialize() A

↳ Parent: PackageSubscriber

Complexity

Conditions	3
Paths	3

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	11
rs	9.4285
cc	3
eloc	5
nc	3
nop	1

<?php

/**
 * This file is part of the Superdesk Web Publisher Bridge Bundle.
 *
 * Copyright 2016 Sourcefabric z.ú. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2016 Sourcefabric z.ú
 * @license http://www.superdesk.org/license
 */

namespace SWP\Bundle\BridgeBundle\Serializer;

use Doctrine\Common\Collections\ArrayCollection;
use JMS\Serializer\EventDispatcher\EventSubscriberInterface;
use JMS\Serializer\EventDispatcher\ObjectEvent;
use SWP\Component\Bridge\Model\PackageInterface;

class PackageSubscriber implements EventSubscriberInterface
{
    /**
     * {@inheritdoc}
     */
    public static function getSubscribedEvents()
    {
        return [
            [
                'event' => 'serializer.post_deserialize',
                'method' => 'onPostDeserialize',
            ],
        ];
    }

    /**
     * @param ObjectEvent $event
     */
    public function onPostDeserialize(ObjectEvent $event)
    {
        if ($event->getObject() instanceof PackageInterface) {
            /** @var PackageInterface $package */
            $package = $event->getObject();

            if (null === $package->getItems()) {
                $package->setItems(new ArrayCollection());

            }
        }
    }
}


1			<?php
2
3			/**
4			* This file is part of the Superdesk Web Publisher Bridge Bundle.
5			*
6			* Copyright 2016 Sourcefabric z.ú. and contributors.
7			*
8			* For the full copyright and license information, please see the
9			* AUTHORS and LICENSE files distributed with this source code.
10			*
11			* @copyright 2016 Sourcefabric z.ú
12			* @license http://www.superdesk.org/license
13			*/
14
15			namespace SWP\Bundle\BridgeBundle\Serializer;
16
17			use Doctrine\Common\Collections\ArrayCollection;
18			use JMS\Serializer\EventDispatcher\EventSubscriberInterface;
19			use JMS\Serializer\EventDispatcher\ObjectEvent;
20			use SWP\Component\Bridge\Model\PackageInterface;
21
22			class PackageSubscriber implements EventSubscriberInterface
23			{
24			/**
25			* {@inheritdoc}
26			*/
27			public static function getSubscribedEvents()
28			{
29			return [
30			[
31			'event' => 'serializer.post_deserialize',
32			'method' => 'onPostDeserialize',
33			],
34			];
35			}
36
37			/**
38			* @param ObjectEvent $event
39			*/
40			public function onPostDeserialize(ObjectEvent $event)
41			{
42			if ($event->getObject() instanceof PackageInterface) {
43			/** @var PackageInterface $package */
44			$package = $event->getObject();
45
46			if (null === $package->getItems()) {
47			$package->setItems(new ArrayCollection());
			0 ignored issues – show Bug introduced 2016-09-09 14:17 UTC by Report Bug Copy Issue Report The method `setItems()` does not seem to exist on `object<SWP\Component\Bri...Model\PackageInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
48			}
49			}
50			}
51			}
52

SuperdeskWebPublisher / SWPBridgeBundle

Push — master ( 7a9694...a77116 )

PackageSubscriber::onPostDeserialize() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like