Backpack\CRUD\PanelTraits\Update

Complexity

Total Complexity

7

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Test Coverage

Coverage

83.33%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	16	1
B	getUpdateFields()	0	30	6

<?php

namespace Backpack\CRUD\PanelTraits;

trait Update
{
    /*
    |--------------------------------------------------------------------------
    |                                   UPDATE
    |--------------------------------------------------------------------------
    */

    /**
     * Update a row in the database.
     *
     * @param  [Int] The entity's id
     * @param  [Request] All inputs to be updated.
     *
     * @return [Eloquent Collection]

The doc-type Eloquent">[Eloquent could not be parsed: Unknown type name "[" at position 0. [(view supported doc-types)

     */
    public function update($id, $data)

The return type could not be reliably inferred; please add a @return annotation.

    {
        $data = $this->decodeJsonCastedAttributes($data, 'update', $id);

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

        $data = $this->compactFakeFields($data, 'update', $id);

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

        $item = $this->model->findOrFail($id);

The property model does not exist. Did you maybe forget to declare it?

        $this->syncPivot($item, $data, 'update');

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

        // ommit the n-n relationships when updating the eloquent item
        $nn_relationships = array_pluck($this->getRelationFieldsWithPivot('update'), 'name');

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

        $data = array_except($data, $nn_relationships);
        $updated = $item->update($data);

$updated is not used, you could remove the assignment.

        return $item;
    }

    /**
     * Get all fields needed for the EDIT ENTRY form.
     *
     * @param  [integer] The id of the entry that is being edited.
     * @param int $id
     *
     * @return [array] The fields with attributes, fake attributes and values.

The doc-type [array] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     */
    public function getUpdateFields($id)

The return type could not be reliably inferred; please add a @return annotation.

    {
        $fields = $this->update_fields;

The property update_fields does not exist. Did you maybe forget to declare it?

        $entry = $this->getEntry($id);

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

        foreach ($fields as $k => $field) {
            // set the value
            if (! isset($fields[$k]['value'])) {
                if (isset($field['subfields'])) {
                    $fields[$k]['value'] = [];
                    foreach ($field['subfields'] as $key => $subfield) {
                        $fields[$k]['value'][] = $entry->{$subfield['name']};
                    }
                } else {
                    $fields[$k]['value'] = $entry->{$field['name']};
                }
            }
        }

        // always have a hidden input for the entry id
        if (! array_key_exists('id', $fields)) {
            $fields['id'] = [
                'name'  => $entry->getKeyName(),
                'value' => $entry->getKey(),
                'type'  => 'hidden',
            ];
        }

        return $fields;
    }
}