ContentBlock_Accordion

Complexity

Total Complexity

4

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	getCMSFields()	0	24	1
A	getBlockType()	0	4	1
A	AccordionClasses()	0	8	2

<?php

/**
 * Models a Content Block Accordion
 *
 * @since 1.0.0
 */
class ContentBlock_Accordion 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.

        'AccordionTitle' => 'Text',
        'AccordionSummary' => 'Text',
        'AccordionContent' => 'HTMLText',
    ];

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

        $fields->push(
            TextField::create('AccordionTitle')
            ->setDescription('Title line for the Accordion')
        );

        $fields->push(
            TextareaField::create('AccordionSummary')
            ->setDescription('Custom Summary for the Accordion')
        );

        $fields->push(
            HTMLEditorField::create('AccordionContent')
            ->setDescription('Content in the accordion')
            ->setRows(20)
        );

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

        return $fields;
    }

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

    /**
     * Gets the configured classes for a column
     *
     * @return string
     */
    public function AccordionClasses()
    {
        $classesArray = Config::inst()->get('ContentBlock_Accordion', 'accordion_classes');

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

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

}