ContentBlock_TileSection::getCMSFields() - Code Metrics - SomarDesignStudios/silverstripe-contentblocks - Measure and Improve Code Quality continuously with Scrutinizer

ContentBlock_TileSection::getCMSFields() B
last analyzed 2017-06-19 21:54 UTC

↳ Parent: ContentBlock_TileSection

Complexity

Conditions	2
Paths	2

Size

Total Lines	37
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	37
rs	8.8571
c	0
b	0
f	0
cc	2
eloc	25
nc	2
nop	0

<?php

/**
 * Models a Content Block Tile Section
 *
 * @since 1.0.0
 */
class ContentBlock_TileSection extends ContentBlock
namespace YourVendor;

class YourClass { }
{
    private static $db = [

        'TilesPerRow' => 'Enum("One, Two, Three")',
        'TileTemplate' => 'Text'
    ];

    private static $has_many = [

        'ContentTiles' => 'TileSection_ContentTile',
    ];

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        $fields->push(DropdownField::create(
            'TilesPerRow',
            'Tiles Per Row',
            singleton('ContentBlock_TileSection')
            ->dbObject('TilesPerRow')
            ->enumValues()
        ));

        $fields->push(DropdownField::create(
            'TileTemplate',
            'Tile Template',
            $this->getAvailableTemplates()
        ));

        if ($this->getField('ID')) {
            $fields->push(GridField::create(
                'ContentTiles',
                'Content Tiles',
                $this->ContentTiles(),
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
                GridFieldConfig_RecordEditor::create()
                ->addComponent(new GridFieldOrderableRows('Sort'))
            ));
        } else {
            $fields->push(LiteralField::create(
                'Unsaved',
                '<p class="message info">Please save first, to begin adding Tiles </p>'
            ));
        }

        $this->extend('updateCMSFields', $fields);

        return $fields;
    }

    /**
     * Gets available templates from the static configuration
     *
     * @return array
     */
     public function getAvailableTemplates()
     {
         $templates = Config::inst()->get('ContentBlock_TileSection', 'tile_templates');

         $this->extend('updateAvailableTemplates', $templates);

         return (!empty($templates)) ? $templates : [] ;
     }


    /**
     * Gets the block type for CMS display
     *
     * @return string
     */
    public function getBlockType()
    {
        return 'Tile Section';
    }
}


1			<?php
2
3			/**
4			* Models a Content Block Tile Section
5			*
6			* @since 1.0.0
7			*/
8			class ContentBlock_TileSection extends ContentBlock
			0 ignored issues – show Coding Style Compatibility introduced 2016-10-14 02:29 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			private static $db = [
			0 ignored issues – show Comprehensibility introduced 2016-10-14 02:29 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2016-10-14 02:29 UTC by Report Bug Copy Issue Report The property `$db` 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...
11			'TilesPerRow' => 'Enum("One, Two, Three")',
12			'TileTemplate' => 'Text'
13			];
14
15			private static $has_many = [
			0 ignored issues – show Comprehensibility introduced 2016-10-14 02:29 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2016-10-14 02:29 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...
16			'ContentTiles' => 'TileSection_ContentTile',
17			];
18
19			public function getCMSFields()
20			{
21			$fields = parent::getCMSFields();
22
23			$fields->push(DropdownField::create(
24			'TilesPerRow',
25			'Tiles Per Row',
26			singleton('ContentBlock_TileSection')
27			->dbObject('TilesPerRow')
28			->enumValues()
29			));
30
31			$fields->push(DropdownField::create(
32			'TileTemplate',
33			'Tile Template',
34			$this->getAvailableTemplates()
35			));
36
37			if ($this->getField('ID')) {
38			$fields->push(GridField::create(
39			'ContentTiles',
40			'Content Tiles',
41			$this->ContentTiles(),
			0 ignored issues – show Documentation Bug introduced 2016-10-14 02:29 UTC by Report Bug Copy Issue Report The method `ContentTiles` does not exist on `object<ContentBlock_TileSection>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
42			GridFieldConfig_RecordEditor::create()
43			->addComponent(new GridFieldOrderableRows('Sort'))
44			));
45			} else {
46			$fields->push(LiteralField::create(
47			'Unsaved',
48			'<p class="message info">Please save first, to begin adding Tiles </p>'
49			));
50			}
51
52			$this->extend('updateCMSFields', $fields);
53
54			return $fields;
55			}
56
57			/**
58			* Gets available templates from the static configuration
59			*
60			* @return array
61			*/
62			public function getAvailableTemplates()
63			{
64			$templates = Config::inst()->get('ContentBlock_TileSection', 'tile_templates');
65
66			$this->extend('updateAvailableTemplates', $templates);
67
68			return (!empty($templates)) ? $templates : [] ;
69			}
70
71
72			/**
73			* Gets the block type for CMS display
74			*
75			* @return string
76			*/
77			public function getBlockType()
78			{
79			return 'Tile Section';
80			}
81			}
82

SomarDesignStudios / silverstripe-contentblocks

GitHub Access Token became invalid

ContentBlock_TileSection::getCMSFields() B last analyzed 2017-06-19 21:54 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ContentBlock_TileSection::getCMSFields() B
last analyzed 2017-06-19 21:54 UTC