Fixing bug where PropertyInfoLoader tried to add validation to non-existent properties

weaverryan · weaverryan · commit 89f72a139bfe · 2019-06-07T11:28:55.000-04:00
diff --git a/src/Symfony/Component/Validator/Mapping/Loader/PropertyInfoLoader.php b/src/Symfony/Component/Validator/Mapping/Loader/PropertyInfoLoader.php
@@ -60,6 +60,10 @@ public function loadClassMetadata(ClassMetadata $metadata)
                 continue;
             }
 
+            if (false === property_exists($className, $property)) {
+                continue;
+            }
+
             $types = $this->typeExtractor->getTypes($className, $property);
             if (null === $types) {
                 continue;
diff --git a/src/Symfony/Component/Validator/Tests/Fixtures/PropertyInfoLoaderEntity.php b/src/Symfony/Component/Validator/Tests/Fixtures/PropertyInfoLoaderEntity.php
@@ -48,4 +48,8 @@ class PropertyInfoLoaderEntity
     public $alreadyPartiallyMappedCollection;
 
     public $readOnly;
+
+    public function setNonExistentField()
+    {
+    }
 }
diff --git a/src/Symfony/Component/Validator/Tests/Mapping/Loader/PropertyInfoLoaderTest.php b/src/Symfony/Component/Validator/Tests/Mapping/Loader/PropertyInfoLoaderTest.php
@@ -46,6 +46,7 @@ public function testLoadClassMetadata()
                 'alreadyMappedNotBlank',
                 'alreadyPartiallyMappedCollection',
                 'readOnly',
+                'nonExistentField',
             ])
         ;
         $propertyInfoStub

Original file line number	Diff line number	Diff line change
`@@ -48,4 +48,8 @@ class PropertyInfoLoaderEntity`
`48`	`48`	`public $alreadyPartiallyMappedCollection;`
`49`	`49`
`50`	`50`	`public $readOnly;`
	`51`	`+`
	`52`	`+ public function setNonExistentField()`
	`53`	`+ {`
	`54`	`+ }`
`51`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -46,6 +46,7 @@ public function testLoadClassMetadata()`
`46`	`46`	`'alreadyMappedNotBlank',`
`47`	`47`	`'alreadyPartiallyMappedCollection',`
`48`	`48`	`'readOnly',`
	`49`	`+ 'nonExistentField',`
`49`	`50`	`])`
`50`	`51`	`;`
`51`	`52`	`$propertyInfoStub`