HomePage::getCMSFields() - Code Metrics - wernerkrauss/silverstripe-homepage - Measure and Improve Code Quality continuously with Scrutinizer

HomePage::getCMSFields() B
last analyzed 2016-03-23 17:13 UTC

↳ Parent: HomePage

Complexity

Conditions	1
Paths	1

Size

Total Lines	27
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	27
rs	8.8571
cc	1
eloc	14
nc	1
nop	0

<?php

/**
 * StartGeneratedWithDataObjectAnnotator
 * @method DataList|HomePageBlock[] HomePageBlocks
 * EndGeneratedWithDataObjectAnnotator
 */
class HomePage extends Page
namespace YourVendor;

class YourClass { }
{

    private static $has_many = [

        'HomePageBlocks' => 'HomePageBlock'
    ];

    public function getCMSFields()

    {
        $self =& $this;
        $this->beforeUpdateCMSFields(function (FieldList $fields) use ($self) {

            /**
             * @var GridFieldConfig_RecordEditor $conf
             */
            $conf = GridFieldConfig_RecordEditor::create();

            $conf->addComponent(new GridFieldSortableRows('SortOrder'));

            $fields->addFieldToTab(
                "Root." . _t('HomePage.BlockTabName', 'Blocks'),
                Gridfield::create(
                    'HomePageBlocks',
                    _t('HomePage.Block', 'Blocks'),
                    $this->HomePageBlocks(),
                    $conf
                )
            );
        });

        $fields = parent::getCMSFields();

        return $fields;
    }
}

class HomePage_Controller extends Page_Controller
namespace YourVendor;

class YourClass { }
{
    public function getActiveHomePageBlocks()

    {
        $blocks = $this->data()->HomePageBlocks();

        if (Versioned::current_stage() === 'Live') {
            $blocks = $blocks->filter('IsActive', 1);
        }

        return $blocks;
    }
}


1			<?php
2
3			/**
4			* StartGeneratedWithDataObjectAnnotator
5			* @method DataList\|HomePageBlock[] HomePageBlocks
6			* EndGeneratedWithDataObjectAnnotator
7			*/
8			class HomePage extends Page
			0 ignored issues – show Coding Style Compatibility introduced 2016-03-23 17:11 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
9			{
10
11			private static $has_many = [
			0 ignored issues – show Unused Code introduced 2016-03-23 17:11 UTC by Report Bug Copy Issue Report The property `$has_many` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
12			'HomePageBlocks' => 'HomePageBlock'
13			];
14
15			public function getCMSFields()
			0 ignored issues – show Documentation introduced 2016-03-23 17:11 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
16			{
17			$self =& $this;
18			$this->beforeUpdateCMSFields(function (FieldList $fields) use ($self) {
19
20			/**
21			* @var GridFieldConfig_RecordEditor $conf
22			*/
23			$conf = GridFieldConfig_RecordEditor::create();
24
25			$conf->addComponent(new GridFieldSortableRows('SortOrder'));
26
27			$fields->addFieldToTab(
28			"Root." . _t('HomePage.BlockTabName', 'Blocks'),
29			Gridfield::create(
30			'HomePageBlocks',
31			_t('HomePage.Block', 'Blocks'),
32			$this->HomePageBlocks(),
33			$conf
34			)
35			);
36			});
37
38			$fields = parent::getCMSFields();
39
40			return $fields;
41			}
42			}
43
44			class HomePage_Controller extends Page_Controller
			0 ignored issues – show Coding Style Compatibility introduced 2016-03-23 17:11 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2016-03-23 17:11 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
45			{
46			public function getActiveHomePageBlocks()
			0 ignored issues – show Documentation introduced 2016-03-23 17:11 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
47			{
48			$blocks = $this->data()->HomePageBlocks();
49
50			if (Versioned::current_stage() === 'Live') {
51			$blocks = $blocks->filter('IsActive', 1);
52			}
53
54			return $blocks;
55			}
56			}
57

wernerkrauss / silverstripe-homepage

HomePage::getCMSFields() B last analyzed 2016-03-23 17:13 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

HomePage::getCMSFields() B
last analyzed 2016-03-23 17:13 UTC