silverstripe / silverstripe-content-widget

code/ContentWidget.php

<?php

/**
 * Content Widget
 *
 * @package silverstripe-content-widget
 */
class ContentWidget extends Widget

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

{
    /**
     * @var array
     */
    public static $db = array(
        "HTML" => "HTMLText"
    );

    /**
     * @var string
     */
    private static $title = "HTML Content";

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

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

    /**
     * @var string
     */
    private static $cmsTitle = "HTML Content";

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

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

    /**
     * @var string
     */
    private static $description = "Custom HTML content widget.";

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

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

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        $fields->push(TextField::create("Title"));
        $fields->push(HtmlEditorField::create("HTML", "Content"));

        return $fields;
    }
}