Cortex\Pages\Transformers\Managerarea\PageTransformer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	transform()	0	15	2

<?php

declare(strict_types=1);

namespace Cortex\Pages\Transformers\Managerarea;

use League\Fractal\TransformerAbstract;
use Rinvex\Pages\Contracts\PageContract;

class PageTransformer extends TransformerAbstract
{
    /**
     * @return array

Consider making the return type a bit more specific; maybe use array<string,integer|string|boolean>.

     */
    public function transform(PageContract $page)
    {
        return [
            'id' => (int) $page->id,

Accessing id on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'uri' => (string) $page->uri === '/' ? '/' : '/'.$page->uri,

Accessing uri on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'view' => (string) $page->view,

Accessing view on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'slug' => (string) $page->slug,

Accessing slug on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'route' => (string) $page->route,

Accessing route on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'title' => (string) $page->title,

Accessing title on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'is_active' => (bool) $page->is_active,

Accessing is_active on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'middleware' => (string) $page->middleware,

Accessing middleware on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'created_at' => (string) $page->created_at,

Accessing created_at on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

            'updated_at' => (string) $page->updated_at,

Accessing updated_at on the interface Rinvex\Pages\Contracts\PageContract suggest that you code against a concrete implementation. How about adding an instanceof check?

        ];
    }
}