Thinktomorrow\Chief\Fragments\ManagesFragments::saveFragmentFields()

Complexity

Conditions	4
Paths	3

Size

Total Lines	25
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
            }