SimpleEventStoreManager\Infrastructure\DataTransformers\YamlEventDataTransformer

Complexity

Total Complexity

3

Size/Duplication

Total Lines	41
Duplicated Lines	12.2 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	transform()	5	31	3

<?php
/**
 * This file is part of the Simple EventStore Manager package.
 *
 * (c) Mauro Cassani<https://github.com/mauretto78>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace SimpleEventStoreManager\Infrastructure\DataTransformers;

use SimpleEventStoreManager\Infrastructure\DataTransformers\Contracts\DataTransformerInterface;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use JMS\Serializer\Serializer;

class YamlEventDataTransformer extends AbstractEventDataTransformer implements DataTransformerInterface
{
    /**
     * @param array $events
     * @param int $eventsCount
     * @param int $page
     * @param int $maxPerPage
     *
     * @return Response
     */
    public function transform($events, $eventsCount, $page, $maxPerPage)
    {
        $pageCount = count($events);
        $response = new Response(
            $this->serializer->serialize(
                [
                    '_meta' => [
                        'page' => $page,
                        'records_per_page' => $maxPerPage,
                        'total_pages' => $numberOfPages = ceil($eventsCount/$maxPerPage),
                        'total_count' => $eventsCount
                    ],
                    '_links' => [
                        $this->calculateLinks($page, $numberOfPages)

$numberOfPages of type double is incompatible with the type integer expected by parameter $numberOfPages of SimpleEventStoreManager\Infrastructure\DataTransformers\AbstractEventDataTransformer::calculateLinks().

                    ],
                    'events' => (is_array($events)) ? $events : $this->convertEventsDataToArray($events)
                ], 'yml'),
            $this->getHttpStatusCode($pageCount)
        );

        // set infinite cache if page is complete
        if ($maxPerPage === $pageCount) {

This code seems to be duplicated across your project.

            $response
                ->setMaxAge(60 * 60 * 24 * 365)
                ->setSharedMaxAge(60 * 60 * 24 * 365);
        }

        $response->headers->set('Content-type', 'text/yaml');
        $response->setCharset('utf-8');

        return $response;
    }
}