ContentBlock_ColumnLayout

Complexity

Total Complexity

9

Size/Duplication

Total Lines	143
Duplicated Lines	11.19 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	63	1
A	ColumnColour()	0	6	1
A	IsLayout()	0	3	1
A	ColumnClasses()	8	8	2
A	RowClasses()	8	8	2
A	getBlockType()	0	4	1
A	RenderBlock()	0	5	1

<?php

/**
 * Models a Content Block Column Layout
 *
 * @since 1.0.0
 */
class ContentBlock_ColumnLayout extends ContentBlock

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{

    private static $db = [

Consider using a different property name as you override a private property of the parent class.

The property $db is not used and could be removed.

        'Layout' => 'Enum("One Column, Two Column, Three Column")',
        'ColumnOneContent' => 'HTMLText',
        'ColumnTwoContent' => 'HTMLText',
        'ColumnThreeContent' => 'HTMLText',
        'ColumnOneColour' => 'Varchar',
        'ColumnTwoColour' => 'Varchar',
        'ColumnThreeColour' => 'Varchar',
    ];

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

        $fields->push(
            DropdownField::create('Layout')
            ->setSource(
                singleton('ContentBlock_ColumnLayout')
                ->dbObject('Layout')
                ->enumValues()
            )
        );

        $colourSource = Config::inst()->get('ContentBlock_ColumnLayout', 'colour_options');

        //Column One -------
        $columnOne = DisplayLogicWrapper::create(CompositeField::create([
             HeaderField::create('ColumnOne', 'Column One'),
             DropdownField::create(
                 'ColumnOneColour',
                 'Colour',
                 $colourSource
             ),
             HTMLEditorField::create('ColumnOneContent', 'Content'),
        ]));
        //Display if One Column || Two Column || Three Column
        $columnOne->displayIf('Layout')->isEqualTo('One Column')
            ->orIf('Layout')->isEqualTo('Two Column')
            ->orIf('Layout')->isEqualTo('Three Column');
        $fields->push($columnOne);

        //Column Two -------
        //Display if Two Column
        $columnTwo = DisplayLogicWrapper::create(CompositeField::create([
             HeaderField::create('ColumnTwo', 'Column Two'),
             DropdownField::create(
                 'ColumnTwoColour',
                 'Colour',
                 $colourSource
             ),
             HTMLEditorField::create('ColumnTwoContent', 'Content'),
        ]));
        $columnTwo->displayIf('Layout')->isEqualTo('Two Column')
            ->orIf('Layout')->isEqualTo('Three Column');
        $fields->push($columnTwo);

        //Column Three -------
        $columnThree =  DisplayLogicWrapper::create(CompositeField::create([
             HeaderField::create('ColumnThree', 'Column Three'),
             DropdownField::create(
                 'ColumnThreeColour',
                 'Colour',
                 $colourSource
             ),
             HTMLEditorField::create('ColumnThreeContent', 'Content'),
        ]));
        $columnThree->displayIf('Layout')->isEqualTo('Three Column');
        $fields->push($columnThree);

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

        return $fields;
    }

    /**
     * Get the colour for a column
     *
     * @return string
     */
    public function ColumnColour($column)
    {
        $property = sprintf('Column%sColour', $column);
        $colour = $this->getField($property);
        return sprintf('#%s', $colour);
    }

    public function IsLayout($layout){
        return ($this->getField('Layout') == $layout);
    }

    /**
     * Gets the configured classes for a column
     *
     * @return string
     */
    public function ColumnClasses()

This method seems to be duplicated in your project.

    {
        $classesArray = Config::inst()->get('ContentBlock_ColumnLayout', 'column_classes');

        $this->extend('updateColumnClasses', $classes);

        return (!empty($classesArray)) ? implode(' ', $classesArray) : '' ;
    }

    /**
     * Gets the configured classes for a row
     *
     * @return string
     */
    public function RowClasses()

This method seems to be duplicated in your project.

    {
        $classesArray = Config::inst()->get('ContentBlock_ColumnLayout', 'row_classes');

        $this->extend('updateRowClasses', $classes);

        return (!empty($classesArray)) ? implode(' ', $classesArray) : '' ;
    }

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

    /**
     * Renders the block for template
     *
     * @return string
     */
    public function RenderBlock()
    {
        Requirements::javascript(CONTENTBLOCKS_DIR . '/javascript/lib/jquery.matchHeight-min.js');
        return parent::RenderBlock();
    }

}