TgBotApi\BotApiBase\Normalizer\EditMessageMediaNormalizer

Complexity

Total Complexity

3

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	supportsNormalization()	0	3	1
A	__construct()	0	6	1
A	normalize()	0	12	1

<?php

declare(strict_types=1);

namespace TgBotApi\BotApiBase\Normalizer;

use Symfony\Component\Serializer\Exception\ExceptionInterface;
use Symfony\Component\Serializer\Normalizer\NormalizerInterface;
use Symfony\Component\Serializer\Serializer;
use TgBotApi\BotApiBase\Method\EditMessageMediaMethod;

/**
 * Class MediaGroupNormalizer.
 */
class EditMessageMediaNormalizer implements NormalizerInterface
{
    /**
     * @var InputMediaNormalizer
     */
    private $inputMediaNormalizer;
    /**
     * @var NormalizerInterface
     */
    private $objectNormalizer;

    /**
     * MediaGroupNormalizer constructor.
     */
    public function __construct(
        InputMediaNormalizer $inputMediaNormalizer,
        NormalizerInterface $objectNormalizer
    ) {
        $this->inputMediaNormalizer = $inputMediaNormalizer;
        $this->objectNormalizer = $objectNormalizer;
    }

    /**
     * @param EditMessageMediaMethod $topic
     * @param null                   $format

Are you sure the doc-type for parameter $format is correct as it would always require null to be passed?

     *
     * @throws ExceptionInterface
     *
     * @return array|bool|float|int|mixed|string
     */
    public function normalize($topic, $format = null, array $context = [])
    {
        $serializer = new Serializer([
            $this->inputMediaNormalizer,
            new JsonSerializableNormalizer($this->objectNormalizer),
            $this->objectNormalizer,
        ]);
        $topic->media = \json_encode($serializer->normalize($topic->media, null, ['skip_null_values' => true]));

It seems like json_encode($serializer->normalize($topic->media, null, array('skip_null_values' => true))) of type string is incompatible with the declared type TgBotApi\BotApiBase\Type\InputMedia\InputMediaType of property $media.

        $topic->replyMarkup = $serializer->normalize($topic->replyMarkup, null, ['skip_null_values' => true]);

        return $serializer->normalize($topic, null, ['skip_null_values' => true]);
    }

    /**
     * @param mixed $data
     * @param null  $format

Are you sure the doc-type for parameter $format is correct as it would always require null to be passed?

     */
    public function supportsNormalization($data, $format = null): bool
    {
        return $data instanceof EditMessageMediaMethod;
    }
}