Rinvex\Menus\Factories\MenuFactory

Complexity

Total Complexity

45

Size/Duplication

Total Lines	442
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes

0

25 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	findBy()	0	10	3
A	findByTitleOrAdd()	0	9	3
A	setViewFactory()	0	6	1
A	setView()	0	6	1
A	setUrlPrefix()	0	6	1
A	setPresenter()	0	6	1
A	getPresenter()	0	4	1
A	presenterExists()	0	4	1
A	setBindings()	0	6	1
B	resolve()	0	21	6
A	resolveItems()	0	10	2
A	add()	0	7	1
A	dropdown()	0	6	1
A	route()	0	4	1
A	url()	0	6	2
A	header()	0	6	1
A	divider()	0	4	1
A	count()	0	4	1
A	destroy()	0	6	1
A	getOrderedItems()	0	4	1
A	render()	0	12	4
A	renderView()	0	4	1
B	renderMenu()	0	25	6
A	formatUrl()	0	6	2

<?php

declare(strict_types=1);

namespace Rinvex\Menus\Factories;

use Countable;
use Rinvex\Menus\Models\MenuItem;
use Illuminate\Support\Collection;
use Illuminate\View\Factory as ViewFactory;
use Rinvex\Menus\Presenters\NavbarPresenter;

class MenuFactory implements Countable
{
    /**
     * The items collection.
     *
     * @var \Illuminate\Support\Collection
     */
    protected $items;

    /**
     * The presenter class.
     *
     * @var string
     */
    protected $presenter = NavbarPresenter::class;

    /**
     * The URL prefix.
     *
     * @var string|null
     */
    protected $urlPrefix;

    /**
     * The view name.
     *
     * @var string
     */
    protected $view;

    /**
     * The laravel view factory instance.
     *
     * @var \Illuminate\View\Factory
     */
    protected $views;

    /**
     * Resolved item binding map.
     *
     * @var array
     */
    protected $bindings = [];

    /**
     * Create a new MenuFactory instance.
     */
    public function __construct()
    {
        $this->items = collect();
    }

    /**
     * Find menu item by given key and value.
     *
     * @param string   $key
     * @param string   $value
     * @param callable $callback

Should the type for parameter $callback not be null|callable? Also, consider making the array more specific, something like array<String>, or String[].

     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    public function findBy(string $key, string $value, callable $callback = null)
    {
        $item = $this->items->filter(function ($item) use ($key, $value) {
            return $item->{$key} === $value;
        })->first();

        (! is_callable($callback) || ! $item) || call_user_func($callback, $item);

        return $item;
    }

    /**
     * Find menu item by given key and value.
     *
     * @param string   $title
     * @param int      $order

Should the type for parameter $order not be null|integer?

     * @param string   $icon

Should the type for parameter $icon not be null|string?

     * @param array    $attributes
     * @param callable $callback

Should the type for parameter $callback not be null|callable? Also, consider making the array more specific, something like array<String>, or String[].

     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    public function findByTitleOrAdd(string $title, int $order = null, string $icon = null, array $attributes = [], callable $callback = null)

This line exceeds maximum limit of 120 characters; contains 142 characters

    {
        if (! ($item = $this->findBy('title', $title, $callback))) {
            $item = $this->add(compact('title', 'order', 'icon', 'attributes'));
            ! is_callable($callback) || call_user_func($callback, $item);
        }

        return $item;
    }

    /**
     * Set view factory instance.
     *
     * @param \Illuminate\View\Factory $views
     *
     * @return $this
     */
    public function setViewFactory(ViewFactory $views)
    {
        $this->views = $views;

        return $this;
    }

    /**
     * Set view.
     *
     * @param string $view
     *
     * @return $this
     */
    public function setView(string $view)
    {
        $this->view = $view;

        return $this;
    }

    /**
     * Set Prefix URL.
     *
     * @param string $prefixUrl

There is no parameter named $prefixUrl. Was it maybe removed?

     *
     * @return $this
     */
    public function setUrlPrefix(string $urlPrefix)
    {
        $this->urlPrefix = $urlPrefix;

        return $this;
    }

    /**
     * Set new presenter class.
     *
     * @param string $presenter
     *
     * @return $this
     */
    public function setPresenter(string $presenter)
    {
        $this->presenter = app('rinvex.menus.presenters')->get($presenter);

        return $this;
    }

    /**
     * Get presenter instance.
     *
     * @return \Rinvex\Menus\Contracts\PresenterContract
     */
    public function getPresenter()
    {
        return new $this->presenter();
    }

    /**
     * Determine if the given name in the presenter style.
     *
     * @param string $presenter
     *
     * @return bool
     */
    public function presenterExists(string $presenter)
    {
        return app('rinvex.menus.presenters')->has($presenter);
    }

    /**
     * Set the resolved item bindings.
     *
     * @param array $bindings
     *
     * @return $this
     */
    public function setBindings(array $bindings)
    {
        $this->bindings = $bindings;

        return $this;
    }

    /**
     * Resolves a key from the bindings array.
     *
     * @param string|array $key
     *
     * @return mixed

Consider making the return type a bit more specific; maybe use array|string.

     */
    public function resolve($key)
    {
        if (is_array($key)) {
            foreach ($key as $k => $v) {
                $key[$k] = $this->resolve($v);
            }
        } elseif (is_string($key)) {
            $matches = [];

            // Search for any {placeholders} and replace with their replacement values
            preg_match_all('/{[\s]*?([^\s]+)[\s]*?}/i', $key, $matches, PREG_SET_ORDER);

            foreach ($matches as $match) {

The expression $matches of type null|array<integer,array<integer,string>> is not guaranteed to be traversable. How about adding an additional type check?

                if (array_key_exists($match[1], $this->bindings)) {
                    $key = preg_replace('/'.$match[0].'/', $this->bindings[$match[1]], $key, 1);
                }
            }
        }

        return $key;
    }

    /**
     * Resolves an array of menu items properties.
     *
     * @param \Illuminate\Support\Collection &$items
     *
     * @return void
     */
    protected function resolveItems(Collection &$items)
    {
        $resolver = function ($property) {
            return $this->resolve($property) ?: $property;
        };

        $items->each(function (MenuItem $item) use ($resolver) {
            $item->fill(array_map($resolver, $item->properties));
        });
    }

    /**
     * Add new child menu.
     *
     * @param array $properties
     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    protected function add(array $properties = [])
    {
        $properties['attributes']['id'] = $properties['attributes']['id'] ?? md5(json_encode($properties));
        $this->items->push($item = new MenuItem($properties));

        return $item;
    }

    /**
     * Create new menu with dropdown.
     *
     * @param callable $callback
     * @param string   $title
     * @param int      $order

Should the type for parameter $order not be null|integer?

     * @param string   $icon

Should the type for parameter $icon not be null|string?

     * @param array    $attributes
     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    public function dropdown(callable $callback, string $title, int $order = null, string $icon = null, array $attributes = [])

This line exceeds maximum limit of 120 characters; contains 127 characters

    {
        call_user_func($callback, $item = $this->add(compact('title', 'order', 'icon', 'attributes')));

        return $item;
    }

    /**
     * Register new menu item using registered route.
     *
     * @param string $route

Should the type for parameter $route not be array? Also, consider making the array more specific, something like array<String>, or String[].

     * @param string $title
     * @param int    $order

Should the type for parameter $order not be null|integer?

     * @param string $icon

Should the type for parameter $icon not be null|string?

     * @param array  $attributes
     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    public function route(array $route, string $title, int $order = null, string $icon = null, array $attributes = [])
    {
        return $this->add(compact('route', 'title', 'order', 'icon', 'attributes'));
    }

    /**
     * Register new menu item using url.
     *
     * @param string $url
     * @param string $title
     * @param int    $order

Should the type for parameter $order not be null|integer?

     * @param string $icon

Should the type for parameter $icon not be null|string?

     * @param array  $attributes
     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    public function url(string $url, string $title, int $order = null, string $icon = null, array $attributes = [])
    {
        ! $this->urlPrefix || $url = $this->formatUrl($url);

The expression $this->urlPrefix of type string|null is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

        return $this->add(compact('url', 'title', 'order', 'icon', 'attributes'));
    }

    /**
     * Add new header item.
     *
     * @param string $title
     * @param int    $order

Should the type for parameter $order not be null|integer?

     * @param string $icon

Should the type for parameter $icon not be null|string?

     * @param array  $attributes
     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    public function header(string $title, int $order = null, string $icon = null, array $attributes = [])
    {
        $type = 'header';

        return $this->add(compact('type', 'url', 'title', 'order', 'icon', 'attributes'));
    }

    /**
     * Add new divider item.
     *
     * @param int   $order

Should the type for parameter $order not be null|integer?

     * @param array $attributes
     *
     * @return \Rinvex\Menus\Models\MenuItem
     */
    public function divider(int $order = null, array $attributes = [])
    {
        return $this->add(['type' => 'divider', 'order' => $order, 'attributes' => $attributes]);
    }

    /**
     * Get items count.
     *
     * @return int
     */
    public function count()
    {
        return $this->items->count();
    }

    /**
     * Empty the current menu items.
     *
     * @return $this
     */
    public function destroy()
    {
        $this->items = collect();

        return $this;
    }

    /**
     * Get menu items and order it by 'order' key.
     *
     * @return \Illuminate\Support\Collection
     */
    protected function getOrderedItems()
    {
        return $this->items->sortBy('properties.order');
    }

    /**
     * Render the menu to HTML tag.
     *
     * @param string $presenter

Should the type for parameter $presenter not be null|string?

     * @param bool   $specialSidebar
     *
     * @return string

Should the return type not be \Illuminate\Contracts\View\View|string?

     */
    public function render(string $presenter = null, bool $specialSidebar = false)
    {
        $this->resolveItems($this->items);

        if (! is_null($this->view)) {
            return $this->renderView($presenter, $specialSidebar);
        }

        (! $presenter || ! $this->presenterExists($presenter)) || $this->setPresenter($presenter);

The expression $presenter of type null|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

        return $this->renderMenu($specialSidebar);
    }

    /**
     * Render menu via view presenter.
     *
     * @param string $view
     * @param bool   $specialSidebar
     *
     * @return \Illuminate\Contracts\View\View
     */
    protected function renderView(string $view, bool $specialSidebar = false)
    {
        return $this->views->make($view, ['items' => $this->getOrderedItems(), 'specialSidebar' => $specialSidebar]);
    }

    /**
     * Render the menu.
     *
     * @param bool $specialSidebar
     *
     * @return string
     */
    protected function renderMenu(bool $specialSidebar = false)
    {
        $presenter = $this->getPresenter();
        $menu = $presenter->getOpenTagWrapper();

        foreach ($this->getOrderedItems() as $item) {
            if ($item->hidden()) {
                continue;
            }

            if ($item->hasChilds()) {
                $menu .= $presenter->getMenuWithDropDownWrapper($item, $specialSidebar);
            } elseif ($item->isHeader()) {
                $menu .= $presenter->getHeaderWrapper($item);
            } elseif ($item->isDivider()) {
                $menu .= $presenter->getDividerWrapper();
            } else {
                $menu .= $presenter->getMenuWithoutDropdownWrapper($item);
            }
        }

        $menu .= $presenter->getCloseTagWrapper();

        return $menu;
    }

    /**
     * Format URL.
     *
     * @param string $url
     *
     * @return string
     */
    protected function formatUrl(string $url)
    {
        $uri = $this->urlPrefix.$url;

        return $uri === '/' ? '/' : ltrim(rtrim($uri, '/'), '/');
    }
}