SilverStripe\Widgets\Model\WidgetArea

Complexity

Total Complexity

9

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	WidgetControllers()	0	14	3
A	Items()	0	4	1
A	ItemsToRender()	0	5	1
A	forTemplate()	0	4	1
A	setTemplate()	0	4	1
A	onBeforeDelete()	0	7	2

<?php

namespace SilverStripe\Widgets\Model;

use SilverStripe\ORM\ArrayList;
use SilverStripe\ORM\DataObject;
use SilverStripe\Widgets\Model\Widget;

/**
 * Represents a set of widgets shown on a page.
 *
 * @package widgets
 */
class WidgetArea extends DataObject
{
    /**
     * @var array
     */
    private static $has_many = array(

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

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

        "Widgets" => Widget::class
    );

    /**
     * @var string
     */
    private static $table_name = 'WidgetArea';

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

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

    /**
     *
     * @var string
     */
    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) {
                $controller = $widget->getController();

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

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

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

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

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

    /**
     * Delete all connected Widgets when this WidgetArea gets deleted
     */
    public function onBeforeDelete()
    {
        parent::onBeforeDelete();
        foreach ($this->Widgets() as $widget) {

The method Widgets does not exist on object<SilverStripe\Widgets\Model\WidgetArea>? Since you implemented __call, maybe consider adding a @method annotation.

            $widget->delete();
        }
    }
}