bug #46511 [Serializer] Added missing __call to TraceableNormalizer and TraceableSerializer (danielburger1337)

derrabus · derrabus · commit 631122917440 · 2022-05-30T20:56:10.000+02:00
This PR was squashed before being merged into the 6.1 branch. Discussion ---------- [Serializer] Added missing __call to TraceableNormalizer and TraceableSerializer | Q | A | ------------- | --- | Branch? | 6.1 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | #46471 | License | MIT | Doc PR | none Added missing `__call` method to the new TraceableNormalizer and TraceableSerializer. This method was forgotten and breaks custom normalizers that have public methods outside of the NormalizerInterface. This approach is already being used for TreaceableAuthenticator, TraceableEventDispatcher, LoggingTranslator etc... Commits ------- 74f3c37 [Serializer] Added missing __call to TraceableNormalizer and TraceableSerializer
diff --git a/src/Symfony/Component/Serializer/Debug/TraceableNormalizer.php b/src/Symfony/Component/Serializer/Debug/TraceableNormalizer.php
@@ -150,4 +150,12 @@ public function hasCacheableSupportsMethod(): bool
     {
         return $this->normalizer instanceof CacheableSupportsMethodInterface && $this->normalizer->hasCacheableSupportsMethod();
     }
+
+    /**
+     * Proxies all method calls to the original normalizer.
+     */
+    public function __call(string $method, array $arguments): mixed
+    {
+        return $this->normalizer->{$method}(...$arguments);
+    }
 }
diff --git a/src/Symfony/Component/Serializer/Debug/TraceableSerializer.php b/src/Symfony/Component/Serializer/Debug/TraceableSerializer.php
@@ -165,4 +165,12 @@ public function supportsDecoding(string $format, array $context = []): bool
     {
         return $this->serializer->supportsDecoding($format, $context);
     }
+
+    /**
+     * Proxies all method calls to the original serializer.
+     */
+    public function __call(string $method, array $arguments): mixed
+    {
+        return $this->serializer->{$method}(...$arguments);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -150,4 +150,12 @@ public function hasCacheableSupportsMethod(): bool`
`150`	`150`	`{`
`151`	`151`	`return $this->normalizer instanceof CacheableSupportsMethodInterface && $this->normalizer->hasCacheableSupportsMethod();`
`152`	`152`	`}`
	`153`	`+`
	`154`	`+ /**`
	`155`	`+ * Proxies all method calls to the original normalizer.`
	`156`	`+ */`
	`157`	`+ public function __call(string $method, array $arguments): mixed`
	`158`	`+ {`
	`159`	`+ return $this->normalizer->{$method}(...$arguments);`
	`160`	`+ }`
`153`	`161`	`}`
Original file line number	Diff line number	Diff line change
`@@ -165,4 +165,12 @@ public function supportsDecoding(string $format, array $context = []): bool`
`165`	`165`	`{`
`166`	`166`	`return $this->serializer->supportsDecoding($format, $context);`
`167`	`167`	`}`
	`168`	`+`
	`169`	`+ /**`
	`170`	`+ * Proxies all method calls to the original serializer.`
	`171`	`+ */`
	`172`	`+ public function __call(string $method, array $arguments): mixed`
	`173`	`+ {`
	`174`	`+ return $this->serializer->{$method}(...$arguments);`
	`175`	`+ }`
`168`	`176`	`}`