DNADesign\Elemental\Models\ElementContent

Complexity

Total Complexity

3

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	8	1
A	getType()	0	3	1
A	getSummary()	0	3	1

<?php

namespace DNADesign\Elemental\Models;

use SilverStripe\Forms\HTMLEditor\HtmlEditorField;
use SilverStripe\Forms\DropdownField;
use SilverStripe\ORM\FieldType\DBField;

class ElementContent extends BaseElement
{
    private static $icon = 'font-icon-block-content';

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

    private static $db = [

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

        'HTML' => 'HTMLText'
    ];

    private static $table_name = 'ElementContent';

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

    private static $singular_name = 'content block';

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

    private static $plural_name = 'content blocks';

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

    private static $description = 'HTML text block';

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

    /**
     * Re-title the HTML field to Content
     *
     * {@inheritDoc}
     */
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function ($fields) {
            $fields
                ->fieldByName('Root.Main.HTML')
                ->setTitle(_t(__CLASS__ . '.ContentLabel', 'Content'));
        });
        return parent::getCMSFields();
    }

    public function getSummary()
    {
        return DBField::create_field('HTMLText', $this->HTML)->Summary(20);

The property HTML does not exist on DNADesign\Elemental\Models\ElementContent. Since you implemented __get, consider adding a @property annotation.

    }

    public function getType()
    {
        return _t(__CLASS__ . '.BlockType', 'Content');
    }
}