Adding to a Collection: Cascade Persist

Hello super person,

So in my project one product can have many pictures with titles and descriptions. And I'm able to get the remover method to work.

BUT it seems that I can't get the adder to work. if I dump $form->getData(); it seems that everything I add gets ignored. While the multiple pictures I manually added in the database doesn't get ignored, and I can edit them in the form. In "inspect element" in the HTML form. I can see that embedded forms gets the necessary [0], [1] etc in the name, but it isn't visible when I post.

Any ideas what I accidentally missed?

Hello, I'm wondering if I could submit forms inside a collection separately? I have very long form collection with buttons to save each subform (Basically filling and validating the form at once would be difficult). So clicking the button suppose to only submit corresponding subform, but it submits whole collection.

`<?php

namespace AppBundle\Controller;

use AppBundle\Entity\Address;
use AppBundle\Entity\Applicant;
use AppBundle\Entity\Company;
use AppBundle\Entity\Director;
use AppBundle\Entity\User;
use AppBundle\Form\UserType;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
use Symfony\Component\HttpFoundation\Request;

/**

• Address controller.
  *
• @Route("admin")
  */

class UserController extends Controller
{

/**
 * Lists all address entities.
 *
 * @Route("/", name="admin_index")
 * @Method("GET")
 */
public function indexAction(Request $request)
{
    $em = $this->getDoctrine()->getManager();

    $user = $this->getUser();

    if(!count($user->getApplicants())) {
        $app = new Applicant();
        $app->setUser($user);
        $user->setApplicants($app);
    }

    if(!count($user->getAddresses())) {
        $address = new Address();
        $address->setUser($user);
        $user->setAddresses($address);
    }

    if(!count($user->getCompanies())) {
        $company = new Company();
        $company->setUser($user);
        $user->setCompanies($company);
    }

    if(!count($user->getDirectors())) {
        $director = new Director();
        $director->setUser($user);
        $user->setDirectors($director);
    }

    $form = $this->createForm('AppBundle\Form\UserType', $user);

    $form->handleRequest($request);

    if ($form->isSubmitted() && $form->isValid()) {

        if ($form->getClickedButton() && 'submitApplicants' === $form->getClickedButton()->getName()) {

            $applicant = $form->getData()->getApplicants()[0];

            $applicant->setUser($user);

            $em->persist($applicant);

            $em->flush();

            return $this->render('admin/index.html.twig', [
                'form' => $form->createView()
            ]);

        }
        if ($form->getClickedButton() && 'submitAddresses' === $form->getClickedButton()->getName()) {

            $address = $form->getData()->getAddresses()[0];

            $em->persist($address);

            $em->flush($address);

            return $this->render('admin/index.html.twig', [
                'form' => $form->createView()
            ]);

        }

        if ($form->getClickedButton() && 'submitCompanies' === $form->getClickedButton()->getName()) {

            $company = $form->getData()->getCompanies()[0];

            $em->persist($company);
            $em->flush($company);

            return $this->render('admin/index.html.twig', [
                'form' => $form->createView()
            ]);

        }

        if ($form->getClickedButton() && 'submitDirectors' === $form->getClickedButton()->getName()) {

            $director = $form->getData()->getDirectors()[0];

            $em->persist($director);

            $em->flush($director);

            return $this->render('admin/index.html.twig', [
                'form' => $form->createView()
            ]);

        }

        //$em->flush();
    }

    return $this->render('admin/index.html.twig', [
        'form' => $form->createView()
    ]);
}

}`

Hello,

I have a problem with the addGenusScientist and removeGenusScientist methods. In my application, I use checkboxes to set or remove entries to or from the mapping table (I call it genusScientist table here for better understanding). When adding, the idea is, that I call addGenusScientist in genus, pass it a genusScientist object where only the user is set. In my opinion, this doesn't work correctly with the current implementation:

public function addGenusScientist(GenusScientist $genusScientist)
{
    if ($this->genusScientists->contains($genusScientist)) {
        return;
    }

    $this->genusScientists[] = $genusScientist;
    // needed to update the owning side of the relationship!
    $genusScientist->setGenus($this);
}

public function removeGenusScientist(GenusScientist $genusScientist)
{
    if (!$this->genusScientists->contains($genusScientist)) {
        return;
    }

    $this->genusScientists->removeElement($genusScientist);
    // needed to update the owning side of the relationship!
    $genusScientist->setGenus(null);
}

I have 3 problems with that:

1. the setGenus line should be the first action in the method, otherwise contains will never return true, even if the combination of genus and user already exists.

2. contains apparently only then finds the entry in the collection, when ALL fields in $genusScientist are set exactly set as in the matching object in $this->genusScientists, i.e. not only user and genus have to be set, but also id and yearsStudies. If not, contains will not find them and I will have duplicate entries in my genusScientist table.

3. the same problem with contains happens in the removeGenusScientist method. Unless I fetched the genusScientist object from the database (i.e. all fields are set) before passing to the removeGenusScientist, contains will never find it. Even worse, the removeElement will then not work.

In my code, I implemented my own contains method, which only searches for the keys (genus and user) in this "mapping table with additional fields". But this feels strange, since the way it is implemented here can also be found on other examples on the internet.

Did I misunderstand the goal of the if-statements in add-/removeGenusScientists or is there really a bug in this code?

Hey there

My guts are telling me that you may forgotten to setup the inverse side of the relationship. Can you double check that?

Cheers!

Hmm, interesting. My second thought is that you didn't add the `cascade={"persist"}` to the property, but if you did then I would have to check your code so I can tell what's wrong. Maybe you could upload it to Github?

Or just take some screenshots to your entities and controller's action

Hmm, your relationships looks good to me, probably the error comes from processing the form submit. Try dumping your objects throughout the process so you can gather more information of how your objects are being built. You would have to call persist directly to "Bestelorder" and "Artikel"