Make ArrayValue implementation compatible with the PHP8.1 and above (#41

)
othercodes · Jan 10, 2023 · 6d2239c · 6d2239c
1 parent 46f9202
commit 6d2239c
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/src/Domain/ValueObjects/ArrayValue.php b/src/Domain/ValueObjects/ArrayValue.php
@@ -187,6 +187,11 @@ public function serialize(): ?string
         return serialize($this->value);
     }
 
+    public function __serialize(): array
+    {
+        return $this->value;
+    }
+
     /**
      * Constructs the object.
      *
@@ -197,6 +202,11 @@ public function unserialize($serialized): void
         $this->value = unserialize($serialized);
     }
 
+    public function __unserialize(array $data): void
+    {
+        $this->value = $data;
+    }
+
     /**
      * Count elements of an object.
      *
@@ -216,4 +226,4 @@ public function __toString(): string
     {
         return json_encode($this->value);
     }
-}
+}
diff --git a/tests/Domain/ValueObjects/ArrayValueTest.php b/tests/Domain/ValueObjects/ArrayValueTest.php
@@ -63,4 +63,14 @@ public function testShouldIterateOverEachItem(): void
             $this->assertIsString($item);
         }
     }
-}
+
+    public function testShouldBeSerializable(): void
+    {
+        $vo = new ShoppingList(['Milk', 'Eggs', 'Yogurt']);
+        $unserialized = unserialize(serialize($vo));
+
+        foreach ($vo as $item) {
+            $this->assertContains($item, $unserialized);
+        }
+    }
+}