Каким образом можно изменить стандартный запрос на сохранение к базе в Laravel?

Question

[Михаил]

Всем доброго времени суток!
Суть вопроса такова:
Есть таблица в БД, в которой хранятся значения сетевых адаптеров, ввиду этого в таблице не создавалось поле "id" а ключевым полем было назначено поле "mac", поскольку физические адреса сами по себе уникальны.
Для сохранения данных о сетевом адаптере после редактирования я написал следующую функцию:

public function saveNic (Request $request)
    {
        $data = $request->all();
        $host = $data['host'];
        unset ($data['host']);
        $tmp= Networkcard::select(['mac', 'name', 'network', 'wol'])->where('mac',$data['mac'])->get();
        $nic=$tmp[0];
        $nic->fill($data);
        $nic->save();
        return redirect("/host/".$host);
    }

Как видно в коде, проблему с выборкой из базы я обошел следующим образом (если это можно улучшить, подскажите пожалуйста как).
Но при сохранении и вызове метода save() Laravel формирует запрос в котором ключевым полем считает поле id. Ввиду этого резонно говорит мне об ошибке.

Подскажите пожалуйста как можно изменить запрос инструментами Laravel на сохранение данных, или может добавить какие то параметры к методу save(), что бы при запросе не искалось соответствие по полю mac, а не по полю id?

Comments

[JhaoDa]

Вы тоже из секты Не читающих Документацию?

Answer 1

[riot26]

В модель Networkcard добавить:
protected $primaryKey = 'mac';

Answer 2

[Станислав Почепко]

https://laravel.com/docs/5.4/eloquent

Primary Keys

Eloquent will also assume that each table has a primary key column named id. You may define a $primaryKey property to override this convention.

Answer 3

[Сергей Семенко]

В модель:

protected $primaryKey = 'mac';
public $incrementing = false;

И как можно так говнокодить на laravel?

Примерно так можно сделать:

Networkcard::where('mac', $request->get('mac'))->update($request->all());
return redirect()->back(); // Думаю это подойдёт

В модели определить $fillable (поля для быстрого обновление)

Comments

[Михаил]

Сергей Семенко Ну вот как то так и можно, я же написал есть предложения как улучшить, я с радостью почитаю)
А с Laravel только начал знакомство, раньше писал на чистом php где был сам себе творец, тут же пока ни ... не понятно как оно работает, мануалы все разнятся приходится писать так как вижу.

[Alex Wells]

Еще можно так:

return redirect()->route('host', ['id' => $request->id]);

А в декларацию всунуть NetworkCardCreateRequest, его наследовать от вашего BaseRequest, где находится код отдачи fields.

[Михаил]

Alex Wells: Спасибо большое! Не очень понял про декларацию, но постараюсь разобраться)

[Михаил]

Сергей Семенко Спасибо, попробовал реализовать ваше предложение, но в связи с тем что в метод update передается весь запрос, появляется ошибка из-за отсутствующего поля _token в БД.
По правилам фреймворка токены должны писаться в базу?

[Alex Wells]

Михаил: Какая у вас версия? По дефолту _token не включен в $request->all(), если не ошибаюсь. А про декларацию, я об этом:

public function saveNetworkCard(NetworkCardCreateRequest $request)

Перед этим php artisan make:request NetworkCardCreateRequest
Потом его модифицируешь как надо. У меня, например, есть BaseRequest, в котором есть метод needed():

public function needed() {
        return $this->only($this->getFields());
    }

    protected function getFields() {
        return array_keys($this->container->call([$this, 'rules']));
    }

Таким образом мы заполняем только валидацию, а в коде, в нужных местах, вместо $request->all() используем $request->needed()

Такой подход в разы удобней чем $fillable, так как $fillable - общий для всего проекта, а needed() - конкретно под реквест.

[Михаил]

Alex Wells: ну при использовании метода fill а потом save, при отладке смотрел содержимое, токен есть но на базу не влияет, а при использовании update($request->all()) я так понимаю он напрямую пытается писать, так как ошибка вполне прозрачно говорит о том, что не найден столбец _token. Версия 5.4 ставил самую свежую, только осваиваю)
За пример спасибо, выглядит сложно, но я думаю разберусь )