Wesnick\WorkflowBundle\Serializer\WorkflowNormalizer

Complexity

Total Complexity

9

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	supportsNormalization()	0	3	1
A	supportsDenormalization()	0	3	1
A	hasCacheableSupportsMethod()	0	3	1
A	__construct()	0	8	1
A	denormalize()	0	3	1
A	setSerializer()	0	3	1
A	normalize()	0	10	3

<?php

declare(strict_types=1);

/*
 * (c) 2019, Wesley O. Nichols
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Wesnick\WorkflowBundle\Serializer;

use ApiPlatform\Core\JsonLd\Serializer\ItemNormalizer;
use Symfony\Component\Serializer\Normalizer\CacheableSupportsMethodInterface;
use Symfony\Component\Serializer\Normalizer\ContextAwareDenormalizerInterface;
use Symfony\Component\Serializer\Normalizer\NormalizerInterface;
use Symfony\Component\Serializer\SerializerAwareInterface;
use Symfony\Component\Serializer\SerializerInterface;
use Wesnick\WorkflowBundle\Model\PotentialActionInterface;
use Wesnick\WorkflowBundle\WorkflowActionGenerator;

/**
 * Class WorkflowNormalizer.
 *
 * @author Wesley O. Nichols <spanishwes@gmail.com>
 */
class WorkflowNormalizer implements NormalizerInterface, CacheableSupportsMethodInterface, ContextAwareDenormalizerInterface, SerializerAwareInterface
{
    /**
     * @var ItemNormalizer
     */
    private $decorated;
    private $customNormalizer;
    private $workflowActions;

    public function __construct(
        NormalizerInterface $decorated,
        NormalizerInterface $customNormalizer,
        WorkflowActionGenerator $workflowActions
    ) {
        $this->decorated = $decorated;

$decorated is of type Symfony\Component\Serializer\Normalizer\NormalizerInterface, but the property $decorated was declared to be of type ApiPlatform\Core\JsonLd\Serializer\ItemNormalizer. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->customNormalizer = $customNormalizer;
        $this->workflowActions = $workflowActions;
    }

    /**
     * {@inheritdoc}
     */
    public function supportsNormalization($data, $format = null): bool
    {
        return $this->decorated->supportsNormalization($data, $format);
    }

    /**
     * {@inheritdoc}
     */
    public function hasCacheableSupportsMethod(): bool
    {
        return true;
    }

    public function normalize($object, $format = null, array $context = [])
    {
        if ($object instanceof PotentialActionInterface) {
            $actions = $this->workflowActions->getActionsForSubject($object);
            foreach ($actions as $action) {
                $object->addPotentialAction($action);
            }
        }

        return $this->decorated->normalize($object, $format, $context);
    }

    public function supportsDenormalization($data, $type, $format = null, array $context = [])
    {
        return $this->decorated->supportsDenormalization($data, $type, $format, $context);

The call to ApiPlatform\Core\JsonLd\Serializer\ItemNormalizer::supportsDenormalization() has too many arguments starting with $context.

    }

    public function denormalize($data, $class, $format = null, array $context = [])
    {
        return $this->decorated->denormalize($data, $class, $format, $context);
    }

    public function setSerializer(SerializerInterface $serializer)
    {
        $this->decorated->setSerializer($serializer);
    }
}