Symfony4 - Как обновить доктрину ArrayCollection?

Если я хорошо помню, когда вы устанавливаете поле как ArrayCollection, это означает, что у вас есть отношение oneToMany между двумя объектами. Из вашего кода я могу сказать вам, что вы пытаетесь сохранить данные в неправильном объекте. Обычно вы добавляете owning_entity_id (1-to-N) в каждый элемент (1-to-N) и сохраняете его. В своем коде вы пытаетесь установить все ссылки сразу, чего никогда не произойдет. Удалите setFaavailability () или заново определите отношения сущностей.

Никогда не следует пытаться массово добавлять отношения внешнего ключа в одну функцию установки super duper. Прокрутите все элементы и установите ссылку на «родительский» объект.

Проблема в этой части: $faPerson = $em->getRepository(FaPerson::class)->findById($animal->faperson->getId());

Методы findBy* попытаются найти несколько объектов и вернуть их в Collection. Если вы ищете одного человека, вы можете использовать вместо него findOneById. Или (при условии, что id настроен как идентификатор в Doctrine) вы даже можете использовать метод find: $faPerson = $em->getRepository(FaPerson::class)->find($animal->faperson->getId());

некоторые общие комментарии:

1. В Doctrine вам никогда не придется работать с идентификаторами. Использовать сущность объекты! Вам нужно только findById, если вы получили идентификатор, например, из параметра запроса.
2. Вам следует пересмотреть наименования ваших переменных, чтобы было ясно, является ли это коллекцией ($ Availability) или отдельной ($ availability).
3. Всегда используйте методы получения / установки вместо полей (typepet vs getTypepet ()).
4. Вызовите flush () один в конце, чтобы обновить все сущности в одной транзакции.

Я переименовал переменные ниже, как я их понял. Однако я до сих пор не уверен, что возвращает $ animal-> faperson-> getFaavailabilities (), поскольку вначале вы хотите просмотреть результаты в цикле, а затем установить его на один с помощью setFaavailability ()?

 //Should be a Doctrine ArrayCollection
 $varFaavailabilities = $animal->faperson->getFaavailabilities();

 foreach($varFaavailability as $availability){
         if ($availability->getName() == $animal->getTypepet()->getName()) {

           //Why do you want to remove an element from the current loop?
           $varFaavailability->removeElement($availability);

           //No need to use Id
           $faPerson = $animal->getFaperson();

           //A single one?
           $faPerson->setFaavailability($availability);

           //More than one? addFaavailability should exist.
           $faPerson->addFaavailability($availability);

           $em->persist($faPerson);

         }
}
$em->flush();