Dynamic\Salsify\TypeHandler\FileHandler

Complexity

Total Complexity

12

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	updateFile()	0	12	3
A	findOrCreateFile()	0	10	2
A	handleFileType()	0	14	3
A	getAssetBySalsifyID()	0	14	4

<?php

namespace Dynamic\Salsify\TypeHandler;

use JsonMachine\JsonMachine;
use SilverStripe\Assets\File;
use SilverStripe\Core\Extension;
use SilverStripe\ORM\DataObject;

/**
 * Class FileHandler
 * @package Dynamic\Salsify\TypeHandler
 *
 * @property-read \Dynamic\Salsify\Model\Mapper|\Dynamic\Salsify\TypeHandler\FileHandler $owner
 */
class FileHandler extends Extension
{
    /**
     * @var array
     */
    private static $field_types = [

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

        'File'
    ];

    /**
     * @var JsonMachine
     */
    protected $assetStream;

    /**
     * @param $value
     * @param $field
     * @param string |DataObject $class
     * @return string
     *
     * @throws \Exception
     */
    public function handleFileType($value, $field, $class)

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

    {
        $data = $this->getAssetBySalsifyID($value);
        if (!$data) {

The condition $data is always false.

            return '';
        }

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

    /**
     * @param $id
     * @return array|bool
     */
    protected function getAssetBySalsifyID($id)
    {
        if (is_array($id)) {
            $id = $id[count($id) - 1];
        }

        $asset = false;
        foreach ($this->owner->getAssetStream() as $name => $data) {

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

            if ($data['salsify:id'] == $id) {
                $asset = $data;
            }
        }
        $this->owner->resetAssetStream();

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

        return $asset;
    }

    /**
     * @param string $id
     * @param string|DataObject $class
     * @return File|\Dyanmic\Salsify\ORM\FileExtension
     */
    protected function findOrCreateFile($id, $class = File::class)
    {
        /** @var File|\Dyanmic\Salsify\ORM\FileExtension $file */
        if ($file = $class::get()->find('SalisfyID', $id)) {
            return $file;
        }

        $file = $class::create();
        $file->SalisfyID = $id;
        return $file;
    }

    /**
     * @param int|string $id
     * @param string $updatedAt
     * @param string $url
     * @param string $name
     * @param string|DataObject $class
     *
     * @return File|bool
     * @throws \Exception
     */
    protected function updateFile($id, $updatedAt, $url, $name, $class = File::class)
    {
        $file = $this->findOrCreateFile($id, $class);
        if ($file->SalsifyUpdatedAt && $file->SalsifyUpdatedAt == $updatedAt) {
            return $file;

The expression return $file also could return the type Dyanmic\Salsify\ORM\FileExtension which is incompatible with the documented return type SilverStripe\Assets\File|boolean.

        }

        $file->SalsifyUpdatedAt = $updatedAt;
        $file->setFromStream(fopen($url, 'r'), $name);

The method setFromStream() does not exist on Dyanmic\Salsify\ORM\FileExtension.

        $file->write();

The method write() does not exist on Dyanmic\Salsify\ORM\FileExtension.

        return $file;

The expression return $file also could return the type Dyanmic\Salsify\ORM\FileExtension which is incompatible with the documented return type SilverStripe\Assets\File|boolean.

    }
}