Merge pull request #55 from keithbrink/patch/carbon-objects

Fix serialization of carbon objects

Author: nunomaduro

composer.json

@@ -21,6 +21,7 @@
         "php": "^7.3|^8.0"
     },
     "require-dev": {
+        "nesbot/carbon": "^2.61",
         "pestphp/pest": "^1.21.3",
         "phpstan/phpstan": "^1.8.2",
         "symfony/var-dumper": "^5.4.11"

src/Serializers/Native.php

@@ -3,6 +3,7 @@
 namespace Laravel\SerializableClosure\Serializers;
 
 use Closure;
+use DateTimeInterface;
 use Laravel\SerializableClosure\Contracts\Serializable;
 use Laravel\SerializableClosure\SerializableClosure;
 use Laravel\SerializableClosure\Support\ClosureScope;
@@ -460,6 +461,12 @@ protected function mapByReference(&$data)
 
             $instance = $data;
 
+            if ($data instanceof DateTimeInterface) {
+                $this->scope[$instance] = $data;
+
+                return;
+            }
+
             if ($data instanceof UnitEnum) {
                 $this->scope[$instance] = $data;
 

tests/SerializerTest.php

@@ -401,6 +401,32 @@ function () {
     expect($f(new Model))->toBeInstanceOf(Model::class);
 })->with('serializers');
 
+test('serializes carbon objects', function () {
+    $carbon = new \Carbon\Carbon('now');
+
+    $closure = function () use ($carbon) {
+        return $carbon;
+    };
+
+    $u = s($closure);
+    $r = $u();
+
+    expect($r)->toEqual($carbon);
+})->with('serializers');
+
+test('serializes carbon immutable objects', function () {
+    $carbon = new \Carbon\CarbonImmutable('now');
+
+    $closure = function () use ($carbon) {
+        return $carbon;
+    };
+
+    $u = s($closure);
+    $r = $u();
+
+    expect($r)->toEqual($carbon);
+})->with('serializers');
+
 class A
 {
     protected static function aStaticProtected()