Epesi\Core\System\Modules\Concerns\HasLocation

Complexity

Total Complexity

4

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	label()	0	9	2
A	location()	0	11	2

<?php

namespace Epesi\Core\System\Modules\Concerns;

trait HasLocation
{
	use HasModule;
	
	/**
	 * Define the label for the module base location caption
	 * e.g Dashboard >> View, you should define $label = 'Dashboard'
	 * 
	 * @var string
	 */
	protected $label = '';
	
	/**
	 * Stores detailed labels for location caption to be displayed
	 * e.g Dashboard >> Edit >> Layout, you should set $location = ['Edit', 'Layout']
	 * 
	 * @var array
	 */
	protected $location = [];
	
	/**
	 * Get or set the location caption displayed
	 * 
	 * @param string | array $location
	 * @return \Epesi\Core\System\Modules\ModuleView|array
	 */
	public function location($location = null)
	{
		if ($location) {
			$this->location = $location;

It seems like $location can also be of type string. However, the property $location is declared as type array. Maybe add an additional type check?

			
			return $this;
		}
		
		$this->label = array_map('trans', (array) $this->label);

It seems like array_map('trans', (array)$this->label) of type array is incompatible with the declared type string of property $label.

		return array_merge($this->label, (array) $this->location);
	}
	
	public function label($label)
	{
		if ($label) {
			$this->label = $label;
			
			return $this;
		}
		
		return $this->label;
	}
}