DNADesign\Elemental\Models\ElementContent

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

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

<?php

namespace DNADesign\Elemental\Models;

use SilverStripe\Forms\FieldList;
use SilverStripe\ORM\FieldType\DBField;

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

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

    private static $db = [

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

        'HTML' => 'HTMLText'
    ];

    private static $table_name = 'ElementContent';

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

    private static $singular_name = 'content block';

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

    private static $plural_name = 'content blocks';

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

    private static $description = 'HTML text block';

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

    /**
     * Re-title the HTML field to Content
     *
     * {@inheritDoc}
     */
    public function getCMSFields()
    {
        $this->beforeUpdateCMSFields(function (FieldList $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 getSummaryData()
    {
        $summaryData = [];
        $summaryData['content'] = $this->getSummary();
        return $summaryData;
    }

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