Почему не срабатывает update?

Question

[DarkByte2015]

У меня есть вот такой экшн

public function actionEditAuthor() {
	$author = new ArticleAuthor();
	$author->load(Yii::$app->request->post());
	$author->update();
	return $this->redirect(['article/edit-step3', 'id' => $author->article_id]);
}

Но почему-то не работает обновление (update возвращает false). Данные приходят в post запросе 100%. Дебажил.

P.S. Сейчас кто-нибудь по любому скажет что надо не делать new, а вместо этого сначала выбрать нужный объект запросом. Но я не понимаю зачем для редактирования мне лишний select запрос? Это же не экономично. Мне бы хотелось чтобы просто делался один update запрос и все. ID (PK) модели я тоже передаю с формы.

Answer 1

[Дмитрий]

Добрый день.

Но я не понимаю зачем для редактирования мне лишний select запрос?

А как скрипт определит, какую запись редактировать?
Поэтому false и возвращает.
Не выдумывайте и не занимайтесь ерундой.
Получайте сначала модель, которую будете редактировать, а уж потом сохраняйте изменения.
То, что Вы сейчас делаете - для создания новой записи. Для сохранения данных используйте save(), хотя и для обновления тоже надо вызывать save().

p.s.
Вот так будет новая запись сделана в базе

public function actionAddAuthor() {
  $author = new ArticleAuthor();
  if($author->load(Yii::$app->request->post() && $author->save()){
      return $this->redirect(['article/edit-step3', 'id' => $author->article_id]);
}
}

А вот так обновление

public function actionEditAuthor($id) {
  $author = $this->findModel($id); // тут select запрос, который возвращает модель для редактирования.
  if($author->load(Yii::$app->request->post() && $author->save()){
      return $this->redirect(['article/edit-step3', 'id' => $author->article_id]);
}
}

Comments

[DarkByte2015]

А как скрипт определить, какую запись редактировать?

Разве он не может подставить в where - PK который как я сказал я передаю с формы в запросе?

update article_author set ... where id = 123 # этого было бы достаточно

Не выдумывайте и не занимайтесь ерундой.

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

То, что Вы сейчас делаете - для создания новой записи. Для сохранения данных используйте save(), хотя и для обновления тоже надо вызывать save().

А зачем тогда вообще нужен метод update?

[Игорь Воротнёв]

DarkByte2015, www.yiiframework.com/forum/index.php/topic/16257-d...

[Дмитрий]

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

DarkByte2015, зачем тогда Вы его используете?

Разве он не может подставить в where - PK?

Как скрипт определит, какой id ему передан?

А зачем тогда вообще нужен метод update?

update()
updateAll()

[DarkByte2015]

зачем тогда Вы его используете?

Затем что начальство сказало. Глаза б мои его не видели.

Как скрипт определит, какой id ему передан?

В третий раз повторяю: id передается в запросе и в модели он заполнен после вызова load(). А Yii по идее должен как-то уметь определить какой ключ является первичным и всего лишь подставить его в where при update.

update()
updateAll()

Да я в курсе что так можно, но так длиннее... Я просто надеялся что Yii сам может такие вещи делать, но видимо моим надеждам не суждено сбыться. :(

[Дмитрий]

DarkByte2015, и метод save()
Если новая запись он вызовет insert(), а если запись обновляется, то он вызовет метод update()

[Дмитрий]

Я просто надеялся что Yii сам может такие вещи делать, но видимо моим надеждам не суждено сбыться. :(

DarkByte2015, уже ответил, метод save() всё сделает. Так что не занимайтесь ерундой и не изобретайте каменный велосипед.

[DarkByte2015]

slo_nik, я уже сказал. Я не против метода save. Мне просто не нравится делать лишний select в начале.

[Дмитрий]

DarkByte2015, вот скучно Вам на работе. Уже бы сделали и забыли. Сидите, изобретаете каменный велосипед. Ну попробуйте updateAll() использовать.
Но!
Используя findModel($id) Вы получаете модель, которую надо обновить, а используя $author = new Author() Вы получаете пустую модель. Так что Вы собираетесь обновлять? Тут не обновлять надо, а делать новую запись в таблицу и этим занимается insert() а не update().

[DarkByte2015]

slo_nik, еще раз для понимания. Если я в начале например сделаю так:

$author = ArticleAuthor::findOne(['id' => $id]);
$author->load(Yii::$app->request->post());

То мне вернется моделька.

Если я сделаю так:

$author = new ArticleAuthor();
$author->load(Yii::$app->request->post()); // повторюсь в n-ый раз - я передаю id тоже в запросе!!!

То мне вернется на 100% такая же моделька со всеми заполненными полями (в т.ч. id). Или же они чем-то будут отличаться? Почему в первом случае вызов save будет делать update, а во втором будет пытаться сделать insert? Может у них какое-то внутреннее состояние меняется когда ты их выбираешь запросом? Может они потом тот же where который ты использовал для их выборки используют и в save?

[Дмитрий]

DarkByte2015, чтобы использовать update() Вам надо сделать что-то подобное

$model->name = // данные из post запроса
$model->update();

а если в модели надо обновить 20 параметров? Будете писать так

$model->name = ***;
$model->surname = ***;
$model->patronymic = ***
// ещё куча параметров и только в конце
$model->update();

Eсли делать нормально, так как в документации и как говорю я, то это всё сократится до двух строк, вызов метода findModel($id) и $model=>save();

[davidnum95]

DarkByte2015, разница в том, что при $author = new ArticleAuthor(); модель никак не привязана к бд, это просто класс с заполненными полями. А при $author = ArticleAuthor::findOne(['id' => $id]); Yii знает что эта модель уже есть в бд и ее нужно просто обновить.

[Дмитрий]

Или же они чем-то будут отличаться?

DarkByte2015, открывайте исходный код и изучайте.
А лучше задайте этот вопрос разработчикам yii2, спросите, почему они такие глупые?

[DarkByte2015]

а если в модели надо обновить 20 параметров? Будете писать так

Что вы вообще несете? Я же загружаю данные из пост запроса вызовом

$author->load(Yii::$app->request->post());

...погодите ка... Он что отлавливает как-то непосредственное присвоения руками, а на загрузку через метод load не реагирует чтоли?

[ThunderCat]

DarkByte2015,
На мотив "не кочегары мы не плотники"
В целях природы обуздания,
В целях рассеять неученья Тьму!
Берем картину мирозданья-да,
И тупо смотрим, что к чему...

$author1->load(Yii::$app->request->post());
var_dump($autor1);
$author2 = ArticleAuthor::findOne(['id' => $id]);
var_dump($autor2);

[DarkByte2015]

ThunderCat, абсолютно ни одного отличия, кроме разве что того что в первом случае те поля которые числовые - будут строками с таким же значением. (id = "123" вместо id = 123) Но пхп язык с динамической типизацией и он обычно сам кастит такие вещи при необходимости, так что врятли в этом дело.

[Дмитрий]

...погодите ка... Он что отлавливает как-то непосредственное присвоения руками, а на загрузку через метод load не реагирует чтоли?

DarkByte2015, вот прежде чем возмущаться тупостью php, глупостью разработчиков - изучайте документацию, общайтесь на формах и смотрите что и как делают другие.

[DarkByte2015]

slo_nik, я как бы и сейчас общаюсь на форуме, не?))))

[Дмитрий]

DarkByte2015, делайте как в документации и не морочьте себе голову. Будет и быстрей и правильней. Но если хотите изобретать велосипед - дело Ваше.

[Дмитрий]

я как бы и сейчас общаюсь на форуме, не?))))

DarkByte2015, Вам ответили уже не только я, что Вы не туда смотрите, но нет, Вы опять за своё.

[Дмитрий]

DarkByte2015, тем более, что используя метод findModel($id) Вы получаете данные, которые выводите в форме. Смотрите и меняете на новые значения. Так что, select запрос лишний? Или как Вы собираетесь узнать, что за данные в таблице?

[DarkByte2015]

slo_nik, данные на форму выводит другой запрос. А в запросе на редактирование мне не нужен селект.

[Дмитрий]

DarkByte2015,
Раз
Два

данные на форму выводит другой запрос.

Вот это надо было сразу писать, в вопросе.
Покажите, как у Вас вообще работает форма обновления, с моделью, контролером и представлением. Только код добавьте в вопрос.

[DarkByte2015]

slo_nik, ладно хрен с ним мне надоело.