Epesi\Base\Dashboard\Integration\DashboardNavMenu

Complexity

Total Complexity

3

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	items()	0	18	3

<?php 

namespace Epesi\Base\Dashboard\Integration;

use Epesi\Base\Layout\Integration\Joints\NavMenuJoint;
use Epesi\Base\Dashboard\Database\Models\Dashboard;
use Illuminate\Support\Facades\Auth;
use Epesi\Core\Integration\ModuleView;

The type Epesi\Core\Integration\ModuleView was not found. Maybe you did not declare it correctly or list all dependencies?

class DashboardNavMenu extends NavMenuJoint
{
	public function items()
	{
		$ret = [];
		
		foreach (Dashboard::where(['user_id' => Auth::user()->id])->orderBy('position')->get() as $dashboard) {

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

			$ret[$dashboard->name] = [
					'access' => true,
					'weight' => $dashboard['position'],
					'link' => ModuleView::moduleLink('dashboard', 'body', ['dashboard' => $dashboard->id])
			];
		}

		return [
				__('DASHBOARD') => count($ret) > 1? [
						'access' => true,
						'group' => $ret,
						'weight' => -10000
				]: array_merge(reset($ret), ['weight' => -10000]),
		];
	}
}