Minor performance gain by putting the ReflectionClass instance into a var instead of creating one each time we want to use it.

Author: WyriHaximus

src/functions.php

@@ -57,9 +57,10 @@ function throwable_decode($json)
 
     array_pop($properties);
 
+    $class = new ReflectionClass($json['class']);
     $throwable = new $json['class']();
     foreach ($properties as $key) {
-        if (!(new ReflectionClass($json['class']))->hasProperty($key)) {
+        if (!$class->hasProperty($key)) {
             continue;
         }
 

tests/MissingAttributes.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace WyriHaximus\Tests;
+
+final class MissingAttributes
+{
+    public $message;
+}

tests/ThrowableDecodeTest.php

@@ -27,4 +27,21 @@ public function test()
         self::assertSame([], $exception->getTrace());
         self::assertSame('whoops', $exception->getMessage());
     }
+
+    public function testWithMissingAttributes()
+    {
+        $json = [
+            'message' => 'whoops',
+            'code' => 13,
+            'file' => __FILE__,
+            'line' => 0,
+            'trace' => [],
+            'class' => 'WyriHaximus\Tests\MissingAttributes',
+        ];
+
+        /** @var MissingAttributes $exception */
+        $exception = WyriHaximus\throwable_decode($json);
+        self::assertSame('whoops', $exception->message);
+        self::assertFalse(isset($exception->code));
+    }
 }