Astroromic\Stancy\Models\Page

Complexity

Total Complexity

12

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	make()	0	7	1
A	parse()	0	13	5
A	page()	0	13	3
A	data()	0	7	1
A	toHtml()	0	3	1
A	__construct()	0	5	1

<?php

namespace Astroromic\Stancy\Models;

use Exception;
use Illuminate\Contracts\Support\Htmlable;
use Illuminate\Contracts\View\Factory as ViewFactory;
use Illuminate\Contracts\View\View;
use Spatie\Sheets\Facades\Sheets;

class Page implements Htmlable
{
    /** @var string */
    protected $view;

    /** @var string|null */
    protected $page;

    /** @var PageData|array */

@var annotation of property \Astroromic\Stancy\Models\Page::$data does not specify type hint for its items.

    protected $data;

    /** @var ViewFactory */
    protected $viewFactory;

    public function __construct(ViewFactory $viewFactory, array $data, ?string $page = null)

Method \Astroromic\Stancy\Models\Page::__construct() does not have @param annotation for its traversable parameter $data.

    {
        $this->viewFactory = $viewFactory;
        $this->page($page);
        $this->data($data);
    }

    public static function make(string $collection, string $name, ?string $page = null): self
    {
        $sheet = Sheets::collection($collection)->get($name);

        return app(static::class, [

The expression return app(static::class, array('data' => $sheet->toArray(), 'page' => $page)) could return the type Illuminate\Contracts\Foundation\Application which is incompatible with the type-hinted return Astroromic\Stancy\Models\Page. Consider adding an additional type-check to rule them out.

            'data' => $sheet->toArray(),
            'page' => $page,
        ]);
    }

    public function page(?string $page): self
    {
        if (is_string($page)) {
            if (! ($page instanceof PageData)) {

$page is never a sub-type of Astroromic\Stancy\Models\PageData.

                throw new Exception(sprintf('The page data class [%s] has to extend %s.', $page, PageData::class));
            }
        }

        $this->page = $page;

        $this->parse();

        return $this;
    }

    public function data(array $data): self

Method \Astroromic\Stancy\Models\Page::data() does not have @param annotation for its traversable parameter $data.

    {
        $this->data = $data;

        $this->parse();

        return $this;
    }

    public function toHtml(): View
    {
        return $this->viewFactory->make($this->view, $this->data);
    }

    protected function parse(): void
    {
        if ($this->page === null) {
            return;
        }

        if (is_array($this->data) && empty($this->data)) {
            return;
        }

        $this->data = forward_static_call(
            [$this->page, 'make'],
            is_array($this->data) ? $this->data : $this->data->toArray()
        );
    }
}