App\Widgets

Complexity

Total Complexity

6

Size/Duplication

Total Lines	45
Duplicated Lines	13.33 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	0	6	1
A	getAll()	6	20	5

<?php
/**
 * Widget base file.
 *
 * @copyright YetiForce Sp. z o.o.
 * @license   YetiForce Public License 3.0 (licenses/LicenseEN.txt or yetiforce.com)
 * @author    Radosław Skrzypczak <r.skrzypczak@yetiforce.com>
 */

namespace App;

/**
 * Widget class.
 */
final class Widgets
{
	/** @var string Module name. */
	private $moduleName;

	/**
	 * Get instance.
	 *
	 * @param string $moduleName
	 *
	 * @return self

Should the return type not be \self?

	 */
	public static function getInstance(string $moduleName): self
	{
		$instance = new self();
		$instance->moduleName = $moduleName;
		return $instance;
	}

	/**
	 * Get all widgets.
	 *
	 * @return array
	 */
	public function getAll(): array
	{
		$widgets = [];
		if (\App\Cache::has('Widgets', $this->moduleName)) {

This code seems to be duplicated across your project.

			$widgetsData = \App\Cache::get('Widgets', $this->moduleName);
		} else {
			$widgetsData = \App\Api::getInstance()->call("{$this->moduleName}/Widgets") ?: [];
			\App\Cache::save('Widgets', $this->moduleName, $widgetsData, \App\Cache::LONG);
		}
		foreach ($widgetsData as $widgetData) {
			if (!\in_array($widgetData['type'], ['RelatedModule'])) {
				continue;
			}
			$handlerModule = \App\Loader::getModuleClassName($this->moduleName, 'Widget', $widgetData['type']);
			$widget = new $handlerModule($this->moduleName);
			$widget->setData($widgetData);
			$widgets[$widgetData['id']] = $widget;
		}
		return $widgets;
	}
}