Imanghafoori\HeyMan\Hooks\ViewHooks

Complexity

Total Complexity

5

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	checkViewExists()	0	4	2
A	whenYouSeeViewFile()	0	5	1
A	normalizeView()	0	9	1
A	whenYouMakeView()	0	3	1

<?php

namespace Imanghafoori\HeyMan\Hooks;

trait ViewHooks
{
    /**
     * @param array|string $views
     * @return \Imanghafoori\HeyMan\YouShouldHave
     */
    public function whenYouSeeViewFile(...$views)
    {
        $views = $this->normalizeView($views);

        return $this->authorize($views);

It seems like authorize() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param array|string $views
     * @return \Imanghafoori\HeyMan\YouShouldHave
     */
    public function whenYouMakeView(...$views)
    {
        return $this->whenYouSeeViewFile(...$views);
    }

    /**
     * @param $views
     * @return array
     */
    private function normalizeView(array $views): array
    {
        $views = $this->normalizeInput($views);

The method normalizeInput() does not exist on Imanghafoori\HeyMan\Hooks\ViewHooks. Did you maybe mean normalizeView()?

        $mapper = function ($view) {
            $this->checkViewExists($view);
            return 'creating: '.\Illuminate\View\ViewName::normalize($view);
        };

        return array_map($mapper, $views);
    }

    private function checkViewExists($view)
    {
        if (strpos($view, '*') === false) {
            view()->getFinder()->find($view);
        }
    }
}