Почему сущность не видит связанную созданную сущность,а пытается создать новую?

Question

[HellWalk]

Вначале пример связи, где все работает нормально:

#[ORM\OneToMany(targetEntity: UploadFile::class, mappedBy: 'files', cascade: ['remove'])]
#[Serializer\Groups([Groups::LIST, Groups::DETAILS])]
#[ORM\JoinColumn(nullable: true)]
private ?Collection $files;

На фронте вначале загружается файл (и создается запись в базе), и затем приходит такой json на создание основной сущности:

// ...
            "files": [
                {
                    "id": "0198efe4-c9ef-76b2-b5d2-b3957f4d8cba",
                    "name": "1191fb00-8085-48c3-b728-cbd0d76847cb",
                    "originalName": "file",
                    "extension": "png",
                    "description": null,
                    "clientOriginalName": "file.png",
                    "createdDatetime": null,
                    "createdAt": "2025-08-28T11:56:45+03:00",
                    "createdBy": null
                }
            ]
// ...

Все создается успешно - симфони понимает, что это данные по уже существующей записи в базе, и просто создает связь.

Но есть другая сущность, тоже со связью с UploadFile, но OneToOne:

#[ORM\OneToOne(cascade: ['remove'])]
#[Serializer\Groups([Groups::LIST, Groups::DETAILS])]
#[ORM\JoinColumn(nullable: true)]
public ?UploadFile $file = null;

С фронта приходят данные:

"file": {
            "id": "0198efdc-5e9b-7d3b-86c6-47b68f0f665d",
            "name": "2c830a0d-c8ba-4655-afcd-566da343cac6",
            "originalName": "file",
            "extension": "png",
            "description": null,
            "clientOriginalName": "file.png",
            "createdDatetime": null,
            "createdAt": "2025-08-28T11:47:33+03:00",
            "createdBy": null
        },

И сохранение сущности ломается:

A new entity was found through the relationship 'App\Entity\Document#file' that was not configured to cascade persist operations for entity: App\Entity\UploadFile@1847. To solve this issue: Either explicitly call EntityManager#persist() on this unknown entity or configure cascade persist this association in the mapping for example @ManyToOne(..,cascade={"persist"}). If you cannot find out which entity causes the problem implement 'App\Entity\UploadFile#__toString()' to get a clue

Если добавляю cascade={"persist"}, то на один файл создается две записи - первая, корректная, созданная во время загрузки файла. И вторая, при создании основной сущности, когда симфони не видит, что связанная сущность уже существует, и создает новую.

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

P.S. Все сущности создаются через сериалайзер:

$data = json_decode($request->getContent(), true);
$entity = $this->serializer->denormalize($data, $entityClass);
$errors = $this->validator->validate($entity);

    if (count($errors) > 0) {
        throw new ConstraintViolationListException($errors);
    }

$repository->add($entity);

Сохраняются также единым образом:

public function add(object $entity, bool $flush = true): void
    {
        if ($entity instanceof CreatedAtInterface) {
            if ($entity->getCreatedAt() === null) {
                $entity->setCreatedAt();
            }
        }
        $this->getEntityManager()->persist($entity);
        if ($flush) {
            $this->getEntityManager()->flush();
        }
    }

Т.е. из json через сериалайзер создается сущность, потом просто сохраняется

Comments

[tukreb]

С чего вы взяли, что Симофни, или в частности Doctrine, должны знать что сущность существует? Вы делали на это проверку?
Ваша задача как программиста сделать проверки, существует ли уже связь и стоит ли её сохранять. Если такая есть, то скорее всего она написана не правильно.

Если добавляю cascade={"persist"}, то на один файл создается две записи - первая, корректная, созданная во время загрузки файла. И вторая, при создании основной сущности, когда симфони не видит, что связанная сущность уже существует, и создает новую.

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

[HellWalk]

tukreb, 1) я на этом проекте вообще ничего не писал 2) ответ из разряда "ошибка находится где-то между монитором и креслом"

[tukreb]

HellWalk, я дополнил ответ, так или иначе вы не предоставили код как вы сохраняете сущности.

[HellWalk]

tukreb, добавил описание - метод add в repository

[tukreb]

То есть вы хотите сказать, что вы просто получаете json, прогоняете через serializer, а потом просто сохраняете, да и ещё всегда в кучу транзакций вместо одной единой?
А бизнес логика где?

[HellWalk]

tukreb,
> вы просто получаете json, прогоняете через serializer, а потом просто сохраняете

да, так проект написан, и мне это не нравится также, как и вам. Я привык к 1) нормально валидации запроса 2) создание dto от запроса 3) потом от dto ручками создается entity

> да и ещё всегда в кучу транзакций вместо одной единой

если на один запрос делается один flush - то какие проблемы?

> А бизнес логика где?

да тут круды в основном, нет особой логики

[tukreb]

Ну что я могу вам сказать, поздравляю :) Вы нашли ограничения работы Doctrine без бизнес логики.

У вас два пути, начать писать бизнес логику, либо раз вы говорите, что через OneToMany всё работает как надо, то просто удалить возможность работы напрямую через OneToOne.
Зачем вам иметь возможно сохранять файл 2 способами? Для одного и для множества, если можно использовать всегда для множества.

[HellWalk]

tukreb,
> Зачем вам иметь возможно сохранять файл 2 способами?

Сущность файлов одна. Но она по разному связана с другими сущностями в проекте.

[tukreb]

HellWalk, ну тогда создайте отдельный Сервис для сохранения файлов, где сможете добавить бизнес логику.

[HellWalk]

tukreb, нет проблемы взять все и переписать как надо, чтобы работало) Вопрос был в том, нет ли в симфони/доктрине какой-то особой настройки, которую вот просто добавляешь, и она понимает, что вот это класс уже созданной сущности, и её не нужно создавать. Но нужно привязать к текущей.

Сама по себе задача решается элементарно:

if ($file = $entity->getFile()) {
    $entity->setFile(
        $uploadFileRepository->findOneBy(['id' => $file->getId()])
    );
}

Но возникает вопрос как это добавить (с минимальными правками), когда всё в проекте максимально абстрактно и универсально json->denormalize->persist->flush

Склоняюсь к написанию своего нормалайзера, и в denormalize() сделать всю нужную логику

[tukreb]

HellWalk, чтобы доктрина понимала, что объект уже есть, она должна иметь к нему доступ на всех этапах.
Возьмите дебагер и посмотрите на каком шаге доктрина перестаёт видеть объект.
Возможно вам нужно будет вручную задать связи, а не надеятся, что Доктрина поймёт

#[ORM\OneToOne(cascade: ['remove'])]
#[ORM\JoinColumn(nullable: true)]
public ?UploadFile $file = null;

Что тут имеется ввиду, обратится к сущности UploadFile и в таблицу за атрибутом file, а в ней к другой связи и т.д. То есть важно пройтись по всем связям и настроить их вручную, прописав нужные связи и параметры, а не надеятся, что Доктрина на основе свойства $file обратится к file. Так вы можете случайно и найдете, где у вас чего и не хватает. Либо наоборот где-то прописана лишняя связь из-за дублируется.

Хотя тут можешь случится петля связей :) если переусердствуете.

[HellWalk]

tukreb, прописать более конкретную связь - это первое, что я попробовал. Бестолку

#[ORM\OneToOne(cascade: ['remove'])]
#[Serializer\Groups([Groups::LIST, Groups::DETAILS])]
#[ORM\JoinColumn(name: 'file_id', referencedColumnName: 'id', nullable: true)]
public ?UploadFile $file = null;

Вас куда-то не туда занесло. Что происходит мне совершенно понятно - доктрина видит какой-то класс, и по-умолчанию считает его новым. Чтобы она поняла, что такие данные уже есть - нужно получить объект из базы:

if ($file = $entity->getFile()) {
    $entity->setFile(
        $uploadFileRepository->findOneBy(['id' => $file->getId()])
    );
}

И все она сразу понимает.

Answer 1

[HellWalk]

Какой-то отдельной опции, которая бы указала доктрине "не создавай новую сущность по этим данным, найди её в базе по id" нет, по этому, чтобы исправить эту ошибку необходимо написать свой нормалайзер, который будет вручную получать сущность из базы по id и привязывать её к родительской сущности:

use Symfony\Component\Serializer\Normalizer\DenormalizerInterface;
use Symfony\Component\Serializer\Normalizer\NormalizerInterface;

class EntityNormalizer implements DenormalizerInterface, NormalizerInterface
{
    public function __construct(
        private readonly ObjectNormalizer $normalizer,
        private readonly EntityManagerInterface $em,
    ) {
    }

    public function denormalize(mixed $data, string $type, ?string $format = null, array $context = []): mixed
    {
        $object = $this->normalizer->denormalize($data, $type, $format, $context);
        
        if ($file = $entity->getFile()) {
            $entity->setFile(
                $this-em->getRepository(UploadFile::class)->findOneBy(['id' => $file->getId()])
            );
        }

        return $object;
    }

    // ...
}