Simplified "Ensuring the database persistence" example

Florian Körner · Florian Körner · commit f4306ac502d2 · 2013-11-21T22:16:26.000+01:00
diff --git a/cookbook/form/form_collections.rst b/cookbook/form/form_collections.rst
@@ -676,11 +676,11 @@ the relationship between the removed ``Tag`` and ``Task`` object.
                 throw $this->createNotFoundException('No task found for is '.$id);
             }
 
-            $originalTags = array();
+            $originalTags = new \Doctrine\Common\Collections\ArrayCollection();
 
-            // Create an array of the current Tag objects in the database
+            // Create an ArrayCollection of the current Tag objects in the database
             foreach ($task->getTags() as $tag) {
-                $originalTags[] = $tag;
+                $originalTags->add($tag);
             }
 
             $editForm = $this->createForm(new TaskType(), $task);
@@ -689,27 +689,20 @@ the relationship between the removed ``Tag`` and ``Task`` object.
 
             if ($editForm->isValid()) {
 
-                // filter $originalTags to contain tags no longer present
-                foreach ($task->getTags() as $tag) {
-                    foreach ($originalTags as $key => $toDel) {
-                        if ($toDel->getId() === $tag->getId()) {
-                            unset($originalTags[$key]);
-                        }
-                    }
-                }
-
                 // remove the relationship between the tag and the Task
                 foreach ($originalTags as $tag) {
-                    // remove the Task from the Tag
-                    $tag->getTasks()->removeElement($task);
+                    if ($task->getTags()->contains($tag) == false) {
+                        // remove the Task from the Tag
+                        $tag->getTasks()->removeElement($task);
 
-                    // if it were a ManyToOne relationship, remove the relationship like this
-                    // $tag->setTask(null);
+                        // if it were a ManyToOne relationship, remove the relationship like this
+                        // $tag->setTask(null);
 
-                    $em->persist($tag);
+                        $em->persist($tag);
 
-                    // if you wanted to delete the Tag entirely, you can also do that
-                    // $em->remove($tag);
+                        // if you wanted to delete the Tag entirely, you can also do that
+                        // $em->remove($tag);
+                    }
                 }
 
                 $em->persist($task);
