SilverStripe\Forms\Tests\FormFactoryTest\EditFormFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	25
Duplicated Lines	68 %

Coupling/Cohesion

Components	0
Dependencies	5

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getFormFields()	9	9	1
A	getFormActions()	8	8	1

<?php

namespace SilverStripe\Forms\Tests\FormFactoryTest;

use SilverStripe\Control\RequestHandler;
use SilverStripe\Forms\DefaultFormFactory;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\FormAction;
use SilverStripe\Forms\HiddenField;
use SilverStripe\Forms\TextField;

/**
 * Test factory
 */
class EditFormFactory extends DefaultFormFactory
{
    private static $extensions = [

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

        ControllerExtension::class
    ];

    protected function getFormFields(RequestHandler $controller = null, $name, $context = [])

This method seems to be duplicated in your project.

    {
        $fields = new FieldList(
            new HiddenField('ID'),
            new TextField('Title')
        );
        $this->invokeWithExtensions('updateFormFields', $fields, $controller, $name, $context);
        return $fields;
    }

    protected function getFormActions(RequestHandler $controller = null, $name, $context = [])

This method seems to be duplicated in your project.

    {
        $actions = new FieldList(
            new FormAction('save', 'Save')
        );
        $this->invokeWithExtensions('updateFormActions', $actions, $controller, $name, $context);
        return $actions;
    }
}