LeKoala\FilePond\FilePondFileExtension

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	onAfterUpload()	0	4	2
A	getObjectFiles()	0	6	1

<?php

namespace LeKoala\FilePond;

use SilverStripe\Assets\File;
use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\DataExtension;
use SilverStripe\AssetAdmin\Controller\AssetAdmin;

The type SilverStripe\AssetAdmin\Controller\AssetAdmin was not found. Maybe you did not declare it correctly or list all dependencies?

class FilePondFileExtension extends DataExtension
{
    private static $db = [

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

        // This helps tracking state of files uploaded through ajax uploaders
        "IsTemporary" => "Boolean",
    ];
    private static $has_one = [

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

        // Record is already used by versioned extensions
        // ChangeSetItem already uses Object convention so we use the same
        "Object" => DataObject::class,
    ];

    /**
     * Get a list of files for the given DataObject
     *
     * This is just a convenience method, not a public api
     *
     * @param DataObject $record
     * @return DataList|File[]
     */
    public static function getObjectFiles(DataObject $record)
    {
        return File::get()->filter([

The expression return SilverStripe\Assets\File::get()->filter(array('ObjectID' => $record->ID, 'ObjectClass' => get_class($record)))->exclude('IsTemporary', 1) returns the type SilverStripe\ORM\DataList which is incompatible with the documented return type SilverStripe\Assets\File[]&LeKoala\FilePond\DataList.

            'ObjectID' => $record->ID,
            'ObjectClass' => get_class($record),
        ])->exclude('IsTemporary', 1);
    }

    /**
     * Called by Upload::loadIntoFile
     * @return void
     */
    public function onAfterUpload()
    {
        if (File::config()->enable_auto_thumbnail) {
            AssetAdmin::create()->generateThumbnails($this->owner);
        }
    }
}