bug symfony#9338 [DoctrineBridge] Added type check to prevent calling clear() on arrays (bschussek)

This PR was merged into the 2.2 branch. Discussion ---------- [DoctrineBridge] Added type check to prevent calling clear() on arrays | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Commits ------- 4ff59d7 [DoctrineBridge] Added type check to prevent calling clear() on arrays

Author: fabpot

src/Symfony/Bridge/Doctrine/Form/EventListener/MergeDoctrineCollectionListener.php

@@ -11,6 +11,7 @@
 
 namespace Symfony\Bridge\Doctrine\Form\EventListener;
 
+use Doctrine\Common\Collections\Collection;
 use Symfony\Component\Form\FormEvents;
 use Symfony\Component\Form\FormEvent;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
@@ -40,7 +41,7 @@ public function onBind(FormEvent $event)
 
  // If all items were removed, call clear which has a higher
  // performance on persistent collections
- if ($collection && count($data) === 0) {
+ if ($collection instanceof Collection && count($data) === 0) {
  $collection->clear();
  }
  }