Bummzack\SortableFile\Tests\Model\TestDataObject

Complexity

Total Complexity

2

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

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

<?php

namespace Bummzack\SortableFile\Tests\Model;

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

class TestDataObject extends DataObject implements TestOnly
{

    private static $has_many = [

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

        'LinkedFilesFile' => FileLinkDataObject::class
    ];

    private static $many_many = [

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

        'Files' => File::class,
        'OtherFiles' => File::class,
        'LinkedFiles' => [
            'through' 			=> FileLinkDataObject::class,
            'from' 				=> 'Owner',
            'to' 				=> 'File',
        ]
    ];

    private static $many_many_extraFields = [

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

        'Files' => [ 'SortOrder' => 'Int' ],
        'OtherFiles' => [ 'Sort' => 'Int' ]
    ];

    private static $owns = [

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

        'FileLinks'
    ];

    public function getCMSFields()
    {
        return FieldList::create(
            SortableUploadField::create('Files'),
            SortableUploadField::create('OtherFiles')->setSortColumn('Sort')
        );
    }

    public function getLinkedFiles()
    {
        return $this->LinkedFiles()->sort('Sort');

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

    }
}