EnvBarSiteConfigExtension::updateCMSFields() - Code Metrics - Inspection of "Feat/control display via cms settings" - signify-nz/silverstripe-environmentindicator - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#9)

unknown

created 2020-12-15 20:45 UTC

EnvBarSiteConfigExtension::updateCMSFields() A

↳ Parent: EnvBarSiteConfigExtension

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

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

<?php

namespace Signify\EnvBar\Extensions;

use BadMethodCallException;
use SilverStripe\Forms\CheckboxField;
use SilverStripe\Forms\FieldList;
use SilverStripe\ORM\DataExtension;

/**
 * Add override boolean to the site config settings
 * @see SilverStripe\SiteConfig\SiteConfig
 * to allow CMS admin to disable automatic display of the Environment Indicator
 * bar on the frontend pages.
 *
 * @author Lani Field <[email protected]>
 * @version 1.1.0
 * @package Signify\EnvBar\Extensions
 */
class EnvBarSiteConfigExtension extends DataExtension
{
    /**
     * Add override column to SiteConfig db record.
     *
     * @var string[]
     */
    private static $db = [

        'EnvBarDisplay' => 'Boolean',
    ];

    /**
     * Add Checkbox field for override value to new Environment Indicator tab
     * in CMS Settings.
     *
     * @param FieldList $fields
     * @return void
     * @throws BadMethodCallException
     */
    public function updateCMSFields(FieldList $fields)
    {
        $fields->addFieldsToTab('Root.EnvironmentIndicator', [
            CheckboxField::create(
                'EnvBarDisplay',
                'Display the environment indicator bar?'
            )->setDescription('Check to show the environment indicator bar on'
                . ' all pages being viewed in this environment.'),
        ]);
    }
}


1			<?php
2
3			namespace Signify\EnvBar\Extensions;
4
5			use BadMethodCallException;
6			use SilverStripe\Forms\CheckboxField;
7			use SilverStripe\Forms\FieldList;
8			use SilverStripe\ORM\DataExtension;
9
10			/**
11			* Add override boolean to the site config settings
12			* @see SilverStripe\SiteConfig\SiteConfig
13			* to allow CMS admin to disable automatic display of the Environment Indicator
14			* bar on the frontend pages.
15			*
16			* @author Lani Field <[email protected]>
17			* @version 1.1.0
18			* @package Signify\EnvBar\Extensions
19			*/
20			class EnvBarSiteConfigExtension extends DataExtension
21			{
22			/**
23			* Add override column to SiteConfig db record.
24			*
25			* @var string[]
26			*/
27			private static $db = [
			0 ignored issues – show introduced 2020-06-23 05:26 UTC by Report Bug Copy Issue Report The private property `$db` is not used, and could be removed. Loading history...
28			'EnvBarDisplay' => 'Boolean',
29			];
30
31			/**
32			* Add Checkbox field for override value to new Environment Indicator tab
33			* in CMS Settings.
34			*
35			* @param FieldList $fields
36			* @return void
37			* @throws BadMethodCallException
38			*/
39			public function updateCMSFields(FieldList $fields)
40			{
41			$fields->addFieldsToTab('Root.EnvironmentIndicator', [
42			CheckboxField::create(
43			'EnvBarDisplay',
44			'Display the environment indicator bar?'
45			)->setDescription('Check to show the environment indicator bar on'
46			. ' all pages being viewed in this environment.'),
47			]);
48			}
49			}
50

signify-nz / silverstripe-environmentindicator

Pull Request — master (#9)

EnvBarSiteConfigExtension::updateCMSFields() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like