Merge pull request #458 from BoShurik/type-guesser-fix

Fix type guesser. Ignore field MappingException.

Author: malarzm

Form/DoctrineMongoDBTypeGuesser.php

@@ -17,7 +17,6 @@
 use Symfony\Component\Form\Guess\Guess;
 use Symfony\Component\Form\Guess\TypeGuess;
 use Symfony\Component\Form\Guess\ValueGuess;
-use Symfony\Component\HttpKernel\Kernel;
 
 /**
  * Tries to guess form types according to ODM mappings
@@ -44,11 +43,15 @@ public function __construct(ManagerRegistry $registry)
     public function guessType($class, $property)
     {
         if (!$ret = $this->getMetadata($class)) {
-            return new TypeGuess($this->typeFQCN ? TextType::class : 'text', [], Guess::LOW_CONFIDENCE);
+            return;
         }
 
         list($metadata, $name) = $ret;
 
+        if (! $metadata->hasField($property)) {
+            return;
+        }
+
         if ($metadata->hasAssociation($property)) {
             $multiple = $metadata->isCollectionValuedAssociation($property);
             $mapping = $metadata->getFieldMapping($property);

Tests/Fixtures/Form/Guesser.php

@@ -47,4 +47,6 @@ class Guesser
 
     /** @ODM\Field(type="collection") */
     public $collectionField;
+
+    public $nonMappedField;
 }

Tests/Form/Type/GuesserTestType.php

@@ -26,6 +26,7 @@ public function buildForm(FormBuilderInterface $builder, array $options)
             ->add('intField')
             ->add('integerField')
             ->add('collectionField')
+            ->add('nonMappedField')
         ;
     }
 

Tests/Form/Type/TypeGuesserTest.php

@@ -70,6 +70,7 @@ public function testTypesShouldBeGuessedCorrectly()
         $this->assertType('integer', $form->get('intField'));
         $this->assertType('integer', $form->get('integerField'));
         $this->assertType('collection', $form->get('collectionField'));
+        $this->assertType('text', $form->get('nonMappedField'));
     }
 
     protected function assertType($type, $form)