Fix deprecation notice when trying to serialize a callable (#1732)

src/Serializer/AbstractSerializer.php

@@ -261,7 +261,7 @@ protected function serializeValue($value)
         }
 
         try {
-            if (\is_callable($value)) {
+            if (@\is_callable($value)) {
                 return $this->serializeCallable($value);
             }
         } catch (\Throwable $exception) {

tests/phpt/test_callable_serialization.phpt

@@ -0,0 +1,66 @@
+--TEST--
+Test that capturing an exception with a callable looking argument does not trigger an deprecation "Deprecated: Use of "static" in callables is deprecated"
+--INI--
+error_reporting=E_ALL
+zend.exception_ignore_args=0
+--FILE--
+<?php
+
+declare(strict_types=1);
+
+namespace Sentry\Tests;
+
+use Exception;
+use Sentry\ClientBuilder;
+use Sentry\Event;
+use Sentry\Options;
+use Sentry\SentrySdk;
+use Sentry\Transport\Result;
+use Sentry\Transport\ResultStatus;
+use Sentry\Transport\TransportInterface;
+
+$vendor = __DIR__;
+
+while (!file_exists($vendor . '/vendor')) {
+    $vendor = \dirname($vendor);
+}
+
+require $vendor . '/vendor/autoload.php';
+
+$transport = new class implements TransportInterface {
+    public function send(Event $event): Result
+    {
+        echo 'Transport called' . PHP_EOL;
+
+        return new Result(ResultStatus::success());
+    }
+
+    public function close(?int $timeout = null): Result
+    {
+        return new Result(ResultStatus::success());
+    }
+};
+
+$options = [
+    'dsn' => 'http://public@example.com/sentry/1',
+];
+
+$client = ClientBuilder::create($options)
+    ->setTransport($transport)
+    ->getClient();
+
+SentrySdk::getCurrentHub()->bindClient($client);
+
+class Foo {
+    function __construct(string $bar) {
+        SentrySdk::getCurrentHub()->captureException(new Exception('doh!'));
+    }
+}
+
+new Foo('static::sort');
+
+echo 'Triggered capture exception' . PHP_EOL;
+?>
+--EXPECT--
+Transport called
+Triggered capture exception