bug symfony#48978 [Serializer] use method_exists() instead of catching reflection exceptions (xabbuh)

This PR was merged into the 5.4 branch.

Discussion
----------

[Serializer] use method_exists() instead of catching reflection exceptions

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | symfony#48233 (comment)
| License       | MIT
| Doc PR        |

Commits
-------

0e883a9 use method_exists() instead of catching reflection exceptions

Author: fabpot

src/Symfony/Component/Serializer/Normalizer/GetSetMethodNormalizer.php

@@ -152,14 +152,7 @@ protected function setAttributeValue(object $object, string $attribute, $value,
         $key = \get_class($object).':'.$setter;
 
         if (!isset(self::$setterAccessibleCache[$key])) {
-            try {
-                // We have to use is_callable() here since method_exists()
-                // does not "see" protected/private methods
-                self::$setterAccessibleCache[$key] = \is_callable([$object, $setter]) && !(new \ReflectionMethod($object, $setter))->isStatic();
-            } catch (\ReflectionException $e) {
-                // Method does not exist in the class, probably a magic method
-                self::$setterAccessibleCache[$key] = false;
-            }
+            self::$setterAccessibleCache[$key] = method_exists($object, $setter) && \is_callable([$object, $setter]) && !(new \ReflectionMethod($object, $setter))->isStatic();
         }
 
         if (self::$setterAccessibleCache[$key]) {