HandleFieldsGroups::getFormFieldsHandleFieldsGroups() - Code Metrics - Inspection of "Fix empty fields groups" - area17/twill - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5f2d64...84176e )

by Quentin

created 2020-02-26 15:15 UTC

getFormFieldsHandleFieldsGroups() A

↳ Parent: HandleFieldsGroups

Complexity

Conditions	4
Paths	4

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	7.9062

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
ccs	3
cts	8
cp	0.375
crap	7.9062
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] = Arr::where(Arr::only($fields, $groupFields), function ($value, $key) {
            $fields[$group] = Arr::where(Arr::only($fields, $groupFields), function ($value, /** @scrutinizer ignore-unused */ $key) {
                return !empty($value);
            });
            
            if (empty($fields[$group])) {
                $fields[$group] = null;
            }
            
            Arr::forget($fields, $groupFields);
        }

        return $fields;
    }
}


area17 / twill

Push — master ( 5f2d64...84176e )

getFormFieldsHandleFieldsGroups() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like