Nwidart\Menus\Presenters\Bootstrap\NavbarRightPresenter

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	48.15 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getOpenTagWrapper()	0	3	1
A	getMenuWithDropDownWrapper()	0	12	1

<?php

namespace KyleMassacre\Menus\Presenters\Bootstrap;

use KyleMassacre\Menus\Contracts\MenuItemContract;

class NavbarRightPresenter extends NavbarPresenter
{
    /**
     * {@inheritdoc}
     */
    public function getOpenTagWrapper(): ?string
    {
        return PHP_EOL . '<ul class="nav navbar-nav navbar-right">' . PHP_EOL;
    }

    /**
     * {@inheritdoc}
     */
    public function getMenuWithDropDownWrapper(MenuItemContract $item): ?string
    {
        return '<li class="dropdown pull-right">
			      <a href="#" class="dropdown-toggle" data-toggle="dropdown">
					' . $item->getIcon() . ' ' . $item->title . '

The call to KyleMassacre\Menus\Contracts\MenuItemContract::getIcon() has too few arguments starting with default.

The property title does not exist on KyleMassacre\Menus\Contracts\MenuItemContract. Since you implemented __get, consider adding a @property annotation.

			      	<b class="caret"></b>
			      </a>
			      <ul class="dropdown-menu">
			      	' . $this->getChildMenuItems($item) . '
			      </ul>
		      	</li>'
        . PHP_EOL;
    }
}