Question

What’s wrong in my code ? I want to update or insert an Moteur object depending on url.
Thanks by advance.

/**
* @Route("/moteur/{moteurid}", name="moteur", requirements={"moteurid" = "\d+"}, defaults={"moteurid" = null})
* @Template()
*
* Cette page permet d'enregistrer de nouveaux moteurs (et de les éditer).
*/
public function moteurAction($moteurid)
{
    $args=array();
    $avertissement = null;
    if (!$this->get('security.context')->isGranted('ROLE_ADMIN'))
    {
        $avertissement = "Vous n'avez pas le droit d'accéder à cet espace.";
        return $this->redirect($this->generateUrl('index', array('avertissement' => $avertissement)));
    }

    $args['menu']['admin'] = 'selected';
    $obj = null;
    if ($moteurid == null)
    {
        $obj = new Moteur();
    }
    else
    {
        $obj = $this->getDoctrine()->getRepository('CreasixtineAFBundle:Moteur')->find($moteurid);
    }
    $form = $this->createForm(new FormMoteur(), $obj);
    $args['form'] = $form->createView();

    if ($this->getRequest()->getMethod() == 'POST')
    {
        $form->bindRequest($this->getRequest());

        if ($form->isValid())
        {
            $obj = $form->getData(); // Type Moteur()
            $pn = $obj->getPnid();

            $em = $this->getDoctrine()->getEntityManager();
            if ($moteurid == null)
            {
                $em->persist($obj);
                $avertissement = "Moteur créé !";
            }
            else 
            {
                // Rien, le moteur sera mis à jour avec flush()
                $avertissement = "Moteur mis à jour !";
            }
            foreach ($pn as $my_pn){$em->persist($my_pn);}
            $em->flush();

            return $this->redirect($this->generateUrl('admin', array('avertissement' => $avertissement)));
        }
        else
        {
            throw new Exception("Le formulaire n'est pas valide.");
        }
    }

    $contenu = $this->rendu($args, "formulaire_moteur.html.twig");
    return $contenu;
}

Answer 1

First, you don’t need this line as PHP5 natively passes object by reference :

$obj = $form->getData(); // Type Moteur()

Then, your relation between Moteur and Pn is a bit confusing. You get a Pn with getPnid() but you get an object you wanna persist ?

Anyway, these Pn objects should be persisted before Moteur, so here is what I would write :

if ($form->isValid())
{
    $em = $this->getDoctrine()->getEntityManager();

    $pn = $obj->getPnid();

    //Persist these related objects BEFORE Moteur
    foreach ($pn as $my_pn)
    {
        $em->persist($my_pn);
    }

    if ($moteurid == null)
    {
        $em->persist($obj);
        $avertissement = "Moteur créé !";
    }
    else 
    {
        // Rien, le moteur sera mis à jour avec flush()
        $avertissement = "Moteur mis à jour !";
    }
    $em->flush();

    return $this->redirect($this->generateUrl('admin', array('avertissement' => $avertissement)));
}
else
{
    throw new Exception("Le formulaire n'est pas valide.");
}