Dynamic\Salsify\TypeHandler\Asset\FileHandler

Complexity

Total Complexity

6

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handleManyFilesType()	0	16	3
A	handleFileType()	0	15	3

<?php

namespace Dynamic\Salsify\TypeHandler\Asset;

use SilverStripe\Assets\File;
use SilverStripe\ORM\DataObject;

/**
 * Class FileHandler
 * @package Dynamic\Salsify\TypeHandler
 *
 * @property-read \Dynamic\Salsify\Model\Mapper|FileHandler $owner
 */
class FileHandler extends AssetHandler
{

    /**
     * @var array
     */
    private static $field_types = [

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

        'File' => [
            'requiresWrite' => true,
            'requiresSalsifyObjects' => false,
            'allowsModification' => true,
        ],
        'ManyFiles' => [
            'requiresWrite' => true,
            'requiresSalsifyObjects' => false,
            'allowsModification' => true,
        ],
    ];

    /**
     * @param string|DataObject $class
     * @param $data
     * @param $dataField
     * @param $config
     * @param $dbField
     * @return string|int
     *
     * @throws \Exception
     */
    public function handleFileType($class, $data, $dataField, $config, $dbField)

The parameter $class is not used and could be removed.

    {
        $data = $this->getAssetBySalsifyID($data[$dataField]);
        if (!$data) {
            return '';
        }

        $asset = $this->updateFile(
            $data['salsify:id'],
            $data['salsify:updated_at'],
            $data['salsify:url'],
            $data['salsify:name'],
            $config['type']
        );
        return preg_match('/ID$/', $dbField) ? $asset->ID : $asset;
    }

    /**
     * @param string|DataObject $class
     * @param $data
     * @param $dataField
     * @param $config
     * @param $dbField
     * @return array
     *
     * @throws \Exception
     */
    public function handleManyFilesType($class, $data, $dataField, $config, $dbField)
    {
        $files = [];
        $fieldData = $data[$dataField];
        // convert to array to prevent problems
        if (!is_array($fieldData)) {
            $fieldData = [$fieldData];
        }

        foreach ($this->owner->yieldSingle($fieldData) as $fileID) {

The method yieldSingle() does not exist on Dynamic\Salsify\TypeHandler\Asset\FileHandler.

            $entryData = array_merge($data, [
                $dataField => $fileID
            ]);
            $files[] = $this->owner->handleFileType($class, $entryData, $dataField, $config, $dbField, $class);

The method handleFileType() does not exist on Dynamic\Salsify\Model\Mapper. Since you implemented __call, consider adding a @method annotation.

The call to Dynamic\Salsify\TypeHandler\Asset\FileHandler::handleFileType() has too many arguments starting with $class.

        }
        return $files;
    }
}