dnadesign / silverstripe-elemental

code/controllers/ElementalAdmin.php

<?php

/**
 * @package elemental
 */
class ElementalAdmin extends ModelAdmin {

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

    private static $managed_models = array(

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

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

        'BaseElement'
    );

    private static $menu_title = 'Content Blocks';

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

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

    private static $url_segment = 'elemental';

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

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

    private static $menu_icon = "elemental/images/blocks.svg";

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

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

    public function getEditForm($id = null, $fields = null) {

The return type could not be reliably inferred; please add a @return annotation.

        $form = parent::getEditForm($id, $fields);

        $grid = $form->Fields()
            ->dataFieldByName($this->sanitiseClassName($this->modelClass));

        $grid->getConfig()->removeComponentsByType('GridFieldAddNewButton');

        return $form;
    }

    /**
     * Exclude our linked elements
     *
     * @return DataList
     */
    public function getList() {
        $list = parent::getList();
        $list = $list->exclude('ClassName', 'ElementVirtualLinked');

        return $list;
    }
}