Событие для сущности перед её обновлением?

Question

[Игорь]

Задача банальная, перед обновлением сущности вызвать метод.

При первоначальном сохранении, срабатывает событие prePersist
Последующие persist не канают, об этом говорится ив документации

prePersist - The prePersist event occurs for a given entity before the respective EntityManager persist operation for that entity is executed. It should be noted that this event is only triggered on initial persist of an entity (i.e. it does not trigger on future updates).

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

app.listeners.category.save_image_in_listener:
    class: App\Listeners\Category\SaveImageInListener
    tags:
      - name: 'doctrine.orm.entity_listener'
        event: 'prePersist'
        method: 'save'
        entity: App\Entity\CategoryProduct
        lazy: true
      - name: 'doctrine.orm.entity_listener'
        event: 'preUpdate'    # не работает
        method: 'save'
        entity: App\Entity\CategoryProduct
        lazy: true

Сейчас пока использую событие postLoad
Но оно срабатывает многократно.

Comments

[Игорь]

Забавно...

postLoad - The postLoad event occurs for an entity after the entity has been loaded into the current EntityManager from the database or after the refresh operation has been applied to it.

[Максим Федоров]

tommy-vercetti, либо в UoW поменять данные, чтобы он видел forChanges данные

[Игорь]

tommy-vercetti,

prePersist срабатывает перед инсертом новой сущности в базу.

Да, это самое первое о чем я знал.
Об этом я писал в вопросе.

Когда я вставил запись и отработал prePersist все ок, я не меняю данные после flush .

[Игорь]

Каюсь, был не прав.
preUpdate прекрасно работает.

Ну я додумался ловить preUpdate не изменив сам объект. )))

Задумка в том, что в preUpdate я изменяю старое изображение на новое.
Но если я только это делаю, не изменив другие данные в сущности, то естественно не видать мне preUpdate

[Максим Федоров]

Игорь,

Задумка в том, что в preUpdate я изменяю старое изображение на новое.

подпишитесь на preFlush() и вызывайте UoW в листенере, насильно в нем меняя список на изменение через метод $uow->recomputeSingleEntityChangeSet()

То есть в Unit of Work вы запихнете изменения и уже flush будет знать, что нужно обновить, тк в переменной UoW $uow->entityChangeSets будут лежать данные к изменению = к апдейту

[Игорь]

Максим Федоров, Проверю, отпишу.

[Игорь]

Максим Федоров, Это ответ.

Сделал что-то типа

/**
     * @param CategoryProduct $entity
     * @param PreFlushEventArgs $eventArgs
     * @return string
     */
    public function save(CategoryProduct $entity, PreFlushEventArgs $eventArgs)
    {
        if ($this->support()) {
            foreach ($this->white_mime_list as $mmt => $ext) {
                if ($mmt == $this->getMimeType()) {
                    $file_name = sprintf("%s-%s.%s", md5($this->getImageContentBase64()), uniqid(), $ext);
                    $relative_path = sprintf("/sale_point_%s/categories/%s", $entity->getSalePoint()->getId(), $file_name);
                    $absolute_path = sprintf("%s%s", $this->images_directory, $relative_path);

                    if (!is_dir(dirname($absolute_path))) {
                        mkdir(dirname($absolute_path), 0777, true);
                    }

                    $entity->setImage($relative_path);
                    $eventArgs->getEntityManager()->getUnitOfWork()->recomputeSingleEntityChangeSet($eventArgs->getEntityManager()->getClassMetadata(CategoryProduct::class), $entity);
                    @file_put_contents($absolute_path, base64_decode($this->getImageContentBase64()));
                }
            }
        }
    }

Я передаю изображение следующим образом.

{
    "category_id": "1",
    "name": "Бургеры",
    "description": "Описание",
    "image": {
        "mime_type": "image/png",
        "content": "iVBORw0KGgoAAAANSUhEUgAAAeAAAAHgCAYAAAB91L6VAAAABmJLR0QA/wD/AP+gvaeTAAAgAElEQVR4nO3dd5weVb348c8mgYQklFCkRQkI0psNRBRFkCsoRcSOFdsV2733p17Ua0WxUr0KdizXXlCQJqCCgKIoAgIWQgdpCUkgjd3fH2f3xSbu7uyZ58ycmWc+79frvNC8Zp/znTPzzPeZmVNAkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJkiRJdRvIHYA0bAPgIGA/YAtgY2AuMCtnUGq9JcCtwF3ATcB5wJnAfTmDkqTcBoDDgQuAFcCQxVJDWQH8AjgMSeqgPYFfkf9ibOl2uRzYB0nqgGnAKeS/8Foso8sJhHNTkvrSHMI7uNwXW4tlrHIu4RyVamEnLNVlDnAJsH3uQKQJXAvsDdyfOxD1vym5A1AnTAW+iclXzbcD8B18HK0aTM0dgDrhROBluYOQJumxwLrA2bkDUX/zEbSq9jRCb2epbZ4GXJw7CPUvE7CqdgmwV+4gpBIuB55C6KAlJec7YFXpcEy+aq89gENyB6H+ZQJWlY7OHYDUI89hSa2zPuWml1xKmBThyTgPtHo3i3AunUg4t2LPx+U4NlhSyxxJ/MXuFmDXHMGqE3YlnGOx5+XLcwQrSWWdTtxF7iFglyyRqkt2I/5O+GtZIlXf8x2wqvKYyO1PBa6qIhBplD8Cp0X+zaOrCESSqnI9cXcZT8oTpjpoD+LOzevyhClJ5Swi7iI3O0+Y6qDZxJ2bi/KEqX7nRByqSuzkBZ6LqpPnp7LzHbAkSRmYgCVJysAELElSBiZgSZIyMAFLkpSBCViSpAxMwJIkZWACliQpAxOwJEkZmIAlScrABCxJUgYmYEmSMjABS5KUgQlYkqQMTMCSJGVgApYkKQMTsCRJGZiAVYV1Ird/oJIopPEtjtx+diVRqNNMwKrCDpHb31FJFNL4bo/cPvaclgqZgFWFl0VuH3sxlHoV+6Mv9pyWpNrtDiwDhiLKV3MEqk47nbhzdCmwa5ZIJWkSHg/cStyFbQjvLlS/I4k/T28BdssRrCSNZRawJ3Ay8Xe+Q8ByYE7tUavr1gdWEH++LgNOIpzzdsxSTwZyBxBhA+AgYD9gC2BjYC4hAai9fkE4plLdLgCemTsI9WQJ4anbXcBNwHnAmcB9OYPqFwPA4YQvSplfq5bml0OQ8jiM/Oe/JX1ZQfhhfxgqbU/gV+Q/mJbqyqW06ymM+s/F5P8eWKorlwP7oEmbBpxC/gNnqb7shZTX3uT/HliqLycQcosmMIfwDD/3wbJUXz6D1AwnkP/7YKm+nIMdPsc1B7iW/AfJUn05G5iK1AzTCBfn3N8LS/XlGkzC/2Ia3vl2pVwDrIfULHMI52bu74el+nIOPo5exUnkPyiWek58k6+aag7eCXelnIAAeBr5D4al+nI8PnZW803Dd8JdKU9FXEL+A2GprlwKPB2pXfYALiL/98dSXbmMjg+DfD75D4IlfVkOnI+TbKj9DiWcy04E1J/lUDLKnf2dCq79FhOmgruTR6aCOwu4P2dQUmJzCFPh7s+qU+E6H3S7ZZ0KN2cCXp8wf2dsb7RlwOeBbxF6LS5JHJdUtxnAvOEyF3gUYe7z1cu04W3XGv67tYf/bQhYMPxvy4AHgUHg3jHKPYQfTDcC8/H7o3abBexIWFHtDcD0yL9fQfgxleWGIWcCPpKwJmeMW4HnAn9KH45UuXnAzsNlJx5JuptmiwjuJiTi+YRx+H8mfL/+QUjiUlvsCvyM8CM2xpHAN9KH02yxC2I/BOySJVIp3jzCr/JTgF8T7lBzv++KKYsJnVROA14NbJe0daRq7AYsJe5c/1qWSDO7iLhGctyWmmoAeDzwduC7wG3kT6BVlLuBnwDvJMzj7bAyNVHsvBIX5Akzr+uJa6Qn5QlTGtP6wBHAqYRXI7mTY46yCPgp8HriH/tJVdmDuPP4ujxh5rWIuEayt6Fym0e4+/sNsJL8CbBJZRD4A/BBQqcYKZfZxP+Q7JzYL7iUw1zgbYR1YwfJn+jaUq4BPoDvjpWH+aWADaSmmknoeGTSTVOuBN6Kq9CoPuaXAjaQmmYH4DjCeNncSasfy1JCJ7X9yD8JkPqb+aWADaQmWBN4JfA78ieoLpW/AG/Bvh2qhvmlgA2knNYhvNu9mfzJqMtlIXAi9qJWWuaXAjaQctiCMKb8AfInH8sjZSnwZcJrAKlXrcgvOd/DxO6074zUi82BY4CjCI+dm2YBYTrIGwmLWtwzqozM4fwgIVE9NPw3iwjDoQaA9Yb/bU3C/LgzWHUe6Y2Gy6N5ZArMjarcoZIGgW8ThjPdkDkWtVcr8osJWP1uY+DdwBsJSSm3WwjzLV81XP5CSLwLJvibqswiJOLteWSO6l2BLcn/fVtJmJ/3Q4QfJVIM80uBVjwiUGutDXyUMKdxrseqC4Gzgf8h9PxtyzCctYG9CZOO/IQwBWWuNlwO/C/NvFtXc5lfCthAqsIUwhje26k/WSwhrMZyNGHhkH6aK3lb4LXA9whLt9XdtvcD/wGsUfWOqi+YXwrYQErtqdQ/nOgG=="
    }
}

Изображения очень маленькие, поэтому в теле JSON

[Игорь]

Пришлось переработать

/**
     * @param CategoryProduct $entity
     * @param PreFlushEventArgs $eventArgs
     * @return string
     */
    public function save(CategoryProduct $entity, $eventArgs)
    {
        if ($this->computed === false) {
            if ($this->support()) {
                foreach ($this->white_mime_list as $mmt => $ext) {
                    if ($mmt == $this->getMimeType()) {
                        $file_name = sprintf("%s-%s.%s", md5($this->getImageContentBase64()), uniqid(), $ext);
                        $relative_path = sprintf("/sale_point_%s/categories/%s", $entity->getSalePoint()->getId(), $file_name);
                        $absolute_path = sprintf("%s%s", $this->images_directory, $relative_path);

                        if (!is_dir(dirname($absolute_path))) {
                            mkdir(dirname($absolute_path), 0777, true);
                        }

                        $entity->setImage($relative_path);

                        $this->computed = true;
                        $eventArgs->getEntityManager()->getUnitOfWork()->computeChangeSets();

                        @file_put_contents($absolute_path, base64_decode($this->getImageContentBase64()));
                    }
                }
            }
        }
    }

Так как

$eventArgs->getEntityManager()->getUnitOfWork()->computeChangeSets();

Снова вызывает preFlush
Это приводит к зацикливанию.

[Максим Федоров]

Игорь, ну и отлично :)

[Максим Федоров]

Игорь, так стоп

Пришлось переработать Так как Это приводит к зацикливанию.

$this->computed = true;

Очень плохое решение.

Вы же помните, что в контейнере все сервисы shared by default. Это значит, что если вам понадобится, чтобы на preFlush() сработал ваш листенер у нескольких сущностей ( вцикле например) или вообще у разных сущностей, то ваш листенер не сработает после первого раза из-за значения $copmuted

А решение всего-то в том, чтобы проверить -- соответствует ли значение в переменной сущности тому значению, что хотите засеттить. И если значение тоже -- просто сделать return, не делая recompute

[Игорь]

Максим Федоров, Точно.

Опять в точку.
Об этом я не подумал.

Вот полный код листенера

<?php


namespace App\Listeners\Category;


use App\Entity\CategoryProduct;
use Doctrine\ORM\Event\PreFlushEventArgs;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\RequestStack;

/**
 * Class SaveImageInListener
 * @package App\Listeners\Category
 */
class SaveImageInListener
{
    private array $white_mime_list = [
        "image/jpeg" => "jpeg",
        "image/jpg" => "jpg",
        "image/png" => "png",
    ];
    private $images_directory;
    /** @var Request|null */
    private ?Request $request;
    /** @var bool */
    private bool $computed;

    /**
     * SaveImageInListener constructor.
     * @param RequestStack $requestStack
     * @param $images_directory
     */
    public function __construct(RequestStack $requestStack, $images_directory)
    {
        $this->images_directory = $images_directory;
        $this->request = $requestStack->getCurrentRequest();
        $this->computed = false;
    }


    /**
     * @param CategoryProduct $entity
     * @param PreFlushEventArgs $eventArgs
     * @return string
     */
    public function save(CategoryProduct $entity, $eventArgs)
    {
        if ($this->computed === false) {
            if ($this->support()) {
                foreach ($this->white_mime_list as $mmt => $ext) {
                    if ($mmt == $this->getMimeType()) {
                        $file_name = sprintf("%s-%s.%s", md5($this->getImageContentBase64()), uniqid(), $ext);
                        $relative_path = sprintf("/sale_point_%s/categories/%s", $entity->getSalePoint()->getId(), $file_name);
                        $absolute_path = sprintf("%s%s", $this->images_directory, $relative_path);

                        if (!is_dir(dirname($absolute_path))) {
                            mkdir(dirname($absolute_path), 0777, true);
                        }

                        // Удаление предыдущей версии файла, если это обновление
                        if (is_file($this->images_directory . $entity->getImage()))
                            @unlink($this->images_directory . $entity->getImage());

                        $entity->setImage($relative_path);

                        $this->computed = true;
                        $eventArgs->getEntityManager()->getUnitOfWork()->computeChangeSets();

                        @file_put_contents($absolute_path, base64_decode($this->getImageContentBase64()));
                    }
                }
            }
        }
    }

    /**
     * @return bool
     */
    private function support()
    {
        if ($this->request->get("image", false)) {
            return true;
        }
        return false;
    }

    /**
     * @return mixed
     */
    private function getImageContentBase64()
    {
        $request_image = $this->request->get("image", false);
        return $request_image["content"];
    }

    /**
     * @return mixed
     */
    private function getMimeType()
    {
        $request_image = $this->request->get("image", false);
        return $request_image["mime_type"];
    }
}

Этот слушатель специально сделан для загрузки изображения.
В цикле не планируется модификация.

А если и будет, то там есть условие

...
if ($this->support())
...

Оно контролирует.

Но я понял в чем моя ошибка.

Спасибо.

[Максим Федоров]

Игорь,

Об этом я не подумал.

Это нормально, потому ревью и показывает себя хорошо :)