SimpleEventStoreManager\Infrastructure\DataTransformers\YamlEventDataTransformer::transform()

Complexity

Conditions	3
Paths	2

Size

Total Lines	31
Code Lines	20

Duplication

Lines	5
Ratio	16.13 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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;
    }