Как каскадно сохранить модель и его связи в Laravel Eloquent ORM?

Question

[docxdocx]

Вопрос касается Eloquent ORM в Laravel 5.2. Допустим есть 3 таблицы (структура во вложенной картинке):

1. authors
   
2. books
   
3. pages
   

Для них есть 3 модельки Author, Book, Page. Связи в классах моделей условно я настроил так:

Author -> hasMany(Book:class)

Book -> hasMany(Page::class)
Book -> belongsTo(Author:class)

Page -> belongsTo(Book:class)

Связи настроены правильно, выборка из БД работает как надо.

В Doctrine ORM или Phalcon ORM есть каскадное сохранение связей. Условно это выглядит вот так:

$firstPage = new Page();
$secondPage = new Page();

$book = new Book();
$book->setPages([$firstPage, $secondPage]);

$author = new Author();

$author->setBooks([$book]);

if ($author->save()) {
    echo 'ok';
} else {
    echo 'error';
}

После этого в трех таблицах создадутся записи с нужными связями. Причем это все сохранится в транзакции БД. Т.е. сохранятся все или никто.

В модельках laravel есть метод push, но то не то. Он сохраняет это не в транзакции и не сохраняет связей.

Как такое сделать в laravel?

Answer 1

[vism]

Просто в Ларавель надо транзакцию запускать и комитить вручную.
Велкам!

Comments

[docxdocx]

Ну это да. Но прикиньте как в моем случае 4 сущности, получается нужно будет рекурсивно пробежатся по связям, поставить id где надо и везде проверять что вернул метод save обернув все это в транзакцию.

На самом деле, это можно сделать. Но я вряд ли сделаю лучше чем ребята из doctrine)

vism а как Вы сами такое делаете?

[vism]

docxdocx, Дык зачем.
Используйте push()

псевдо-говно-код, но думаю понятно) там уже хоть if, хоть try. как угодно.

beginTransaction();
....
if(push())
commit
else
rollback
....

[docxdocx]

vism, он не сохраняет связи. В таблице pages поле book_id сохранится как null. В таблице books поле author_id будет null. А если в таблице поля указаны как not null, то кинется QueryException.

[vism]

docxdocx, он как раз таки сохраняет связи. для этого он был задуман.

[docxdocx]

vism, почитайте код фреймворка. Попробуйте пару примеров. Он связи не сохраняет.

[docxdocx]

vism, в этом классе vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php метод push

[vism]

docxdocx, дело в том, что он сохраняет релейшены.
Аналогично тому как вы описали в доктрине.

Recursively Saving Models & Relationships
If you would like to save your model and all of its !!!associated!!! relationships, you may use the push method:

Вот серьезно, почитайте код фреймворка, я то читал его.

associate()

Answer 2

[Константин Б.]

Есть методы save, sync и еще несколько
А транзакция только одна на Try cath например
Какой смысл от транзакции если она для каждого запроса своя?))

Вот тебе пример

$this->db->beginTransaction();
try {
    $book = new Book;
    $author = $request->user();
    $author->books()->save($book);
    $this->db->commit();
} catch (\Exception $e) {
    $this->db->rollback();
    return 'Error';
}

return 'Success';

Comments

[docxdocx]

Вы меня не правильно поняли по транзакции. Нужно чтобы все сохранялось в одной транзакции. Как у Вас в примере.

Спасибо за ответ. Я не заметил что метод save кидает исключение QueryException. Я думал что он молча выдает true или false. Тогда с транзакцией вопрос решен. Спасибо.

Но непонятно как быть с каскадным сохранением. У Вас легкий случай. так как сохраняется только одна сущность. Мне же нужно сохранить сразу 3 сущности. Я конечно могу вызвать метод save несколько раз. Но мне от этого как-то не спокойно на душе. В doctrine или phalcon я привык в нужных классах биндить связи и в конце один раз вызвать метод save.

Ок, спасибо за ответ.

[vism]

docxdocx,

Я не заметил что метод save кидает исключение QueryException.

эээ, где?

[docxdocx]

vism, не понял Вас. Если не удалось выполнить SQL запрос метод save выдает исключение Illuminate\Database\QueryException. Возможно не сам метод save, а кто-то дальше который обращается в БД.

[vism]

docxdocx, Ну, да если произошла ошибка.
Тоесть вы хотите сказать, что save не возвращает false?

[vism]

docxdocx, Что вы будете делать, если save вернет false и вы закомитите транзакцию?)

[docxdocx]

vism, метод save не возвращает false, если например поле не должно быть null. Он либо вернет true, либо кинет exception QueryException. Ну это я судя по коду laravel говорю.

false возвращается только если событийные методы модели (типа saving, creating и т.д.) возвращают false. По событиям я пока не силен, но все равно это не особо влияет на сохранение если этих методов в модели нет.

[vism]

docxdocx, Ну да. Но... Если вы напишете код, а через год добавится событие... и вернет false..
Ох, ладно. слава богу это не в моих проектах.
Чем больше так пишут, тем больше мне работы)

[docxdocx]

vism, с этим да. Тут получается надо оборачивать в транзакцию и в ней проверять что возвращает save. Это прям боль и лапша получается ((

[Константин Б.]

docxdocx, код нужно писать так, чтобы такая ситуация в принципе не могла случиться, а вы пытаетесь отгородить себя не понятно от чего

[docxdocx]

Константин Б., и как же написать мой пример? 4 сущности и они связаны между собой.

Задача: Сохранить все транзакционно и малым количеством кода.

Всегда легко сказать что ваш код гавно. В вопросе я привел прекрасный пример, который прекрасно работает на doctrine2. Как такое хорошее решение сделать на eloquent?

[vism]

docxdocx, прочесть документацию :)
https://laravel.com/docs/5.7/eloquent-relationships

eloquent впринципе "казуальнее" доктрины)

[dmitriy]

Propel2 как раз делает то что вам нужно

[Константин Б.]

docxdocx, вы конечно туговаты, объясню вам ещё раз, писать код должны вы и управлять им, а не он вами как в вашем случае. Я вам говорю не должно быть такого что что-то там чего-то может вернуть, а что вдруг не может. Для исключения таких вот ошибок есть валидация с помощью которой вы управляете набором данных из которых будут созданы объекты, и тут у меня возникает вопрос, на каком основании у вас может пройти ситуация когда один из объектов может быть не создан, а уж тем более не сохранен? Ну как такое может быть допущено? Я вам привет пример того как работает транзакция, в случае ошибки будет произведен откат, ну а если у вас что-то не так настроено то никакой другой ORM вас не спасет

[docxdocx]

Константин Б., ну возможно я и туговат. Мой вопрос косательно конкретного ORM. Учитывая возможности Eloquent и возможные варианты у меня появляются сложности. А Вы как-то абстрактно уходите в философию. Ваш пример хороший, но он не покрывает все варианты событий. Как сказал vism, нужно проверять что возвращает метод save. Он в каких-то случаях может вернуть false. При таком случае Ваш код сохранит в БД не целостные данные.

Спасибо что показали как работает транзакция, но я и сам знаю как она работает. Вопрос именно про возможности Eloquent ORM.

объектов может быть не создан, а уж тем более не сохранен?

— есть определенные моменты когда save вернет false. Если в событийных методах вернется false. Только не говорите мне что нельзя пользоваться событиями модельки в Laravel.

Моя цель не понтануться и найти не соответствия в Вашем примере, а понять как сделать правильно. Поэтому прошу быть менее агрессивным.

[docxdocx]

vism, это да. Но я попробовал примеры, чето не сохраняет связи. Сегодня проверю еще раз и отпишусь. Спасибо за ответ.

[vism]

docxdocx, я под своим ответом уже по полочкам прямо разложил, какой метод нужно использовать и как пуш работает)

[vism]

Константин Б., да вобще-то тут вы кажетесь туговатым. Автор вопроса все по делу пишет, просто доки не читал внимательно.

Всмысле валидация все проверит?
А если база заглючит, а если другой программист накосячит, а если сатурн в созвездии костыля нарушит потоки данных?
Если есть ответ - он должен быть обработан.
Если есть эксепшн - он произойдет.

Привыкли к пет проектам блин, а потом когда проект вырастает авгиевы конюшни приходится разгребать.

[Константин Б.]

vism, это видимо такие как ты и привыкли писать не смотря на то что написано другими. Свой ответ мне обоснуй ПРИМЕРОМ КОДА того как нужно, иначе ты просто дешевая чмошка

[Константин Б.]

vism, если специально не сделать столбец нулевым, то тогда при сохранении будет кинуто исключение после которого сработает откат транзакции, автор же утверждает что нужно якобы каждую модель обернуть в транзакцию - по вашему это правильно? если вы с этим согласны тогда пожалуй вам можно идти в продавцы

[docxdocx]

Константин Б., я не говорил что каждую модель нужно обернуть в транзакцию. Я говорил что нужно проверять возвращаемый ответ метода save каждой модели. Это судя по возможностям eloquent.

И вообще почему ты такой злой))))

[vism]

Константин Б., У вас компилятор сломался. Не верно преобразовывает слова автора в мыслеобразы у вас в голове :)
Привести пример кода компилятора из вашей головы не представляется возможным)

[vism]

docxdocx, Я тоже злой кстати )))

[Константин Б.]

docxdocx, потому что ты не хрена не понимаешь)) Нет такого метода как тебе нужно. Если тебя не устраивает мой вариант с транзакцией делай проверку через if
if ($model->save())
if ($model->id)
И т.д. в зависимости о того, от чего ты хочешь обезопасить свое приложение

[docxdocx]

Константин Б., пффффффффффффффф. Все закройся.

[Константин Б.]

docxdocx, пошел в будку