HandleFieldsGroups::getFormFieldsHandleFieldsGroups() - Code Metrics - Inspection of "Add a feature to group fields in a JSON column (#4..." - area17/twill - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — 1.2 ( 00711e...5564e4 )

by Quentin

created 2019-11-12 21:11 UTC

getFormFieldsHandleFieldsGroups() A

↳ Parent: HandleFieldsGroups

Complexity

Conditions	4
Paths	4

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	7
c	1
b	0
f	0
nc	4
nop	2
dl	0
loc	14
rs	10

<?php

namespace A17\Twill\Repositories\Behaviors;

use Illuminate\Support\Arr;

trait HandleFieldsGroups
{

    /**
     * @param \A17\Twill\Models\Model|null $object
     * @param array $fields
     * @return array
     */
    public function prepareFieldsBeforeSaveHandleFieldsGroups($object, $fields)
    public function prepareFieldsBeforeSaveHandleFieldsGroups(/** @scrutinizer ignore-unused */ $object, $fields)
    {
        return $this->handleFieldsGroups($fields);
    }

    /**
     * @param \A17\Twill\Models\Model|null $object
     * @param array $fields
     * @return array
     */
    public function prepareFieldsBeforeCreateHandleFieldsGroups($fields)
    {
        return $this->handleFieldsGroups($fields);
    }

    /**
     * @param \A17\Twill\Models\Model $object
     * @param array $fields
     * @return array
     */
    public function getFormFieldsHandleFieldsGroups($object, $fields)
    {
        foreach ($this->fieldsGroups as $group => $groupFields) {
            if ($object->$group) {
                $decoded_fields = json_decode($object->$group, true);
                // In case that some field read the value through $item->$name
                foreach($decoded_fields as $field_name => $field_value) {
                    $object->setAttribute($field_name, $field_value);
                }
                $fields = array_merge($fields, $decoded_fields);
            }
        }

        return $fields;
    }

    protected function handleFieldsGroups($fields) {
        foreach ($this->fieldsGroups as $group => $groupFields) {
            $fields[$group] = json_encode(Arr::where(Arr::only($fields, $groupFields), function ($value, $key) {
            $fields[$group] = json_encode(Arr::where(Arr::only($fields, $groupFields), function ($value, /** @scrutinizer ignore-unused */ $key) {
                return !empty($value);
            }));
            Arr::forget($fields, $groupFields);
        }

        return $fields;
    }
}


area17 / twill

Push — 1.2 ( 00711e...5564e4 )

getFormFieldsHandleFieldsGroups() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like