InheritableSettingsModel::__set() - Code Metrics - Inspection of "Merge branch 'feature/seo-settings-field-ux' into..." - nystudio107/craft-seomatic - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( d05d13...055ab5 )

by Andrew

created 2021-08-16 22:45 UTC

InheritableSettingsModel::__set() A

↳ Parent: InheritableSettingsModel

Complexity

Conditions	4
Paths	4

Size

Total Lines	19
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	11
nc	4
nop	2
dl	0
loc	19
ccs	0
cts	11
cp	0
crap	20
rs	9.9
c	1
b	0
f	0

<?php
/**
 * SEOmatic plugin for Craft CMS 3.x
 *
 * A turnkey SEO implementation for Craft CMS that is comprehensive, powerful,
 * and flexible
 *
 * @link      https://nystudio107.com
 * @copyright Copyright (c) 2017 nystudio107
 */

namespace nystudio107\seomatic\base;

use craft\helpers\StringHelper;

/**
 * InheritableSettingsModel allows keeping track of inherited/overridden settings.
 *
 * @author    nystudio107

 * @package   Seomatic
 * @since     3.4.0
 */
abstract class InheritableSettingsModel extends VarsModel
{

    /**
     * @var array A list of all the settings for which the inherited values should be used.
     */
    public $inherited = [];

    /**
     * @var array A list of all the settings for which the inherited values are overridden.
     */
    public $overrides = [];

    /**
     * Keep track of which settings to use the override for.
     *
     * @param string $name

     * @param mixed $value

     * @throws \yii\base\UnknownPropertyException

     */
    public function __set($name, $value)
    {
        if (StringHelper::startsWith($name, 'override-')) {
            $remainder = StringHelper::removeLeft($name, 'override-');

            if ($remainder) {
                if ($value) {
                    $this->overrides[$remainder] = true;
                    unset($this->inherited[$remainder]);
                } else {
                    $this->inherited[$remainder] = true;
                    unset($this->overrides[$remainder]);
                }
            }

            return;
        }

        parent::__set($name, $value);
    }
}


1			<?php
2			/**
3			* SEOmatic plugin for Craft CMS 3.x
4			*
5			* A turnkey SEO implementation for Craft CMS that is comprehensive, powerful,
6			* and flexible
7			*
8			* @link https://nystudio107.com
9			* @copyright Copyright (c) 2017 nystudio107
10			*/
11
12			namespace nystudio107\seomatic\base;
13
14			use craft\helpers\StringHelper;
15
16			/**
17			* InheritableSettingsModel allows keeping track of inherited/overridden settings.
18			*
19			* @author nystudio107
			0 ignored issues – show Coding Style introduced 2021-08-16 23:02 UTC by Report Bug Copy Issue Report The tag in position 1 should be the @package tag Loading history... Coding Style introduced 2021-08-16 23:02 UTC by Report Bug Copy Issue Report Content of the @author tag must be in the form "Display Name <[email protected]>" Loading history...
20			* @package Seomatic
21			* @since 3.4.0
22			*/
23			abstract class InheritableSettingsModel extends VarsModel
24			{
25
26			/**
27			* @var array A list of all the settings for which the inherited values should be used.
28			*/
29			public $inherited = [];
30
31			/**
32			* @var array A list of all the settings for which the inherited values are overridden.
33			*/
34			public $overrides = [];
35
36			/**
37			* Keep track of which settings to use the override for.
38			*
39			* @param string $name
			0 ignored issues – show Coding Style introduced 2021-08-16 23:02 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
40			* @param mixed $value
			0 ignored issues – show Coding Style introduced 2021-08-16 23:02 UTC by Report Bug Copy Issue Report Expected 2 spaces after parameter type; 1 found Loading history... Coding Style introduced 2021-08-16 23:02 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
41			* @throws \yii\base\UnknownPropertyException
			0 ignored issues – show Coding Style introduced 2021-08-16 23:02 UTC by Report Bug Copy Issue Report Tag @throws cannot be grouped with parameter tags in a doc comment Loading history...
42			*/
43			public function __set($name, $value)
44			{
45			if (StringHelper::startsWith($name, 'override-')) {
46			$remainder = StringHelper::removeLeft($name, 'override-');
47
48			if ($remainder) {
49			if ($value) {
50			$this->overrides[$remainder] = true;
51			unset($this->inherited[$remainder]);
52			} else {
53			$this->inherited[$remainder] = true;
54			unset($this->overrides[$remainder]);
55			}
56			}
57
58			return;
59			}
60
61			parent::__set($name, $value);
62			}
63			}
64

nystudio107 / craft-seomatic

Push — develop ( d05d13...055ab5 )

InheritableSettingsModel::__set() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like