UploadField::getAttributes() - Code Metrics - Inspection of "ENHANCEMENT Enable UploadField.js within non-react..." - silverstripe/silverstripe-asset-admin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#331)

unknown

created 2016-12-18 20:40 UTC

UploadField::getAttributes() A

↳ Parent: UploadField

Complexity

Conditions	1
Paths	1

Size

Total Lines	17
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	17
rs	9.4285
cc	1
eloc	11
nc	1
nop	0

<?php

namespace SilverStripe\AssetAdmin\Forms;

use SilverStripe\AssetAdmin\Controller\AssetAdmin;
use SilverStripe\Assets\Folder;
use SilverStripe\Forms\FileUploadReceiver;
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 UploadField extends FormField
{
    use FileUploadReceiver;

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


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


    protected $schemaDataType = FormField::SCHEMA_DATA_TYPE_CUSTOM;

    protected $schemaComponent = 'UploadField';

    /**
     * @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->constructFileUploadReceiver();
        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();
        $state['value'] = $this->Value() ?: [ 'Files' => []];
        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();

        // Disabled for has_one components
        if ($record && DataObject::getSchema()->hasOneComponent(get_class($record), $name)) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            return false;
        }
        return true;
    }

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

    public function getAttributes()
    {
        $attributes = array(
            'class' => $this->extraClass(),
            'type' => 'file',
            'multiple' => $this->getIsMultiUpload(),
            'id' => $this->ID(),
            'data-schema' => json_encode($this->getSchemaData()),
            'data-state' => json_encode($this->getSchemaState()),
        );

        $attributes = array_merge($attributes, $this->attributes);

        $this->extend('updateAttributes', $attributes);

        return $attributes;
    }

    public function Type()
    {
        return 'entwine-uploadfield uploadfield';
    }
}


1			<?php
2
3			namespace SilverStripe\AssetAdmin\Forms;
4
5			use SilverStripe\AssetAdmin\Controller\AssetAdmin;
6			use SilverStripe\Assets\Folder;
7			use SilverStripe\Forms\FileUploadReceiver;
8			use SilverStripe\Forms\FormField;
9			use SilverStripe\ORM\DataObject;
10			use SilverStripe\ORM\SS_List;
11
12			/**
13			* Represents a file upload field with ReactJS based frontend.
14			*
15			* Allows writing to a parent record with the following relation types:
16			* - has_one
17			* - has_many
18			* - many_many
19			*
20			* Additionally supports writing directly to the File table not attached
21			* to any parent record.
22			*/
23			class UploadField extends FormField
24			{
25			use FileUploadReceiver;
26
27			/**
28			* @config
29			* @var int
30			*/
31			private static $thumbnail_width = 60;
			0 ignored issues – show Unused Code introduced 2016-11-29 02:42 UTC by Report Bug Copy Issue Report The property `$thumbnail_width` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
32
33			/**
34			* @config
35			* @var int
36			*/
37			private static $thumbnail_height = 60;
			0 ignored issues – show Unused Code introduced 2016-11-29 02:42 UTC by Report Bug Copy Issue Report The property `$thumbnail_height` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
38
39			protected $schemaDataType = FormField::SCHEMA_DATA_TYPE_CUSTOM;
40
41			protected $schemaComponent = 'UploadField';
42
43			/**
44			* @var bool\|null
45			*/
46			protected $multiUpload = null;
47
48			/**
49			* Create a new file field.
50			*
51			* @param string $name The internal field name, passed to forms.
52			* @param string $title The field label.
53			* @param SS_List $items Items assigned to this field
54			*/
55			public function __construct($name, $title = null, SS_List $items = null)
56			{
57			$this->constructFileUploadReceiver();
58			parent::__construct($name, $title);
59			if ($items) {
60			$this->setItems($items);
61			}
62			}
63
64			public function getSchemaDataDefaults()
65			{
66			$defaults = parent::getSchemaDataDefaults();
67			$uploadLink = AssetAdmin::singleton()->Link('api/createFile');
68			$defaults['data']['createFileEndpoint'] = [
69			'url' => $uploadLink,
70			'method' => 'post',
71			'payloadFormat' => 'urlencoded',
72			];
73			$defaults['data']['multi'] = $this->getIsMultiUpload();
74			$defaults['data']['parentid'] = $this->getFolderID();
75			return $defaults;
76			}
77
78			/**
79			* Get ID of target parent folder
80			*
81			* @return int
82			*/
83			protected function getFolderID()
84			{
85			$folderName = $this->getFolderName();
86			if (!$folderName) {
87			return 0;
88			}
89			$folder = Folder::find_or_make($folderName);
90			return $folder ? $folder->ID : 0;
91			}
92
93			public function getSchemaStateDefaults()
94			{
95			$state = parent::getSchemaStateDefaults();
96			$state['data']['files'] = $this->getEncodedItems();
97			$state['value'] = $this->Value() ?: [ 'Files' => []];
98			return $state;
99			}
100
101			/**
102			* Encode selected values for react
103			*
104			* @return array
105			*/
106			protected function getEncodedItems()
107			{
108			$assetAdmin = AssetAdmin::singleton();
109			$fileData = [];
110			foreach ($this->getItems() as $file) {
111			$fileData[] = $assetAdmin->getObjectFromData($file);
112			}
113			return $fileData;
114			}
115
116			/**
117			* Check if allowed to upload more than one file
118			*
119			* @return bool
120			*/
121			public function getIsMultiUpload()
122			{
123			if (isset($this->multiUpload)) {
124			return $this->multiUpload;
125			}
126			// Guess from record
127			$record = $this->getRecord();
128			$name = $this->getName();
129
130			// Disabled for has_one components
131			if ($record && DataObject::getSchema()->hasOneComponent(get_class($record), $name)) {
			0 ignored issues – show Bug Best Practice introduced 2016-11-22 04:11 UTC by Report Bug Copy Issue Report The expression `\SilverStripe\ORM\DataOb..._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. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
132			return false;
133			}
134			return true;
135			}
136
137			/**
138			* Set upload type to multi / single
139			*
140			* @param $multi
141			* @return $this
142			*/
143			public function setIsMultiUpload($multi)
144			{
145			$this->multiUpload = $multi;
146			return $this;
147			}
148
149			public function getAttributes()
150			{
151			$attributes = array(
152			'class' => $this->extraClass(),
153			'type' => 'file',
154			'multiple' => $this->getIsMultiUpload(),
155			'id' => $this->ID(),
156			'data-schema' => json_encode($this->getSchemaData()),
157			'data-state' => json_encode($this->getSchemaState()),
158			);
159
160			$attributes = array_merge($attributes, $this->attributes);
161
162			$this->extend('updateAttributes', $attributes);
163
164			return $attributes;
165			}
166
167			public function Type()
168			{
169			return 'entwine-uploadfield uploadfield';
170			}
171			}
172

silverstripe / silverstripe-asset-admin

Pull Request — master (#331)

UploadField::getAttributes() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like