А
так он и должен отлавливать изменения до сейва. после сэйва это уже считай новая модель
Size: a a a
2021 June 12
PI
Спасибо! очень приятная цветовая гамма
В
ну вот. поэтому не нашел другого способа как кешировать до save и передавать отдельным параметром
А
ладно, сейчас запутаю вас только...
сделали, работает, супер.
только $origianl = clone $model; замените все же на $parent_id = $model->parent_id
сделали, работает, супер.
только $origianl = clone $model; замените все же на $parent_id = $model->parent_id
А
когда сервак будет перебирать последние байты оперативки он вам спасибо скажет, и поклон земной в лог бросит
АА
А в этом примере это очень критично по памяти?
A
некоторые message брокеры имеют ограничения по памяти для сообщения. это может быть критично там.
A
но тоже крайне редко
В
если бы это были операция на фронте полльзователями, там без вариантов, клонирование это была бы дорогая операция, а в админке, это редкая операция
А
конкретно в этом не думаю.
но бывают модели километровые. зачем выделять 20кб если можно выделить 20 байт?
но бывают модели километровые. зачем выделять 20кб если можно выделить 20 байт?
В
хотя за рекомендейшн спасибо, согласен с вами
АА
А в целом если реализовать слежение за какими то изменениями модели, то такую логику лучше в сервис вынести? Который получает внутри себя новую и старую модель и в зависимости от условий события генерирует
A
для простого круд генерить такое не стоит. слишком дорого.
A
а для чего-то сложного, там будут события другие.
A
Переслано от Adel
с нмии всё так. но надо генерить не MenuUpdated на всё. А MenuParentChanged. Или MenuTitleChanged например
АА
Я про то, что проверки на изменения в самом контроллере оставить или куда-то в отдельное место ввнести
АА
Вынести*
A
если очень надо, то конечно можно вынести. Просто это крайне редко вообще надо. выяснять что именно изменилось. если очень часто надо такое на проекте, может имеет смысл немного прееосмыслить код. или даже UI...
A
если изменения парента так важно, может в UI сделать отдельное - Поменять категорию. кнопку. сделать это отдельный действием
A
и тогда ничего вычислять не надо будет