From ad4827d28188cc9a8dc72c9c00a77d8a19b04512 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Moritz=20Schr=C3=B6der?= <moritz.schroeder@molabs.de>
Date: Mon, 6 Jun 2016 12:56:58 +0200
Subject: [PATCH] Set option "by_reference" to false, so the underlying entity
 property (addXX(), removeXX()) gets changed not the collection (#424)

Removed unneccessary commented code
---
 Form/Type/ModelType.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Form/Type/ModelType.php b/Form/Type/ModelType.php
index d28970c..2f89162 100644
--- a/Form/Type/ModelType.php
+++ b/Form/Type/ModelType.php
@@ -121,7 +121,6 @@ class ModelType extends AbstractType
     {
         if ($options['multiple']) {
             $builder
-            #    ->addEventSubscriber(new MergeDoctrineCollectionListener())
                 ->addViewTransformer(new CollectionToArrayTransformer(), true)
             ;
         }
@@ -233,6 +232,7 @@ class ModelType extends AbstractType
             'choice_name' => $choiceName,
             'choice_value' => $choiceValue,
             'choice_translation_domain' => false,
+            'by_reference' => false,
         ]);
 
         $resolver->setRequired(array('class'));