Symfony2 Embedded forms, How to persist objects

Question

I am trying to implement Embedded Forms (Symfony2, 2.7), with Task and Tag entities, One2Many.

To save reference to the Task object into a Tag record, I am able to define Task's createAction() only by:

/**
 * Creates a new Task entity.
 *
 * @Route("/", name="MyName_Task_create")
 * @Method("POST")
 * @Template("MyNameBundleBlogBundle:Task:new.html.twig")
 */
public function createAction(Request $request)
{
    $task = new Task();
    $form = $this->createCreateForm($task);
    $form->handleRequest($request);

    if ($form->isValid()) {
        $em = $this->getDoctrine()->getManager();
        $tags = $task->getTags();
        foreach($tags as $tg){$tg->setTask($task); $em->persist($tg);}   // <-- do I really need to loop?
        $em->persist($task);
        $em->flush();

        return $this->redirect($this->generateUrl('MyName_Task_show', array('id' => $task->getId())));
    }

    return array(
        'entity' => $task,
        'form'   => $form->createView(),
    );
}

EDIT: I know it should work without the loop straightforwardly, but it does not. Question is: What should I look for which I might have written wrong? See related question

Note, I have:

class Task{
....
 /**
 *
 * @ORM\OneToMany(targetEntity="Tag", mappedBy="Task", cascade={"persist"} )
 */
private $Tags;
....
 /**
 * Add tag
 *
 * @param \MyName\Bundle\BlogBundle\Entity\Tag $tag
 *
 * @return Task
 */
public function addTag(\MyName\Bundle\BlogBundle\Entity\Tag $tag)
{
    $this->tags[] = $tag;
    $tag->setTask($this);
    return $this;
}
}

Persisting the task is all you need to do. The cascade=persist will take care of persisting the tags. — Cerad
– Cerad, Commented Feb 2, 2016 at 21:18
Thanks. Actually, I have a three layers Job/Task/Tag situation. Does it make a difference? I'll try to fix, or make another question. — mario
– mario, Commented Feb 3, 2016 at 16:29
As long as you have your cascade=persist set correctly on your relations then one persist is all you need. — Cerad
– Cerad, Commented Feb 3, 2016 at 16:31
@Cerad, all, if you spot something wrong stackoverflow.com/questions/35225089/…, I appreciate — mario
– mario, Commented Feb 5, 2016 at 13:15
Before starting a new question you really should finish this one. You have two correct answers. Really should accept one or state why not. — Cerad
– Cerad, Commented Feb 5, 2016 at 13:30

Turdaliev Nursultan · Accepted Answer · 2016-02-02 21:53:08Z

1

No, you don't need to loop through all tags and explicitly set task, Symfony will do that for you if you configure it correctly.

The only thing you need to add is set by_reference to false inside your form builder. In this case, symfony will explicitly will call setTask on every tag.

For more info 'by_reference'

answered Feb 2, 2016 at 21:53

Turdaliev Nursultan

2,5761 gold badge25 silver badges28 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Cerad Over a year ago

No need for by_reference here. His setTag implementation takes care of the cross referencing.

Turdaliev Nursultan Over a year ago

He can remove that line too.

Cerad Over a year ago

Not sure which line you are talking about but the $tag->setTask($this) line should stay and makes keeping the object references in sync much easier.

chalasr · Accepted Answer · 2016-02-02 21:47:48Z

0

According to @Cerad comment, the only thing you have to do is persist the Task.

All related tags will be automatically persisted, thanks to cascade={"persist"} in your association mapping.

Your code should be :

if ($form->isValid()) {
    $em = $this->getDoctrine()->getManager();
    $em->persist($task);
    $em->flush();

    return $this->redirect($this->generateUrl('MyName_Task_show', array('id' => $task->getId())));
}

See the Etablishing associations and Cascade operations chapters of Doctrine documentation.

answered Feb 2, 2016 at 21:47

chalasr

13.2k5 gold badges45 silver badges83 bronze badges

Collectives™ on Stack Overflow

Symfony2 Embedded forms, How to persist objects

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related