SilverStripe\AssetAdmin\Forms\FileField

Complexity

Total Complexity

14

Size/Duplication

Total Lines	118
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	getSchemaDataDefaults()	0	13	1
A	getFolderID()	0	8	3
A	getSchemaStateDefaults()	0	6	1
A	getEncodedItems()	0	9	2
A	getIsMultiUpload()	0	14	4
A	setIsMultiUpload()	0	3	1

<?php

namespace SilverStripe\AssetAdmin\Forms;

use SilverStripe\AssetAdmin\Controller\AssetAdmin;
use SilverStripe\Assets\Folder;
use SilverStripe\Forms\FileUploadable;
use SilverStripe\Forms\FormField;
use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\SS_List;

/**
 * Represents a file upload field with ReactJS based frontend.
 *
 * Allows writing to a parent record with the following relation types:
 *   - has_one
 *   - has_many
 *   - many_many
 *
 * Additionally supports writing directly to the File table not attached
 * to any parent record.
 */
class FileField extends FormField
{
    use FileUploadable;

    /**
     * @config
     * @var int
     */
    private static $thumbnail_width = 60;

The property $thumbnail_width is not used and could be removed.

    /**
     * @config
     * @var int
     */
    private static $thumbnail_height = 60;

The property $thumbnail_height is not used and could be removed.

    protected $schemaDataType = FormField::SCHEMA_DATA_TYPE_CUSTOM;

    protected $schemaComponent = 'FileField';

    /**
     * @var bool|null
     */
    protected $multiUpload = null;

	/**
	 * Create a new file field.
	 *
	 * @param string $name The internal field name, passed to forms.
	 * @param string $title The field label.
	 * @param SS_List $items Items assigned to this field
	 */
	public function __construct($name, $title = null, SS_List $items = null) {
		$this->constructFileUploadable();
		parent::__construct($name, $title);
		if($items) {
			$this->setItems($items);
		}
	}

	public function getSchemaDataDefaults()
    {
        $defaults = parent::getSchemaDataDefaults();
        $uploadLink = AssetAdmin::singleton()->Link('api/createFile');
        $defaults['data']['createFileEndpoint'] = [
            'url' => $uploadLink,
            'method' => 'post',
            'payloadFormat' => 'urlencoded',
        ];
        $defaults['data']['multi'] = $this->getIsMultiUpload();
        $defaults['data']['parentid'] = $this->getFolderID();
        return $defaults;
    }

    /**
     * Get ID of target parent folder
     *
     * @return int
     */
    protected function getFolderID() {
        $folderName = $this->getFolderName();
        if (!$folderName) {
            return 0;
        }
        $folder = Folder::find_or_make($folderName);
        return $folder ? $folder->ID : 0;
    }

    public function getSchemaStateDefaults()
    {
        $state = parent::getSchemaStateDefaults();
        $state['data']['files'] = $this->getEncodedItems();
        return $state;
    }

    /**
     * Encode selected values for react
     *
     * @return array
     */
    protected function getEncodedItems()
    {
        $assetAdmin = AssetAdmin::singleton();
        $fileData = [];
        foreach ($this->getItems() as $file) {
            $fileData[] = $assetAdmin->getObjectFromData($file);
        }
        return $fileData;
    }

    /**
     * Check if allowed to upload more than one file
     *
     * @return bool
     */
    public function getIsMultiUpload() {
        if (isset($this->multiUpload)) {
            return $this->multiUpload;
        }
        // Guess from record
        $record = $this->getRecord();
        $name = $this->getName();

Consider using $this->name. There is an issue with getName() and APC-enabled PHP versions.

        // Disabled for has_one components
        if ($record && DataObject::getSchema()->hasOneComponent(get_class($record), $name)) {

The expression \SilverStripe\ORM\DataObject::getSchema()->hasOneComponent(get_class($record), $name) of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            return false;
        }
        return true;
    }

    /**
     * Set upload type to multi / single
     * @param $multi
     */
    public function setIsMultiUpload($multi) {
        $this->multiUpload = $multi;
    }
}