Fixes mapping of iterables to objects using the fromIterable method.

composer.json

@@ -1,7 +1,6 @@
 {
     "name": "tiny-blocks/mapper",
     "type": "library",
-    "version": "1.0.4",
     "license": "MIT",
     "homepage": "https://github.com/tiny-blocks/mapper",
     "description": "Allows mapping data between different formats, such as JSON, arrays, and DTOs, providing flexibility in transforming and serializing information.",
@@ -44,7 +43,7 @@
     },
     "require": {
         "php": "^8.3",
-        "tiny-blocks/collection": "1.9.0"
+        "tiny-blocks/collection": "^1"
     },
     "require-dev": {
         "phpmd/phpmd": "^2.15",

phpmd.xml

@@ -36,8 +36,12 @@
     <rule ref="rulesets/design.xml/EvalExpression"/>
     <rule ref="rulesets/design.xml/NumberOfChildren"/>
     <rule ref="rulesets/design.xml/DepthOfInheritance"/>
-    <rule ref="rulesets/design.xml/CouplingBetweenObjects"/>
     <rule ref="rulesets/design.xml/DevelopmentCodeFragment"/>
+    <rule ref="rulesets/design.xml/CouplingBetweenObjects">
+        <properties>
+            <property name="maximum" value="15"/>
+        </properties>
+    </rule>
 
     <rule ref="rulesets/naming.xml/LongVariable"/>
     <rule ref="rulesets/naming.xml/ShortVariable">

phpstan.neon.dist

@@ -7,6 +7,7 @@ parameters:
     - '#method#'
     - '#expects#'
     - '#should return#'
+    - '#type mixed supplied#'
     - '#not specify its types#'
     - '#no value type specified#'
   reportUnmatchedIgnoredErrors: false

src/Internal/Mappers/Object/Casters/ArrayIteratorCaster.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters;
+
+use ArrayIterator;
+
+final readonly class ArrayIteratorCaster implements Caster
+{
+    public function castValue(mixed $value): ArrayIterator
+    {
+        return new ArrayIterator($value);
+    }
+}

src/Internal/Mappers/Object/Casters/TypeCaster.php → src/Internal/Mappers/Object/Casters/Caster.php

@@ -9,14 +9,14 @@
 /**
  * Responsible for applying a cast to values, based on a specific type.
  */
-interface TypeCaster
+interface Caster
 {
     /**
-     * Applies a cast to the provided value.
+     * Casts the given value to a specific type.
      *
      * @param mixed $value The value to be cast.
      * @return mixed The cast value.
-     * @throws InvalidCast Thrown when the value cannot be cast to the expected type.
+     * @throws InvalidCast If the value cannot be cast to the expected type.
      */
-    public function applyCast(mixed $value): mixed;
+    public function castValue(mixed $value): mixed;
 }

src/Internal/Mappers/Object/Casters/CasterHandler.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters;
+
+use ArrayIterator;
+use DateTimeInterface;
+use Generator;
+use ReflectionParameter;
+use TinyBlocks\Collection\Collectible;
+use UnitEnum;
+
+final readonly class CasterHandler
+{
+    public function __construct(private ReflectionParameter $parameter)
+    {
+    }
+
+    public function castValue(mixed $value): mixed
+    {
+        $class = $this->parameter->getType()->getName();
+
+        $caster = match (true) {
+            $class === Generator::class,                     => new GeneratorCaster(),
+            $class === ArrayIterator::class,                 => new ArrayIteratorCaster(),
+            is_subclass_of($class, UnitEnum::class)          => new EnumCaster(class: $class),
+            is_subclass_of($class, Collectible::class)       => new CollectionCaster(class: $class),
+            is_subclass_of($class, DateTimeInterface::class) => new DateTimeCaster(),
+            default                                          => new DefaultCaster(class: $class)
+        };
+
+        return $caster->castValue(value: $value);
+    }
+}

src/Internal/Mappers/Object/Casters/CollectionCaster.php

@@ -0,0 +1,38 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters;
+
+use TinyBlocks\Collection\Collectible;
+use TinyBlocks\Mapper\Internal\Mappers\Object\ObjectMapper;
+use TinyBlocks\Mapper\Internal\Mappers\Object\Reflector;
+use TinyBlocks\Mapper\IterableMapper;
+
+final readonly class CollectionCaster implements Caster
+{
+    public function __construct(private string $class)
+    {
+    }
+
+    public function castValue(mixed $value): Collectible
+    {
+        $reflectionClass = Reflector::reflectFrom(class: $this->class);
+        /** @var IterableMapper & Collectible $instance */
+        $instance = $reflectionClass->newInstanceWithoutConstructor();
+
+        $type = $instance->getType();
+
+        if ($type === $this->class) {
+            return $instance->createFrom(elements: $value);
+        }
+
+        $mapped = [];
+
+        foreach ($value as $item) {
+            $mapped[] = (new ObjectMapper())->map(iterable: $item, class: $type);
+        }
+
+        return $instance->createFrom(elements: $mapped);
+    }
+}

src/Internal/Mappers/Object/Casters/DateTimeCaster.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters;
+
+use DateTimeImmutable;
+
+final readonly class DateTimeCaster implements Caster
+{
+    public function castValue(mixed $value): DateTimeImmutable
+    {
+        return new DateTimeImmutable($value);
+    }
+}

src/Internal/Mappers/Object/Casters/DefaultCaster.php

@@ -0,0 +1,23 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters;
+
+use TinyBlocks\Mapper\Internal\Mappers\Object\ObjectMapper;
+
+final readonly class DefaultCaster implements Caster
+{
+    public function __construct(private string $class)
+    {
+    }
+
+    public function castValue(mixed $value): mixed
+    {
+        if (!class_exists($this->class)) {
+            return $value;
+        }
+
+        return (new ObjectMapper())->map(iterable: $value, class: $this->class);
+    }
+}

src/Internal/Mappers/Object/Casters/Types/EnumCaster.php → src/Internal/Mappers/Object/Casters/EnumCaster.php

@@ -2,20 +2,19 @@
 
 declare(strict_types=1);
 
-namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters\Types;
+namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters;
 
 use ReflectionEnum;
 use TinyBlocks\Mapper\Internal\Exceptions\InvalidCast;
-use TinyBlocks\Mapper\Internal\Mappers\Object\Casters\TypeCaster;
 use UnitEnum;
 
-final readonly class EnumCaster implements TypeCaster
+final readonly class EnumCaster implements Caster
 {
     public function __construct(public string $class)
     {
     }
 
-    public function applyCast(mixed $value): UnitEnum
+    public function castValue(mixed $value): UnitEnum
     {
         $reflectionEnum = new ReflectionEnum($this->class);
 

src/Internal/Mappers/Object/Casters/GeneratorCaster.php

@@ -0,0 +1,23 @@
+<?php
+
+declare(strict_types=1);
+
+namespace TinyBlocks\Mapper\Internal\Mappers\Object\Casters;
+
+use Generator;
+
+final readonly class GeneratorCaster implements Caster
+{
+    public function castValue(mixed $value): Generator
+    {
+        if (is_iterable($value)) {
+            foreach ($value as $item) {
+                yield $item;
+            }
+
+            return;
+        }
+
+        yield $value;
+    }
+}