silverstripe / silverstripe-widgets

src/Model/WidgetArea.php

<?php

namespace SilverStripe\Widgets\Model;

use SilverStripe\Control\Controller;
use SilverStripe\ORM\ArrayList;
use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\HasManyList;
use SilverStripe\ORM\SS_List;
use SilverStripe\Versioned\Versioned;

/**
 * Represents a set of widgets shown on a page.
 */
class WidgetArea extends DataObject
{
    private static $has_many = [

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

        "Widgets" => Widget::class
    ];

    private static $owns = [

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

        'Widgets',
    ];

    private static $cascade_deletes = [

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

        'Widgets',
    ];

    private static $extensions = [

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

        Versioned::class,
    ];

    private static $table_name = 'WidgetArea';

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

    public $template = __CLASS__;

    /**
     * Used in template instead of {@link Widgets()} to wrap each widget in its
     * controller, making it easier to access and process form logic and
     * actions stored in {@link WidgetController}.
     *
     * @return SS_List - Collection of {@link WidgetController} instances.
     */
    public function WidgetControllers()
    {
        $controllers = new ArrayList();
        $items = $this->ItemsToRender();
        if (!is_null($items)) {
            foreach ($items as $widget) {
                /** @var Widget $widget */

                /** @var Controller $controller */
                $controller = $widget->getController();

                $controller->doInit();
                $controllers->push($controller);
            }
        }
        return $controllers;
    }

    /**
     * @return HasManyList
     */
    public function Items()
    {
        return $this->Widgets();

The method Widgets() does not exist on SilverStripe\Widgets\Model\WidgetArea. Since you implemented __call, consider adding a @method annotation.

    }

    /**
     * @return HasManyList
     */
    public function ItemsToRender()
    {
        return $this->Items()->filter('Enabled', 1);
    }

    /**
     * @return string - HTML
     */
    public function forTemplate()
    {
        return $this->renderWith($this->template);
    }

    /**
     *
     * @param string $template
     */
    public function setTemplate($template)
    {
        $this->template = $template;
    }
}