A17\Twill\Repositories\BlockRepository

Complexity

Total Complexity

15

Size/Duplication

Total Lines	114
Duplicated Lines	0 %

Test Coverage

Coverage

51.02%

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	afterDelete()	0	7	2
A	getCrops()	0	3	1
A	hydrate()	0	24	3
A	afterSave()	0	11	3
A	buildFromCmsArray()	0	17	3
A	__construct()	0	4	1
A	getBlockInstanceForBlockData()	0	10	2

<?php

namespace A17\Twill\Repositories;

use A17\Twill\Models\Behaviors\HasFiles;
use A17\Twill\Models\Behaviors\HasMedias;
use A17\Twill\Models\Block;
use A17\Twill\Repositories\Behaviors\HandleFiles;
use A17\Twill\Repositories\Behaviors\HandleMedias;
use A17\Twill\Services\Blocks\Block as BlockConfig;
use A17\Twill\Services\Blocks\BlockCollection;
use Illuminate\Config\Repository as Config;
use Illuminate\Support\Collection;
use Log;
use ReflectionException;
use Schema;

class BlockRepository extends ModuleRepository
{
    use HandleMedias, HandleFiles;

The trait A17\Twill\Repositories\Behaviors\HandleMedias requires some properties which are not provided by A17\Twill\Repositories\BlockRepository: $medias, $crop_y, $pivot, $metadatas, $mediasParams, $crop_h, $ratio, $crop_x, $crop_w

The trait A17\Twill\Repositories\Behaviors\HandleFiles requires some properties which are not provided by A17\Twill\Repositories\BlockRepository: $filesParams, $files

    /**
     * @var Config
     */
    protected $config;

    /**
     * @param Block $model
     */
    public function __construct(Block $model, Config $config)
    {
        $this->model = $model;

It seems like $model of type A17\Twill\Models\Block is incompatible with the declared type A17\Twill\Models\Model of property $model.

        $this->config = $config;
    }

    /**
     * @param string $role
     * @return array
     */
    public function getCrops($role)
    {
        return $this->config->get('twill.block_editor.crops')[$role];
    }

    public function hydrate($object, $fields)
    {
        if (Schema::hasTable(config('twill.related_table', 'twill_related'))) {
            $relatedItems = Collection::make();

            Collection::make($fields['browsers'])->each(function ($items, $browserName) use (&$relatedItems) {
                Collection::make($items)->each(function ($item) use ($browserName, &$relatedItems) {
                    try {
                        $repository = $this->getModelRepository($item['endpointType'] ?? $browserName);
                        $relatedItems->push((object) [
                            'related' => $repository->getById($item['id']),
                            'browser_name' => $browserName,
                        ]);

                    } catch (ReflectionException $e) {
                        Log::error($e);
                    }
                });
            });

            $object->setRelation('relatedItems', $relatedItems);
        }

        return parent::hydrate($object, $fields);
    }

    /**
     * @param HasMedias|HasFiles $object
     * @return void
     */
    public function afterSave($object, $fields)
    {
        if (Schema::hasTable(config('twill.related_table', 'twill_related'))) {
            if (isset($fields['browsers'])) {
                Collection::make($fields['browsers'])->each(function ($items, $browserName) use ($object) {
                    $object->saveRelated($items, $browserName);

It seems like saveRelated() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                });
            }
        }

        parent::afterSave($object, $fields);

$object of type A17\Twill\Models\Behaviors\HasFiles|A17\Twill\Models\Behaviors\HasMedias is incompatible with the type A17\Twill\Models\Model expected by parameter $object of A17\Twill\Repositories\ModuleRepository::afterSave().

    }

    public function afterDelete($object)
    {
        $object->medias()->sync([]);
        $object->files()->sync([]);

        if (Schema::hasTable(config('twill.related_table', 'twill_related'))) {
            $object->relatedItems()->delete();
        }
    }

    /**
     * @param array $block
     * @param bool $repeater
     * @return array
     */
    public function buildFromCmsArray($block, $repeater = false)
    {
        $blockInstance = $this->getBlockInstanceForBlockData($block, $repeater);

        $block['type'] = $blockInstance['name'];

        $block['content'] = empty($block['content']) ? new \stdClass : (object) $block['content'];

        if ($block['browsers']) {
            $browsers = Collection::make($block['browsers'])->map(function ($items) {
                return Collection::make($items)->pluck('id');
            })->toArray();

            $block['content']->browsers = $browsers;
        }

        return $block;
    }

    public function getBlockInstanceForBlockData(array $block, bool $repeater = false): Collection
    {
        if ($repeater) {
            $blocksList = app(BlockCollection::class)->getRepeaterList();
        } else {
            $blocksList = app(BlockCollection::class)->getBlockList();
        }

        return $blocksList->first(function(Collection $blockConfig) use ($block) {
            return $blockConfig['component'] === $block['type'];
        });
    }
}