Dynamic\Salsify\TypeHandler\Asset\AssetHandler

Complexity

Total Complexity

20

Size/Duplication

Total Lines	155
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	isTransformOutOfDate()	0	8	2
A	updateFile()	0	16	5
A	findOrCreateFile()	0	29	5
A	getAssetBySalsifyID()	0	19	5
A	fetchAsset()	0	21	1
A	writeFile()	0	10	2

<?php

namespace Dynamic\Salsify\TypeHandler\Asset;

use Dynamic\Salsify\Model\Fetcher;
use Dynamic\Salsify\ORM\FileDataExtension;
use Dynamic\Salsify\ORM\ImageDataExtension;
use Dynamic\Salsify\ORM\SalsifyIDExtension;
use Dynamic\Salsify\Traits\Yieldable;
use GuzzleHttp\Client;
use SilverStripe\Assets\File;
use SilverStripe\Assets\Image;
use SilverStripe\Core\Extension;
use SilverStripe\ORM\DataObject;

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

    /**
     * @param $id
     * @return array
     * @throws \Exception
     */
    protected function fetchAsset($id)
    {
        $apiKey = $this->owner->config()->get('apiKey');//Config::inst()->get(Fetcher::class, 'apiKey');

The method config() does not exist on Dynamic\Salsify\TypeHandler\Asset\AssetHandler.

        $timeout = $this->owner->config()->get('timeout');
        $orgID = $this->owner->config()->get('organizationID');

        $url = "v1/orgs/{$orgID}/digital_assets/{$id}";

        $client = new Client([
            'base_uri' => Fetcher::API_BASE_URL,
            'timeout' => $timeout,
            'http_errors' => false,
            'verify' => true,
            'headers' => [
                'Authorization' => 'Bearer ' . $apiKey,
                'Content-Type' => 'application/json',
            ],
        ]);

        $response = $client->get($url);
        return json_decode($response->getBody(), true);
    }

    /**
     * @param $id
     * @return array|bool
     * @throws \Exception
     */
    protected function getAssetBySalsifyID($id)
    {
        if (is_array($id)) {
            $id = $id[0];
        }

        if ($this->owner->hasFile() === false) {

The method hasFile() does not exist on Dynamic\Salsify\TypeHandler\Asset\AssetHandler.

            return $this->fetchAsset($id);
        }

        $asset = false;
        $assetGenerator = $this->owner->yieldKeyVal($this->owner->getAssetStream(), $this->owner->resetAssetStream());

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

The method yieldKeyVal() does not exist on Dynamic\Salsify\TypeHandler\Asset\AssetHandler.

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

Are you sure the usage of $this->owner->resetAssetStream() targeting Dynamic\Salsify\Model\Mapper::resetAssetStream() seems to always return null.

        foreach ($assetGenerator as $name => $data) {
            if ($data['salsify:id'] == $id) {
                $asset = $data;
                $assetGenerator->send(Yieldable::$STOP_GENERATOR);
            }
        }
        return $asset;
    }

    /**
     * @param string $id
     * @param string $type
     * @param string|DataObject $class
     * @return File|SalsifyIDExtension|FileDataExtension
     */
    protected function findOrCreateFile($id, $type, $class = File::class)
    {
        $filter = [
            'SalsifyID' => $id,
            'Type' => $type,
        ];
        /** @var File|SalsifyIDExtension|FileDataExtension $file */
        if ($file = $class::get()->filter($filter)->first()) {
            return $file;
        }

        // TODO - remove at a later date
        $filter = [
            'SalsifyID' => $id,
            'Type' => null,
        ];
        if ($file = $class::get()->filter($filter)->first()) {
            // checks for changes from image to file before trying to update
            if (!($class === File::class && $file->getClassName() !== File::class)) {
                $file->Type = $type;
                return $this->writeFile($file);
            }
        }
        // end of TODO removal

        $file = $class::create();
        $file->SalsifyID = $id;
        $file->Type = $type;
        return $file;
    }

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

The property SalsifyUpdatedAt does not seem to exist on Dynamic\Salsify\ORM\FileDataExtension.

            if (!$this->isTransformOutOfDate($file, $class, $transformation)) {

It seems like $file can also be of type Dynamic\Salsify\ORM\FileDataExtension and Dynamic\Salsify\ORM\SalsifyIDExtension; however, parameter $file of Dynamic\Salsify\TypeHandler\Asset\AssetHandler::isTransformOutOfDate() does only seem to accept SilverStripe\ORM\DataObject, maybe add an additional type check?

                return $file;

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

            }
        }

        $file->SalsifyUpdatedAt = $updatedAt;
        if ($file->hasField('Transformation')) {

The method hasField() does not exist on Dynamic\Salsify\ORM\FileDataExtension.

The method hasField() does not exist on Dynamic\Salsify\ORM\SalsifyIDExtension.

            $file->Transformation = $transformation;

The property Transformation does not seem to exist on Dynamic\Salsify\ORM\FileDataExtension.

The property Transformation does not seem to exist on Dynamic\Salsify\ORM\SalsifyIDExtension.

        }
        $file->setFromStream(fopen($url, 'r'), $name);

The method setFromStream() does not exist on Dynamic\Salsify\ORM\SalsifyIDExtension.

The method setFromStream() does not exist on Dynamic\Salsify\ORM\FileDataExtension.

        return $this->writeFile($file);

It seems like $file can also be of type Dynamic\Salsify\ORM\FileDataExtension and Dynamic\Salsify\ORM\SalsifyIDExtension; however, parameter $file of Dynamic\Salsify\TypeHandler\Asset\AssetHandler::writeFile() does only seem to accept SilverStripe\Assets\File, maybe add an additional type check?

    }

    /**
     * @param DataObject $file
     * @param string $class
     * @param string $transformation
     *
     * @return bool
     */
    private function isTransformOutOfDate($file, $class, $transformation)

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

    {
        if (!$file instanceof Image) {
            return false;
        }

        /** @var Image|ImageDataExtension $file */
        return $file->Transformation != $transformation;
    }

    /**
     * @param File $file
     * @return File
     */
    private function writeFile($file)
    {
        $published = $file->isPublished();
        $file->write();

        if ($published) {
            $file->publishSingle();
        }

        return $file;
    }
}