Bummzack\SortableFile\Tests\Model\TestManyManyThroughDataObject

Complexity

Total Complexity

2

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	4	1
A	getLinkedFiles()	0	3	1

<?php

namespace Bummzack\SortableFile\Tests\Model;

use Bummzack\SortableFile\Forms\SortableUploadField;
use SilverStripe\Dev\TestOnly;
use SilverStripe\Forms\FieldList;
use SilverStripe\ORM\DataObject;
use SilverStripe\Versioned\Versioned;

class TestManyManyThroughDataObject extends DataObject implements TestOnly
{
    private static $many_many = [

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

        'Files' => [
            'through' => TestFileLinkDataObject::class,
            'from' => 'Owner',
            'to' => 'File',
        ]
    ];

    private static $owns = [

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

        'Files'
    ];

    private static $cascade_deletes = [

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

        'Files'
    ];

    private static $extensions = [

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

        Versioned::class
    ];

    private static $table_name = 'Bummzack_TestManyManyThroughDataObject';

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

    public function getCMSFields()
    {
        return FieldList::create(
            SortableUploadField::create('Files')
        );
    }

    public function getLinkedFiles()
    {
        return $this->Files()->sort('SortOrder');

The method Files() does not exist on Bummzack\SortableFile\Tests\Model\TestManyManyThroughDataObject. Since you implemented __call, consider adding a @method annotation.

    }
}