Почему при вызове метода increment модели wasChanged принимает значение true у других аттрибутов?

Question

[mrFlyer]

Приветсвую.

$book = Book::find(300);
$book->full_name = Str::random();
$book->increment('some_value');

$book->wasChanged('full_name'); //true

Почему тут true? Ведь full_name в базе не обновился ...

Должно быть true если бы я после инкремента выполнил $book->save();

Comments

[N]

Возможно, потому что, ты взял экземпляр "book" и сначала присовил ему full_name, затем сделал increment и тем самым "сохранив" этот экземпляр "book", и, далее, "ожидаешь/проверяешь" на changed этот же экземпляр...не?

UPD:

Почему тут true? Ведь full_name в базе не обновился ...

По факту реально так? В базе действительно не обновился full_name?
Тогда странное поведение ORM...хотя тут как подумать...

[mrFlyer]

N, Не обновился, в том то и дело ...

У модели ORM метод incrementOrDecrement и там исполняется $this->syncChanges();

[N]

mrFlyer, Я не просматривал методы глубоко внутри Laravel в данном случае...
Тогда подождём ещё кого-нибудь...но поведение тогда нелогично выглядит...хотя с другой стороны - есть другие причины: "почему" не сохранилось full_name по факту...

Наверняка в bug report по Laravel это где-то уже рассмотрено...

[mrFlyer]

N, Ну причин много может быть. Не обязательно это full_name. При каком то условии нужно кинуть обновленеи в базу и продолжить работу с моделью. Самое паганое, что срабатыает еще и событие updated ... и вот там все не сохраненные данные уже начинают считаться что уже сохранены и запускается цепочка следующих событий.

[N]

mrFlyer, А, ну тут...уже подстраивайся под "цепочку" событий логики приложения...
Это нормально. Просто продумай "цепочку действий" в зависимости от событий и "хотелок"...

[mrFlyer]

N, Подстроился ... раньше increment не кидал событие updated, а в 10ке вот стал.
Ладно, может кто чего дельного подскажет. Пока похоже просто перекрытием метода закоментил выбрасывание события. Костыль.

[N]

mrFlyer,
Из документации:

Метод wasChanged определяет, были ли изменены какие-либо атрибуты при последнем сохранении модели в текущем цикле запроса.

Тобишь в Вашем примере кода - "цикл запроса" заканчивается на increment, но по факту full_name(из Ваших слов не меняется в БД).

Тут, как подумать:

- Либо "циклом" считается экземпляр модели(book);
- Либо "само сохранение по факту" (save) - чего по факту не сделано.

Отсюда и непонимание...+ при increment логично, что срабатывает событие update...

[N]

mrFlyer,

Подстроился ... раньше increment не кидал событие updated, а в 10ке вот стал.

Вот "по-человечески" 10-ка получается ведёт себя логичнее...

[mrFlyer]

ну теперь получается в событии updated нет понимания какие данные были сохранрены в бд а какие нет ...

[N]

mrFlyer, Мне кажется с хорошим переводом тут кроется истина...

https://laravel.com/docs/10.x/eloquent#examining-a...

[tukreb]

Это рукожопость ларки с её магией.
Если вы напрямую обращаетесь к свойства извне модели:

$book = Book::find(300);
$book->full_name = Str::random()

То это считает как модификация.
ОДНАКО!
если вы сделаете внутри модели

class Book extends Model

public function updateName(): void
{
    $this->full_name = Str::random()
}

И потом сделаете:

$book = Book::find(300);
$book->updateName();
$book->save(); //или push()

То такая херня даже не сохранится, так как гавно ларка не заметит изменений. При чём в некоторых ситуациях она всё же сохранится, если какие-то магические действия до этого делались. Или функция принимает множество значений. Например такое действия ларка заметит как модифакацию:

class Book extends Model

public function updateName(string $name): void
{
    $this->full_name = $name
}

Решение - не использовать Ларку, а Симфони и Доктрину.

П.С Я на такое гавно с ларкой попал в 5 версии (ещё есть где используется), и возможно это даже не баг, а фича

[N]

tukreb, Половина Laravel построена на Симфони...тут как рассматривать решения...есть и у Симвофни просто "изделия" - а не "решения"... :)

[tukreb]

N, построена да, только всё накручено своей магией, используется антипатер ActiveRecord и всё это гавно работает на какой то магией и матери.

[mrFlyer]

tukreb, Ну ларка хороша порогом входа. А теперь понимаю что пора на симфони переходить ... но емае, проект уже на 100 моделей... херова туча роутов и контроллеров.

[tukreb]

mrFlyer, я такой путь начинал с Yii конечон было не 100, а 50 моделей, потом на хайпе перевёл на ларку, а потом, когда понял в какой магический ад попал перешёл на симфони и ощутил нирвану в раю (был на дворе 19 год, сейчас там вообще блаженство). Заодно и весь проект на чистую архитектуру перевел, теперь любая смена фреймворка это практически минутное дело (утрировано конечно)

П.С В современном симфони больше ничего можно не настраивать в конфигах, всё делает через PHP атрибуты и это невероятно удобно + любая IDE понимает.

[N]

tukreb, Ну у каждого свой опыт))
Я тоже самое прошёл только с 2005 примерно...под каждые задачи - своё решение...

P.S.:
Лучший код - процедурный под задачу...но, кому - "оно" - надо... :)

[Владимир Римидалв]

tukreb, Претензии к ларавель в данном случае не уместны, это проблема eloquent'а

Answer 1

[mrFlyer]

В общем в гитхабе сказали делать так:

You need to use

$book->increment('some_value', extra: ['full_name' => $book->full_name]);
or
$book->increment('some_value', extra: ['full_name' => Str::random()]);
or
$book->increment('some_value', extra: $book->getDirty());

Т.е. указываем какие поля сохранять вместе с инкрементом ... так себе решение.

Answer 2

[pLavrenov]

Инкримент вызывает save(), он придуман для того чтобы упростить рутинную задачу меньшим количеством символов, а не вставлять его без дела в середину процесса.

есть два пути сохранения:
1)

$book = Book::find(300);
$book->full_name = Str::random()
$book->some_value++;
$book->save()

2)

Book::find(300)->update([
full_name => Str::random()
])

а ты смешал их вместе и ожидаешь логичного поведения