Laravel\DataTables\Traits\Updatable

Complexity

Total Complexity

2

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	3
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	update()	0	8	2

<?php

namespace Laravel\DataTables\Traits;

use Arr;
use Illuminate\Database\Eloquent\Model;

trait Updatable
{
    /**
     * Allow Entity Updating.
     *
     * @var bool
     */
    public $allowUpdating = false;

    /**
     * @param Model $model
     * @param array $data
     * @return Model
     */
    public function update(Model $model, array $data = []): ?Model
    {
        if (! $this->allowUpdating) {
            return null;
        }

        return tap($model, function ($model) use ($data) {
            $model->update(Arr::only($data, $this->getUpdatableColumns()));

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

        });
    }
}