Thinktomorrow\Chief\Fragments\ManagesFragments

Complexity

Total Complexity

5

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	saveFragmentFields()	0	25	4
A	saveFragmentImageFields()	0	3	1

<?php

namespace Thinktomorrow\Chief\Fragments;

use Illuminate\Http\Request;
use Thinktomorrow\Chief\Fields\Types\FileField;
use Thinktomorrow\Chief\Fields\Types\ImageField;
use Thinktomorrow\Chief\Media\Application\FileFieldHandler;
use Thinktomorrow\Chief\Media\Application\ImageFieldHandler;

trait ManagesFragments
{
    public function saveFragmentFields(FragmentField $fragmentField, Request $request)
    {
        $payload = $request->input($fragmentField->getDottedName(), []);
        $imagePayload = $request->input('images.'.$fragmentField->getDottedName(), []);

        // Compose Fragment instances for each payload entry
        $fragments = array_map(function($fragmentPayload) use($fragmentField){
            return Fragment::fromRequestPayload($fragmentField->getKey(), $fragmentPayload); // (new Fragment($field->getKey(), $fragmentPayload));
        }, $payload);

        // remove all dead fragments
        $this->existingModel()->removeAllFragments($fragmentField->getKey(), array_map(function (Fragment $fragment) {

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

            return $fragment->hasModelId() ? $fragment->getModelId() : null;
        }, $fragments));

        // Save each fragment
        foreach($fragments as $i => $fragment) {
            $modelId = $this->existingModel()->saveFragment($fragment, $i);

            // Attach any asset
            if(isset($imagePayload[$i])) {
                $fieldKey = key($imagePayload[$i]);
                $imageField = $this->fields()->keyed($fragmentField->getKey())->first()->getFields()->keyed($fieldKey)->first();

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

                $this->saveFragmentImageFields($imageField, $imagePayload[$i][$fieldKey], FragmentModel::find($modelId), $request);
            }
        }

    }

    private function saveFragmentImageFields(ImageField $field, array $values, FragmentModel $model, Request $request)
    {
        app(ImageFieldHandler::class)->handle($model, $field, $values, $request);
    }
}