SilverStripe\Widgets\Tests\WidgetControllerTest\TestWidgetController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	Form()	0	15	1
A	doAction()	0	8	1

<?php

namespace SilverStripe\Widgets\Tests\WidgetControllerTest;

use SilverStripe\Dev\TestOnly;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\Form;
use SilverStripe\Forms\FormAction;
use SilverStripe\Forms\TextField;
use SilverStripe\Widgets\Controllers\WidgetController;

/**
 * @package widgets
 * @subpackage tests
 */
class TestWidgetController extends WidgetController implements TestOnly
{
    private static $allowed_actions = array(

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

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

        'Form'
    );

    public function Form()
    {
        $widgetform = new Form(
            $this,
            'Form',
            new FieldList(
                new TextField('TestValue')
            ),
            new FieldList(
                new FormAction('doAction')
            )
        );

        return $widgetform;
    }

    public function doAction($data, $form)

The parameter $form is not used and could be removed.

    {
        return sprintf(
            'TestValue: %s\nWidget ID: %d',
            $data['TestValue'],
            $this->widget->ID
        );
    }
}