Configurable::setConfigRepository() - Code Metrics - Inspection of "Update ConfigurableInterface." - figtree-php/config - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — merge-configs (cba865)

by Alan

created 2021-05-05 15:49 UTC

Configurable::setConfigRepository() A

↳ Parent: Configurable

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	5
rs	10
c	0
b	0
f	0

<?php

namespace FigTree\Config\Concerns;

use FigTree\Config\Contracts\{
	ConfigRepositoryInterface,
	ConfigInterface,
	ConfigurableInterface,
};

trait Configurable
{
	/**
	 * ConfigRepository instance.
	 *
	 * @var \FigTree\Config\Contracts\ConfigRepositoryInterface
	 */
	protected ConfigRepositoryInterface $configRepo;

	/**
	 * Set the Config instance.
	 *
	 * @param \FigTree\Config\Contracts\ConfigRepositoryInterface $configRepo
	 *
	 * @return $this
	 */
	public function setConfigRepository(ConfigRepositoryInterface $configRepo): ConfigurableInterface
	{
		$this->configRepo = $configRepo;

		return $this;

	}

	/**
	 * Shorthand to either get the associated Config instance or the given
	 * top-level key from the Config file.
	 *
	 * @param string $fileName
	 *
	 * @return \FigTree\Config\Contracts\ConfigInterface
	 */
	protected function config(string $fileName): ConfigInterface
	{
		return $this->configRepo->get($fileName);
	}
}


1			<?php
2
3			namespace FigTree\Config\Concerns;
4
5			use FigTree\Config\Contracts\{
6			ConfigRepositoryInterface,
7			ConfigInterface,
8			ConfigurableInterface,
9			};
10
11			trait Configurable
12			{
13			/**
14			* ConfigRepository instance.
15			*
16			* @var \FigTree\Config\Contracts\ConfigRepositoryInterface
17			*/
18			protected ConfigRepositoryInterface $configRepo;
19
20			/**
21			* Set the Config instance.
22			*
23			* @param \FigTree\Config\Contracts\ConfigRepositoryInterface $configRepo
24			*
25			* @return $this
26			*/
27			public function setConfigRepository(ConfigRepositoryInterface $configRepo): ConfigurableInterface
28			{
29			$this->configRepo = $configRepo;
30
31			return $this;
			0 ignored issues – show Bug Best Practice introduced 2021-05-05 15:52 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `FigTree\Config\Concerns\Configurable` which is incompatible with the type-hinted return `FigTree\Config\Contracts\ConfigurableInterface`. Loading history...
32			}
33
34			/**
35			* Shorthand to either get the associated Config instance or the given
36			* top-level key from the Config file.
37			*
38			* @param string $fileName
39			*
40			* @return \FigTree\Config\Contracts\ConfigInterface
41			*/
42			protected function config(string $fileName): ConfigInterface
43			{
44			return $this->configRepo->get($fileName);
45			}
46			}
47

figtree-php / config

Branch — merge-configs (cba865)

Configurable::setConfigRepository() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like