bug #41104 Fix return type in isAllowedProperty method on ReflectionExtractor class (Tomanhez)

fabpot · fabpot · commit 3f572553cf9c · 2021-05-06T13:14:11.000+02:00
This PR was merged into the 5.2 branch. Discussion ---------- Fix return type in isAllowedProperty method on ReflectionExtractor class | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #41103 | License | MIT | Doc PR | symfony/symfony-docs#...  Commits ------- 8c0e8f7 Fix return type on isAllowedProperty method
diff --git a/src/Symfony/Component/PropertyInfo/Extractor/ReflectionExtractor.php b/src/Symfony/Component/PropertyInfo/Extractor/ReflectionExtractor.php
@@ -592,7 +592,7 @@ private function isAllowedProperty(string $class, string $property): bool
         try {
             $reflectionProperty = new \ReflectionProperty($class, $property);
 
-            return $reflectionProperty->getModifiers() & $this->propertyReflectionFlags;
+            return (bool) ($reflectionProperty->getModifiers() & $this->propertyReflectionFlags);
         } catch (\ReflectionException $e) {
             // Return false if the property doesn't exist
         }

Original file line number	Diff line number	Diff line change
`@@ -592,7 +592,7 @@ private function isAllowedProperty(string $class, string $property): bool`
`592`	`592`	`try {`
`593`	`593`	`$reflectionProperty = new \ReflectionProperty($class, $property);`
`594`	`594`
`595`		`- return $reflectionProperty->getModifiers() & $this->propertyReflectionFlags;`
	`595`	`+ return (bool) ($reflectionProperty->getModifiers() & $this->propertyReflectionFlags);`
`596`	`596`	`} catch (\ReflectionException $e) {`
`597`	`597`	`// Return false if the property doesn't exist`
`598`	`598`	`}`