Merge branch '6.4' into 7.0

* 6.4:
  [Cache] Fix test failure related to Redis6Proxy on appveyor
  [Messenger] Fix reading pending messages with Redis
  [Serializer] Revert #54488 to fix BC Break
  [Validator] reviewed Polish translation for unit 113
  [Validator] Missing translations for Croatian (hr)
  Add hint which version of Symfony not longer require proxy manager bridge
  review and fix hungarian validation message
  Update translation
  review uk translations v2
  [Validator] Update message translations for unit 113 (TLD)
  [Intl] Remove resources data from classmap generation

Author: nicolas-grekas

Mapping/Loader/AttributeLoader.php

@@ -125,7 +125,7 @@ public function loadClassMetadata(ClassMetadataInterface $classMetadata): bool
 
             $accessorOrMutator = preg_match('/^(get|is|has|set)(.+)$/i', $method->name, $matches);
             if ($accessorOrMutator) {
-                $attributeName = $reflectionClass->hasProperty($method->name) ? $method->name : lcfirst($matches[2]);
+                $attributeName = lcfirst($matches[2]);
 
                 if (isset($attributesMetadata[$attributeName])) {
                     $attributeMetadata = $attributesMetadata[$attributeName];

Normalizer/ObjectNormalizer.php

@@ -86,25 +86,17 @@ protected function extractAttributes(object $object, ?string $format = null, arr
 
             if (str_starts_with($name, 'get') || str_starts_with($name, 'has') || str_starts_with($name, 'can')) {
                 // getters, hassers and canners
-                $attributeName = $name;
+                $attributeName = substr($name, 3);
 
                 if (!$reflClass->hasProperty($attributeName)) {
-                    $attributeName = substr($attributeName, 3);
-
-                    if (!$reflClass->hasProperty($attributeName)) {
-                        $attributeName = lcfirst($attributeName);
-                    }
+                    $attributeName = lcfirst($attributeName);
                 }
             } elseif (str_starts_with($name, 'is')) {
                 // issers
-                $attributeName = $name;
+                $attributeName = substr($name, 2);
 
                 if (!$reflClass->hasProperty($attributeName)) {
-                    $attributeName = substr($attributeName, 2);
-
-                    if (!$reflClass->hasProperty($attributeName)) {
-                        $attributeName = lcfirst($attributeName);
-                    }
+                    $attributeName = lcfirst($attributeName);
                 }
             }
 

Tests/Fixtures/SamePropertyAsMethodDummy.php

@@ -42,9 +42,6 @@
 use Symfony\Component\Serializer\Tests\Fixtures\Php74Dummy;
 use Symfony\Component\Serializer\Tests\Fixtures\Php74DummyPrivate;
 use Symfony\Component\Serializer\Tests\Fixtures\Php80Dummy;
-use Symfony\Component\Serializer\Tests\Fixtures\SamePropertyAsMethodDummy;
-use Symfony\Component\Serializer\Tests\Fixtures\SamePropertyAsMethodWithMethodSerializedNameDummy;
-use Symfony\Component\Serializer\Tests\Fixtures\SamePropertyAsMethodWithPropertySerializedNameDummy;
 use Symfony\Component\Serializer\Tests\Fixtures\SiblingHolder;
 use Symfony\Component\Serializer\Tests\Normalizer\Features\AttributesTestTrait;
 use Symfony\Component\Serializer\Tests\Normalizer\Features\CacheableObjectAttributesTestTrait;
@@ -840,53 +837,6 @@ public function testNormalizeStdClass()
         $this->assertSame(['baz' => 'baz'], $this->normalizer->normalize($o2));
     }
 
-    public function testSamePropertyAsMethod()
-    {
-        $object = new SamePropertyAsMethodDummy('free_trial', 'has_subscribe', 'get_ready', 'is_active');
-        $expected = [
-            'freeTrial' => 'free_trial',
-            'hasSubscribe' => 'has_subscribe',
-            'getReady' => 'get_ready',
-            'isActive' => 'is_active',
-        ];
-
-        $this->assertSame($expected, $this->normalizer->normalize($object));
-    }
-
-    public function testSamePropertyAsMethodWithPropertySerializedName()
-    {
-        $classMetadataFactory = new ClassMetadataFactory(new AttributeLoader());
-        $this->normalizer = new ObjectNormalizer($classMetadataFactory, new MetadataAwareNameConverter($classMetadataFactory));
-        $this->normalizer->setSerializer($this->serializer);
-
-        $object = new SamePropertyAsMethodWithPropertySerializedNameDummy('free_trial', 'has_subscribe', 'get_ready', 'is_active');
-        $expected = [
-            'free_trial_property' => 'free_trial',
-            'has_subscribe_property' => 'has_subscribe',
-            'get_ready_property' => 'get_ready',
-            'is_active_property' => 'is_active',
-        ];
-
-        $this->assertSame($expected, $this->normalizer->normalize($object));
-    }
-
-    public function testSamePropertyAsMethodWithMethodSerializedName()
-    {
-        $classMetadataFactory = new ClassMetadataFactory(new AttributeLoader());
-        $this->normalizer = new ObjectNormalizer($classMetadataFactory, new MetadataAwareNameConverter($classMetadataFactory));
-        $this->normalizer->setSerializer($this->serializer);
-
-        $object = new SamePropertyAsMethodWithMethodSerializedNameDummy('free_trial', 'has_subscribe', 'get_ready', 'is_active');
-        $expected = [
-            'free_trial_method' => 'free_trial',
-            'has_subscribe_method' => 'has_subscribe',
-            'get_ready_method' => 'get_ready',
-            'is_active_method' => 'is_active',
-        ];
-
-        $this->assertSame($expected, $this->normalizer->normalize($object));
-    }
-
     public function testNormalizeWithoutSerializerSet()
     {
         $normalizer = new ObjectNormalizer(new ClassMetadataFactory(new AttributeLoader()));