Lej\Component\Infrastructure\Serialization\IdSubscribingHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

16.66%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getSubscribingMethods()	16	16	2
A	serializeId()	4	4	1

<?php

declare(strict_types=1);

namespace Lej\Component\Infrastructure\Serialization;

use JMS\Serializer\GraphNavigator;
use JMS\Serializer\Handler\SubscribingHandlerInterface;
use JMS\Serializer\VisitorInterface;
use Lej\Component\Domain\Model\Id;

class IdSubscribingHandler implements SubscribingHandlerInterface

This class seems to be duplicated in your project.

{
    /**
     * {@inheritdoc}
     */
    public static function getSubscribingMethods()
    {
        $methods = [];
        $formats = ['json'];

        foreach ($formats as $format) {
            $methods[] = [
                'type' => 'Id',
                'format' => $format,
                'direction' => GraphNavigator::DIRECTION_SERIALIZATION,
                'method' => 'serializeId',
            ];
        }

        return $methods;
    }

    /**
     * @param VisitorInterface $visitor
     * @param Id $id
     * @return string
     */
    public function serializeId(VisitorInterface $visitor, Id $id)

The parameter $visitor is not used and could be removed.

    {
        return $id->toString();
    }
}