Fix case when SerializableClosure uses as class property

Author: Dry7

.gitignore

@@ -2,3 +2,4 @@
 composer.lock
 /phpunit.xml
 .phpunit.result.cache
+.idea

src/Serializers/Native.php

@@ -194,13 +194,33 @@ public function __unserialize($data)
 
         $this->closure = $this->closure->bindTo($this->code['this'], $this->code['scope']);
 
+        $this->bindObjectsIfNeeded();
+
+        $this->code = $this->code['function'];
+    }
+
+    private function bindObjectsIfNeeded()
+    {
         if (! empty($this->code['objects'])) {
-            foreach ($this->code['objects'] as $item) {
-                $item['property']->setValue($item['instance'], $item['object']->getClosure());
-            }
+            $this->bindObjects();
         }
+    }
 
-        $this->code = $this->code['function'];
+    private function bindObjects()
+    {
+        foreach ($this->code['objects'] as $item) {
+            $item['property']->setValue($item['instance'], $this->calculateObjectValue($item));
+        }
+    }
+
+    private function calculateObjectValue($item)
+    {
+        return $this->isSerializableClosure($item['object']) ? $item['object'] : $item['object']->getClosure();
+    }
+
+    private function isSerializableClosure($object): bool
+    {
+        return $object instanceof SerializableClosure || $object instanceof UnsignedSerializableClosure;
     }
 
     /**

tests/Fixtures/ClassWithPublicProperty.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Fixtures;
+
+class ClassWithPublicProperty
+{
+    public $closure;
+
+    public function __construct($closure)
+    {
+        $this->closure = $closure;
+    }
+}

tests/SerializerTest.php

@@ -6,6 +6,7 @@
 use Laravel\SerializableClosure\Serializers\Signed;
 use Laravel\SerializableClosure\Support\ReflectionClosure;
 use Laravel\SerializableClosure\UnsignedSerializableClosure;
+use Tests\Fixtures\ClassWithPublicProperty;
 use Tests\Fixtures\Model;
 
 test('closure with simple const', function () {
@@ -485,6 +486,34 @@ function () {
     new CarbonImmutable,
 ]);
 
+test('SerializableClosure in the class property', function () {
+    SerializableClosure::setSecretKey('foo');
+
+    $innerClosure = new ClassWithPublicProperty(new SerializableClosure(function () {
+    }));
+    $outerClosure = new SerializableClosure(function () use ($innerClosure) {
+        return $innerClosure->closure;
+    });
+
+    $unSerializedOuterClosure = unserialize(serialize($outerClosure));
+
+    expect($outerClosure())->toBeInstanceOf(SerializableClosure::class);
+    expect($unSerializedOuterClosure())->toBeInstanceOf(SerializableClosure::class);
+});
+
+test('UnsignedSerializableClosure in the class property', function () {
+    $innerClosure = new ClassWithPublicProperty(SerializableClosure::unsigned(function () {
+    }));
+    $outerClosure = SerializableClosure::unsigned(function () use ($innerClosure) {
+        return $innerClosure->closure;
+    });
+
+    $unSerializedOuterClosure = unserialize(serialize($outerClosure));
+
+    expect($outerClosure())->toBeInstanceOf(UnsignedSerializableClosure::class);
+    expect($unSerializedOuterClosure())->toBeInstanceOf(UnsignedSerializableClosure::class);
+});
+
 function serializer_php_74_switch_statement_test_is_two($a)
 {
     return $a === 2;