PermissionFormFields::fieldsFromModel() - Code Metrics - Inspection of "Merge pull request #55 from jeremykenedy/developme..." - jeremykenedy/laravel-roles - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d0984d...8e3192 )

by Jeremy

created 2019-05-28 07:26 UTC

PermissionFormFields::fieldsFromModel() A

↳ Parent: PermissionFormFields

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace jeremykenedy\LaravelRoles\App\Services;

use jeremykenedy\LaravelRoles\Traits\RolesAndPermissionsHelpersTrait;

class PermissionFormFields
{
    use RolesAndPermissionsHelpersTrait;


    /**
     * List of fields and default value for each field.
     *
     * @var array
     */
    protected $fieldList = [
        'name'          => '',
        'slug'          => '',
        'description'   => '',
        'model'         => '',
    ];

    /**
     * Create a new job instance.
     *
     * @param int $id
     *
     * @return void
     */
    public function __construct($id = null)
    {
        $this->id = $id;

    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        $fields = $this->fieldList;

        if ($this->id) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            $fields = $this->fieldsFromModel($this->id, $fields);
        }

        foreach ($fields as $fieldName => $fieldValue) {
            $fields[$fieldName] = old($fieldName, $fieldValue);
        }

        // Get the additional data for the form fields
        $permissionFormFieldData = $this->permissionFormFieldData();

        return array_merge(

            $fields,
            $permissionFormFieldData
        );

        return $fields;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
    }

    /**
     * Return the field values from the model.
     *
     * @param int   $id
     * @param array $fields
     *
     * @return array
     */
    protected function fieldsFromModel($id, array $fields)
    {
        $permission = config('roles.models.permission')::findOrFail($id);

        $fieldNames = array_keys(array_except($fields, ['permissions']));
        $fieldNames = array_keys(/** @scrutinizer ignore-deprecated */ array_except($fields, ['permissions']));

        $fields = [
            'id' => $id,
        ];
        foreach ($fieldNames as $field) {
            $fields[$field] = $permission->{$field};
        }

        return $fields;
    }

    /**
     * Get the additonal form fields data.
     *
     * @return array
     */
    protected function permissionFormFieldData()
    {
        return [
            'permissionModels' => $this->getPermissionModels(),
        ];
    }
}


jeremykenedy / laravel-roles

Push — master ( d0984d...8e3192 )

PermissionFormFields::fieldsFromModel() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like