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

У меня есть член моей сущности - arrayCollection. С классическим конструктором форм работает нормально, я могу выбрать несколько элементов и сохранить их. Но когда я пытаюсь обновить объект в контроллере, я получаю сообщение об ошибке: «Вызов функции-члена setFaavailability () в массиве».

Резюме моей организации:

/**
 * @ORM\ManyToOne(targetEntity="App\Entity\FaAvailability", 
   inversedBy="faavailability")
 * @ORM\JoinColumn(nullable=true)
 * @ORM\Column(type="array")
 */
 public $faavailability;

 /**
 * @return mixed
 */
public function getFaavailability()
{
    return $this->faavailability;
}

/**
 * @param mixed $faavailability
 */
public function setFaavailability($faavailability)
{
    $this->faavailability = $faavailability;
}

В моем контроллере:

 $varFaavailability = $animal->faperson->getFaavailability();
 foreach($varFaavailability as $availability){
         if($availability->getName() == $animal->typepet->getName()){
           $varFaavailability->removeElement($availability);
           $faPerson = $em->getRepository(FaPerson::class) >findById($animal->faperson->getId());
           $faPerson->setFaavailability($varFaavailability);
           $em->persist($faPerson);
           $em->flush();
         }
}

Любые идеи ?


php symfony doctrine arraycollection
person user3659832    schedule 27.12.2018    source источник
comment
Попробуйте удалить @ORM \ Column (type = array) из аннотаций $ faavailability.   -  person Jannes Botis    schedule 28.12.2018
comment
Отношение ManyToOne, не возвращает arrayCollection, Only ManyToMany или oneToMany   -  person hous    schedule 29.12.2018

Ответы (3)


arrow_upward
-1
arrow_downward

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

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

person John Krit    schedule 27.12.2018

arrow_upward
-1
arrow_downward

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

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

person Timo    schedule 28.12.2018
comment
С вашим решением у меня нет ошибки, но она ничего не сохраняет. Ты знаешь почему ? - person user3659832; 28.12.2018
comment
Вам, вероятно, потребуется настроить каскадирование, чтобы также сохранить доступность, когда вы сохраняете человека. Вы можете сделать это с помощью постоянного каскада: doctrine-project.org/projects/doctrine-orm/en/2.6/reference/ - person Timo; 02.01.2019

arrow_upward
-1
arrow_downward

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

  1. В Doctrine вам никогда не придется работать с идентификаторами. Используйте объекты сущности! Вам нужно только findById, если вы получили идентификатор, например, из параметра запроса.
  2. Вам следует пересмотреть наименования ваших переменных, чтобы было ясно, является ли это коллекцией ($ available) или отдельной ($ 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();
person Arne    schedule 28.12.2018
comment
Да ты прав, этот код лучше. Я просто хочу удалить один элемент из arrayCollection, когда условие if истинно. если я выполняю $ faPerson- ›setFaavailability ($ varFaavailability), в свойстве Faavailability ничего не сохраняется. Я меняю на $ faPerson- ›setFaavailability ($ varFaavailability-› removeElement ($ availability)) Сохраняется значение «1» ... Это очень странно. У тебя есть идея? - person user3659832; 28.12.2018
comment
Просто вызовите $ varFaavailability- ›removeElement ($ availability); без setFaavailability () - person Arne; 29.12.2018