silverstripe / silverstripe-elemental

src/Models/ElementContent.php

<?php

namespace DNADesign\Elemental\Models;

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

/**
 * @property string $HTML
 */
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) {
            /** @var HTMLEditorField $editorField */
            $editorField = $fields->fieldByName('Root.Main.HTML');

Are you sure the assignment to $editorField is correct as $fields->fieldByName('Root.Main.HTML') targeting SilverStripe\Forms\FieldList::fieldByName() seems to always return null.

            $editorField->setTitle(_t(__CLASS__ . '.ContentLabel', 'Content'));
        });

        return parent::getCMSFields();
    }

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

    protected function provideBlockSchema()
    {
        $blockSchema = parent::provideBlockSchema();
        $blockSchema['content'] = $this->getSummary();
        return $blockSchema;
    }

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