SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\PolymorphicPeopleGroup

Complexity

Total Complexity

1

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	13	1

<?php

namespace SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest;

use SilverStripe\Dev\TestOnly;
use SilverStripe\Forms\GridField\GridField;
use SilverStripe\Forms\GridField\GridFieldConfig_RelationEditor;
use SilverStripe\ORM\DataObject;

class PolymorphicPeopleGroup extends DataObject implements TestOnly
{
    private static $table_name = 'GridFieldDetailFormTest_PolymorphicPeopleGroup';

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

    private static $db = [

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

        'Name' => 'Varchar'
    ];

    private static $has_many = [

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

        'People' => Person::class
    ];

    private static $default_sort = '"Name"';

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

    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->replaceField(
            'People',
            GridField::create(
                'People',
                'People',
                $this->People(),

The method People() does not exist on SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest\PolymorphicPeopleGroup. Since you implemented __call, consider adding a @method annotation.

                GridFieldConfig_RelationEditor::create()
            )
        );
        return $fields;
    }
}