[DoctrineBridge] added a bc layer for symfony#18069

HeahDude · HeahDude · commit 88b6e79e30ea · 2016-04-29T23:39:44.000+02:00
diff --git a/src/Symfony/Bridge/Doctrine/Form/EventListener/MergeDoctrineCollectionListener.php b/src/Symfony/Bridge/Doctrine/Form/EventListener/MergeDoctrineCollectionListener.php
@@ -29,6 +29,7 @@ class MergeDoctrineCollectionListener implements EventSubscriberInterface
 {
     // Keeps BC. To be removed in 4.0
     private $bc = true;
+    private $bcLayer = false;
 
     public static function getSubscribedEvents()
     {
@@ -45,11 +46,13 @@ public static function getSubscribedEvents()
 
     public function onSubmit(FormEvent $event)
     {
-        // If onBind() is overridden then logic has been executed
         if ($this->bc) {
             @trigger_error('The onBind() method is deprecated since version 3.1 and will be removed in 4.0. Use the onSubmit() method instead.', E_USER_DEPRECATED);
 
-            return;
+            if (!$this->bcLayer) {
+                // If onBind() is overridden then logic has been executed
+                return;
+            }
         }
 
         $collection = $event->getForm()->getData();
@@ -72,6 +75,9 @@ public function onBind()
     {
         if (__CLASS__ === get_class($this)) {
             $this->bc = false;
+        } else {
+            // parent::onBind() has been called
+            $this->bcLayer = true;
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,7 @@ class MergeDoctrineCollectionListener implements EventSubscriberInterface`
`29`	`29`	`{`
`30`	`30`	`// Keeps BC. To be removed in 4.0`
`31`	`31`	`private $bc = true;`
	`32`	`+ private $bcLayer = false;`
`32`	`33`
`33`	`34`	`public static function getSubscribedEvents()`
`34`	`35`	`{`
`@@ -45,11 +46,13 @@ public static function getSubscribedEvents()`
`45`	`46`
`46`	`47`	`public function onSubmit(FormEvent $event)`
`47`	`48`	`{`
`48`		`- // If onBind() is overridden then logic has been executed`
`49`	`49`	`if ($this->bc) {`
`50`	`50`	`@trigger_error('The onBind() method is deprecated since version 3.1 and will be removed in 4.0. Use the onSubmit() method instead.', E_USER_DEPRECATED);`
`51`	`51`
`52`		`- return;`
	`52`	`+ if (!$this->bcLayer) {`
	`53`	`+ // If onBind() is overridden then logic has been executed`
	`54`	`+ return;`
	`55`	`+ }`
`53`	`56`	`}`
`54`	`57`
`55`	`58`	`$collection = $event->getForm()->getData();`
`@@ -72,6 +75,9 @@ public function onBind()`
`72`	`75`	`{`
`73`	`76`	`if (__CLASS__ === get_class($this)) {`
`74`	`77`	`$this->bc = false;`
	`78`	`+ } else {`
	`79`	`+ // parent::onBind() has been called`
	`80`	`+ $this->bcLayer = true;`
`75`	`81`	`}`
`76`	`82`	`}`
`77`	`83`	`}`