FeatureManager::enabled() - Code Metrics - Inspection of "Make it pass on testing" - m1guelpf/laravel-feature - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e48d2d...d89272 )

by Miguel

created 2018-04-06 11:18 UTC

FeatureManager::enabled() B

↳ Parent: FeatureManager

Complexity

Conditions	5
Paths	5

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	9
nc	5
nop	2
dl	0
loc	18
rs	8.8571
c	0
b	0
f	0

<?php

namespace M1guelpf\Feature;

class FeatureManager
{
    /** @var bool|null */
    protected static $masterSwitch = null;

    /**
     * Create a new FeatureManager Instance.
     *
     * @param array $features Features to load
     *
     * @return void

     */
    public function __construct(array $features = [])
    {
        $this->features = $features;
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

    /**
     * Check if a feature is enabled.
     *
     * @param string $name Name of the feature to check
     * @param string $function Check for a specific function
     *
     * @return bool
     */
    public function enabled(string $name, string $function = '') : bool
    {
        if (! is_null(static::$masterSwitch)) {
            return static::$masterSwitch;
        }
        
        if ($this->runningTests()) {
            return true;
        }

        $feature = $this->feature($name);

        if (! empty(trim($function))) {
            return is_array($feature) ? value(array_get($feature, $function, false)) : (bool) value($feature);
        }

        return (bool) value($feature);
    }

    /**
     * Get feature options.
     *
     * @param string $name Name of the feature to check
     *
     * @return bool
     */
    public function feature(string $name)
    {
        return array_get($this->features, $name, false);
    }

    /**
     * Set the master switch.
     *
     * @param bool $value Value to set the master switch to
     *
     * @return void
     */
    public static function setMasterSwitch(bool $value)
    {
        static::$masterSwitch = $value;
    }
    
    /**
     * Check if the current app instance is running unit tests.
     *
     * @return bool
     */
    public function runningTests()
    {
        return function_exists('app') && method_exists($app = app(), 'runningUnitTests') && $app->runningUnitTests();
    }
}


1			<?php
2
3			namespace M1guelpf\Feature;
4
5			class FeatureManager
6			{
7			/** @var bool\|null */
8			protected static $masterSwitch = null;
9
10			/**
11			* Create a new FeatureManager Instance.
12			*
13			* @param array $features Features to load
14			*
15			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2018-02-04 18:50 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
16			*/
17			public function __construct(array $features = [])
18			{
19			$this->features = $features;
			0 ignored issues – show Bug introduced 2018-02-04 18:50 UTC by Report Bug Copy Issue Report The property `features` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
20			}
21
22			/**
23			* Check if a feature is enabled.
24			*
25			* @param string $name Name of the feature to check
26			* @param string $function Check for a specific function
27			*
28			* @return bool
29			*/
30			public function enabled(string $name, string $function = '') : bool
31			{
32			if (! is_null(static::$masterSwitch)) {
33			return static::$masterSwitch;
34			}
35
36			if ($this->runningTests()) {
37			return true;
38			}
39
40			$feature = $this->feature($name);
41
42			if (! empty(trim($function))) {
43			return is_array($feature) ? value(array_get($feature, $function, false)) : (bool) value($feature);
44			}
45
46			return (bool) value($feature);
47			}
48
49			/**
50			* Get feature options.
51			*
52			* @param string $name Name of the feature to check
53			*
54			* @return bool
55			*/
56			public function feature(string $name)
57			{
58			return array_get($this->features, $name, false);
59			}
60
61			/**
62			* Set the master switch.
63			*
64			* @param bool $value Value to set the master switch to
65			*
66			* @return void
67			*/
68			public static function setMasterSwitch(bool $value)
69			{
70			static::$masterSwitch = $value;
71			}
72
73			/**
74			* Check if the current app instance is running unit tests.
75			*
76			* @return bool
77			*/
78			public function runningTests()
79			{
80			return function_exists('app') && method_exists($app = app(), 'runningUnitTests') && $app->runningUnitTests();
81			}
82			}
83

m1guelpf / laravel-feature

Push — master ( e48d2d...d89272 )

FeatureManager::enabled() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like