Merge pull request #7 from barryosull/feature/method_deserialization

Can deserialize a payload into a callable method.

src/Contracts/MethodDeserializer.php

@@ -0,0 +1,6 @@
+<?php namespace EventSourced\ValueObject\Contracts;
+
+interface MethodDeserializer
+{
+    public function deserializeMethod($object, $method, $parameters);
+}

src/Deserializer/Deserializer.php

@@ -1,15 +1,17 @@
 <?php namespace EventSourced\ValueObject\Deserializer;
 
 use EventSourced\ValueObject\Contracts;
+use EventSourced\ValueObject\Deserializer\Deserializer\MethodArguments;
 use EventSourced\ValueObject\Extensions\ExtensionRepository;
 use EventSourced\ValueObject\ValueObject\Type;
 
-class Deserializer implements Contracts\Deserializer 
+class Deserializer implements Contracts\Deserializer, Contracts\MethodDeserializer
 {
     private $single_value;
     private $composite;
     private $set;
     private $type_entity;
+    private $method_arguments;
     private $extensionRepository;
 
     public function __construct(
@@ -20,6 +22,7 @@ public function __construct(
         $this->composite = new Deserializer\Composite($this, $reflector);
         $this->set = new Deserializer\Set($this);
         $this->type_entity = new Deserializer\TypeEntity($this, $reflector);
+        $this->method_arguments = new MethodArguments($this, $reflector);
         $this->extensionRepository = $repository;
     }
 
@@ -57,4 +60,11 @@ private function is_instance_of($class, $parent_class)
         return is_subclass_of($class, $parent_class)
             || ($class == $parent_class);
     }
+
+    public function deserializeMethod($object, $method, $serialized_parameters)
+    {
+        $arguments = $this->method_arguments->deserialize(get_class($object), $method, $serialized_parameters);
+
+        return new Method($object, $method, $arguments);
+    }
 }

src/Deserializer/Deserializer/Composite.php

@@ -2,9 +2,6 @@
 
 use EventSourced\ValueObject\Deserializer\Deserializer;
 use EventSourced\ValueObject\Deserializer\Reflector;
-use EventSourced\ValueObject\Deserializer\Exception;
-use EventSourced\ValueObject\Assert\Exception as AssertException;
-use ReflectionParameter;
 
 class Composite
 {
@@ -19,53 +16,12 @@ public function __construct(Deserializer $deserializer, Reflector $reflector)
 
     public function deserialize($class, $serialized)
     {
-        if (is_array($serialized)) {
-            $serialized = (object)$serialized;
-        }
+        $parameters_deserializer = new Parameters($this->deserializer);
 
-        $errors = [];
-        $deserialized_parameters = [];
+        $reflection_parameters = $this->reflector->get_constructor_parameters($class);
 
-        $parameters = $this->reflector->get_constructor_parameters($class);
-        foreach ($parameters as $parameter) {
-            $name = $parameter->getName();
+        $parameters = $parameters_deserializer->deserialize($reflection_parameters, $serialized);
 
-            try {
-                $deserialized_parameters[$name] = $this->make_parameter($parameter, $serialized);
-            } catch (AssertException $e) {
-                $errors[$name] = $e->error_messages();
-            } catch (Exception $e) {
-                $errors[$name] = $e->error_messages();
-            }
-        }
-
-        if (count($errors) != 0) {
-            throw new Exception($errors);
-        }
-
-        return $this->reflector->call_constructor($class, $deserialized_parameters);
-    }
-
-    private function make_parameter($parameter, $serialized)
-    {
-        $name = $parameter->getName();
-        $parameter_class = $parameter->getClass()->getName();
-
-        if ($this->is_nullable_parameter($parameter) && is_null($serialized)) {
-            return null;
-        }
-
-        if (!property_exists($serialized, $name)) {
-            throw new Exception(["Property '$name' is missing"]);
-        }
-
-        return $this->deserializer->deserialize(
-            $parameter_class, $serialized->$name
-        );
-    }
-
-    private function is_nullable_parameter(ReflectionParameter $parameter)
-    {
-        return ($parameter->isOptional() && $parameter->getDefaultValue() === null);
+        return $this->reflector->call_constructor($class, $parameters);
     }
 }

src/Deserializer/Deserializer/MethodArguments.php

@@ -0,0 +1,25 @@
+<?php namespace EventSourced\ValueObject\Deserializer\Deserializer;
+
+use EventSourced\ValueObject\Deserializer\Deserializer;
+use EventSourced\ValueObject\Deserializer\Reflector;
+
+class MethodArguments
+{
+    private $deserializer;
+    private $reflector;
+
+    public function __construct(Deserializer $deserializer, Reflector $reflector)
+    {
+        $this->deserializer = $deserializer;
+        $this->reflector = $reflector;
+    }
+
+    public function deserialize($class, $method, $serialized)
+    {
+        $reflection_parameters = new Parameters($this->deserializer);
+
+        $parameters = $this->reflector->get_method_parameters($class, $method);
+
+        return $reflection_parameters->deserialize($parameters, $serialized);
+    }
+}