Fixed bug.

Signed-off-by: Joshua Parker <[email protected]>
QubusPHP · Feb 16, 2025 · 507f27c · 507f27c
1 parent 8a27100
commit 507f27c
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 11 deletions.
diff --git a/Collection/BaseCollection.php b/Collection/BaseCollection.php
@@ -15,6 +15,7 @@
 
 use Qubus\Exception\Data\TypeException;
 use Qubus\Support\ArrayHelper;
+use Qubus\Support\DataType;
 use ReflectionException;
 
 use function count;
@@ -152,11 +153,10 @@ public function filter(callable $callable): BaseCollection
      * @param mixed $key
      * @return mixed
      * @throws TypeException
-     * @throws ReflectionException
      */
     public function get(mixed $key): mixed
     {
-        return ArrayHelper::getInstance()->get($this->items, $key);
+        return (new DataType())->array->get($this->items, $key);
     }
 
     /**
@@ -251,11 +251,10 @@ public function values(): static
      * flattened array of items.
      *
      * @return self
-     * @throws ReflectionException
      */
     public function flatten(): static
     {
-        return new static(ArrayHelper::getInstance()->flatten($this->items));
+        return new static((new DataType())->array->flatten($this->items));
     }
 
     /**
@@ -444,11 +443,11 @@ public function sum(mixed $callback = null): int|float
     /**
      * Merge items with current collection.
      *
-     * @throws TypeException|ReflectionException
+     * @throws TypeException
      */
     public function merge(array $items): static
     {
-        return new static(ArrayHelper::getInstance()->merge($this->items, $items));
+        return new static((new DataType())->array->merge($this->items, $items));
     }
 
     /**

diff --git a/Collection/ValueExtractorAware.php b/Collection/ValueExtractorAware.php
@@ -24,19 +24,32 @@ trait ValueExtractorAware
      * Extracts the value of the given property or method from the object.
      *
      * @param mixed $object The object to extract the value from.
-     * @param string $propertyOrMethod The property or method for which the
-     *                                 value should be extracted.
+     * @param string|null $propertyOrMethod The property or method for which the
+     *                                      value should be extracted.
      * @return mixed the value extracted from the specified property or method.
      * @throws ValueExtractionException if the method or property is not defined.
      */
-    protected function extractValue(mixed $object, string $propertyOrMethod): mixed
+    protected function extractValue(mixed $object, ?string $propertyOrMethod = null): mixed
     {
-        if (!is_object(value: $object)) {
+        if ($propertyOrMethod === null) {
+            return $object;
+        }
+
+        if (!is_object(value: $object) && !is_array($object)) {
             throw new ValueExtractionException(message: 'Unable to extract a value from a non-object.');
         }
 
+        if (is_array($object)) {
+            return $object[$propertyOrMethod] ?? throw new InvalidPropertyOrMethodException(
+                sprintf(
+                    'Key or index "%s" not found in collection elements',
+                    $propertyOrMethod,
+                )
+            );
+        }
+
         if (property_exists($object, $propertyOrMethod)) {
-            return $object->$propertyOrMethod;
+            return $object->{$propertyOrMethod};
         }
 
         if (method_exists($object, $propertyOrMethod)) {

diff --git a/Collection/ValueToStringAware.php b/Collection/ValueToStringAware.php
@@ -68,6 +68,7 @@ protected function toolValueToString(mixed $value): string
         }
 
         // From here, $value should be an object.
+        assert(is_object($value));
 
         // __toString() is implemented
         if (is_callable([$value, '__toString'])) {