antonioribeiro / ia-collection

src/Support/Collection.php

Doc Comments +3 added lines, -3 removed lines

@@ -359,7 +359,7 @@ 
      * Get the first item from the collection passing the given truth test.
      *
      * @param  callable|null  $callback
-     * @param  mixed  $default
+     * @param  stdClass  $default
      * @return mixed
      */
     public function first(callable $callback = null, $default = null)
@@ -406,7 +406,7 @@ 
     /**
      * Get an item from the collection by key.
      *
-     * @param  mixed  $key
+     * @param  integer  $key
      * @param  mixed  $default
      * @return mixed
      */
@@ -784,7 +784,7 @@ 
     /**
      * Get and remove the last item from the collection.
      *
-     * @return mixed
+     * @return string
      */
     public function pop()
     {

Spacing +18 added lines, -18 removed lines

@@ -80,10 +80,10 @@ 
     {
         $callback = $this->valueRetriever($callback);
 
-        $items = $this->map(function ($value) use ($callback) {
+        $items = $this->map(function($value) use ($callback) {
             return $callback($value);
-        })->filter(function ($value) {
-            return ! is_null($value);
+        })->filter(function($value) {
+            return !is_null($value);
         });
 
         if ($count = $items->count()) {
@@ -100,8 +100,8 @@ 
     public function median($key = null)
     {
         $values = (isset($key) ? $this->pluck($key) : $this)
-            ->filter(function ($item) {
-                return ! is_null($item);
+            ->filter(function($item) {
+                return !is_null($item);
             })->sort()->values();
 
         $count = $values->count();
@@ -137,7 +137,7 @@ 
 
         $counts = new self;
 
-        $collection->each(function ($value) use ($counts) {
+        $collection->each(function($value) use ($counts) {
             $counts[$value] = isset($counts[$value]) ? $counts[$value] + 1 : 1;
         });
 
@@ -145,7 +145,7 @@ 
 
         $highestValue = $sorted->last();
 
-        return $sorted->filter(function ($value) use ($highestValue) {
+        return $sorted->filter(function($value) use ($highestValue) {
             return $value == $highestValue;
         })->sort()->keys()->all();
     }
@@ -313,12 +313,12 @@ 
     protected function duplicateComparator($strict)
     {
         if ($strict) {
-            return function ($a, $b) {
+            return function($a, $b) {
                 return $a === $b;
             };
         }
 
-        return function ($a, $b) {
+        return function($a, $b) {
             return $a == $b;
         };
     }
@@ -333,7 +333,7 @@ 
     {
         if ($keys instanceof Enumerable) {
             $keys = $keys->all();
-        } elseif (! is_array($keys)) {
+        } elseif (!is_array($keys)) {
             $keys = func_get_args();
         }
 
@@ -441,14 +441,14 @@ 
         foreach ($this->items as $key => $value) {
             $groupKeys = $groupBy($value, $key);
 
-            if (! is_array($groupKeys)) {
+            if (!is_array($groupKeys)) {
                 $groupKeys = [$groupKeys];
             }
 
             foreach ($groupKeys as $groupKey) {
                 $groupKey = is_bool($groupKey) ? (int) $groupKey : $groupKey;
 
-                if (! array_key_exists($groupKey, $results)) {
+                if (!array_key_exists($groupKey, $results)) {
                     $results[$groupKey] = new static;
                 }
 
@@ -458,7 +458,7 @@ 
 
         $result = new static($results);
 
-        if (! empty($nextGroups)) {
+        if (!empty($nextGroups)) {
             return $result->map->groupBy($nextGroups, $preserveKeys);
         }
 
@@ -501,7 +501,7 @@ 
         $keys = is_array($key) ? $key : func_get_args();
 
         foreach ($keys as $value) {
-            if (! $this->offsetExists($value)) {
+            if (!$this->offsetExists($value)) {
                 return false;
             }
         }
@@ -659,7 +659,7 @@ 
 
             $value = reset($pair);
 
-            if (! isset($dictionary[$key])) {
+            if (!isset($dictionary[$key])) {
                 $dictionary[$key] = [];
             }
 
@@ -931,7 +931,7 @@ 
      */
     public function search($value, $strict = false)
     {
-        if (! $this->useAsCallable($value)) {
+        if (!$this->useAsCallable($value)) {
             return array_search($value, $this->items, $strict);
         }
 
@@ -1202,11 +1202,11 @@ 
      */
     public function zip($items)
     {
-        $arrayableItems = array_map(function ($items) {
+        $arrayableItems = array_map(function($items) {
             return $this->getArrayableItems($items);
         }, func_get_args());
 
-        $params = array_merge([function () {
+        $params = array_merge([function() {
             return new static(func_get_args());
         }, $this->items], $arrayableItems);
 

src/Support/LazyCollection.php

Doc Comments +2 added lines, -2 removed lines

@@ -322,7 +322,7 @@ 
      * Get the first item from the enumerable passing the given truth test.
      *
      * @param  callable|null  $callback
-     * @param  mixed  $default
+     * @param  stdClass  $default
      * @return mixed
      */
     public function first(callable $callback = null, $default = null)
@@ -1187,7 +1187,7 @@ 
      * Explode the "value" and "key" arguments passed to "pluck".
      *
      * @param  string|array  $value
-     * @param  string|array|null  $key
+     * @param  string|null  $key
      * @return array
      */
     protected function explodePluckParameters($value, $key)

Spacing +34 added lines, -34 removed lines

@@ -60,7 +60,7 @@ 
             return new static;
         }
 
-        $instance = new static(function () use ($number) {
+        $instance = new static(function() use ($number) {
             for ($current = 1; $current <= $number; $current++) {
                 yield $current;
             }
@@ -78,7 +78,7 @@ 
      */
     public static function range($from, $to)
     {
-        return new static(function () use ($from, $to) {
+        return new static(function() use ($from, $to) {
             for (; $from <= $to; $from++) {
                 yield $from;
             }
@@ -139,7 +139,7 @@ 
      */
     public function collapse()
     {
-        return new static(function () {
+        return new static(function() {
             foreach ($this as $values) {
                 if (is_array($values) || $values instanceof Enumerable) {
                     foreach ($values as $value) {
@@ -304,12 +304,12 @@ 
     public function filter(callable $callback = null)
     {
         if (is_null($callback)) {
-            $callback = function ($value) {
+            $callback = function($value) {
                 return (bool) $value;
             };
         }
 
-        return new static(function () use ($callback) {
+        return new static(function() use ($callback) {
             foreach ($this as $key => $value) {
                 if ($callback($value, $key)) {
                     yield $key => $value;
@@ -330,7 +330,7 @@ 
         $iterator = $this->getIterator();
 
         if (is_null($callback)) {
-            if (! $iterator->valid()) {
+            if (!$iterator->valid()) {
                 return value($default);
             }
 
@@ -354,9 +354,9 @@ 
      */
     public function flatten($depth = INF)
     {
-        $instance = new static(function () use ($depth) {
+        $instance = new static(function() use ($depth) {
             foreach ($this as $item) {
-                if (! is_array($item) && ! $item instanceof Enumerable) {
+                if (!is_array($item) && !$item instanceof Enumerable) {
                     yield $item;
                 } elseif ($depth === 1) {
                     yield from $item;
@@ -376,7 +376,7 @@ 
      */
     public function flip()
     {
-        return new static(function () {
+        return new static(function() {
             foreach ($this as $key => $value) {
                 yield $value => $key;
             }
@@ -425,7 +425,7 @@ 
      */
     public function keyBy($keyBy)
     {
-        return new static(function () use ($keyBy) {
+        return new static(function() use ($keyBy) {
             $keyBy = $this->valueRetriever($keyBy);
 
             foreach ($this as $key => $item) {
@@ -501,7 +501,7 @@ 
      */
     public function isEmpty()
     {
-        return ! $this->getIterator()->valid();
+        return !$this->getIterator()->valid();
     }
 
     /**
@@ -523,7 +523,7 @@ 
      */
     public function keys()
     {
-        return new static(function () {
+        return new static(function() {
             foreach ($this as $key => $value) {
                 yield $key;
             }
@@ -559,7 +559,7 @@ 
      */
     public function pluck($value, $key = null)
     {
-        return new static(function () use ($value, $key) {
+        return new static(function() use ($value, $key) {
             [$value, $key] = $this->explodePluckParameters($value, $key);
 
             foreach ($this as $item) {
@@ -588,7 +588,7 @@ 
      */
     public function map(callable $callback)
     {
-        return new static(function () use ($callback) {
+        return new static(function() use ($callback) {
             foreach ($this as $key => $value) {
                 yield $key => $callback($value, $key);
             }
@@ -618,7 +618,7 @@ 
      */
     public function mapWithKeys(callable $callback)
     {
-        return new static(function () use ($callback) {
+        return new static(function() use ($callback) {
             foreach ($this as $key => $value) {
                 yield from $callback($value, $key);
             }
@@ -655,13 +655,13 @@ 
      */
     public function combine($values)
     {
-        return new static(function () use ($values) {
+        return new static(function() use ($values) {
             $values = $this->makeIterator($values);
 
             $errorMessage = 'Both parameters should have an equal number of elements';
 
             foreach ($this as $key) {
-                if (! $values->valid()) {
+                if (!$values->valid()) {
                     trigger_error($errorMessage, E_USER_WARNING);
 
                     break;
@@ -698,7 +698,7 @@ 
      */
     public function nth($step, $offset = 0)
     {
-        return new static(function () use ($step, $offset) {
+        return new static(function() use ($step, $offset) {
             $position = 0;
 
             foreach ($this as $item) {
@@ -721,11 +721,11 @@ 
     {
         if ($keys instanceof Enumerable) {
             $keys = $keys->all();
-        } elseif (! is_null($keys)) {
+        } elseif (!is_null($keys)) {
             $keys = is_array($keys) ? $keys : func_get_args();
         }
 
-        return new static(function () use ($keys) {
+        return new static(function() use ($keys) {
             if (is_null($keys)) {
                 yield from $this;
             } else {
@@ -754,7 +754,7 @@ 
      */
     public function concat($source)
     {
-        return (new static(function () use ($source) {
+        return (new static(function() use ($source) {
             yield from $this;
             yield from $source;
         }))->values();
@@ -801,7 +801,7 @@ 
      */
     public function replace($items)
     {
-        return new static(function () use ($items) {
+        return new static(function() use ($items) {
             $items = $this->getArrayableItems($items);
 
             foreach ($this as $key => $value) {
@@ -852,7 +852,7 @@ 
     {
         $predicate = $this->useAsCallable($value)
             ? $value
-            : function ($item) use ($value, $strict) {
+            : function($item) use ($value, $strict) {
                 return $strict ? $item === $value : $item == $value;
             };
 
@@ -884,7 +884,7 @@ 
      */
     public function skip($count)
     {
-        return new static(function () use ($count) {
+        return new static(function() use ($count) {
             $iterator = $this->getIterator();
 
             while ($iterator->valid() && $count--) {
@@ -940,7 +940,7 @@ 
             return static::empty();
         }
 
-        return new static(function () use ($size) {
+        return new static(function() use ($size) {
             $iterator = $this->getIterator();
 
             while ($iterator->valid()) {
@@ -952,7 +952,7 @@ 
                     if (count($chunk) < $size) {
                         $iterator->next();
 
-                        if (! $iterator->valid()) {
+                        if (!$iterator->valid()) {
                             break;
                         }
                     } else {
@@ -1038,11 +1038,11 @@ 
             return $this->passthru('take', func_get_args());
         }
 
-        return new static(function () use ($limit) {
+        return new static(function() use ($limit) {
             $iterator = $this->getIterator();
 
             while ($limit--) {
-                if (! $iterator->valid()) {
+                if (!$iterator->valid()) {
                     break;
                 }
 
@@ -1063,7 +1063,7 @@ 
      */
     public function tapEach(callable $callback)
     {
-        return new static(function () use ($callback) {
+        return new static(function() use ($callback) {
             foreach ($this as $key => $value) {
                 $callback($value, $key);
 
@@ -1079,7 +1079,7 @@ 
      */
     public function values()
     {
-        return new static(function () {
+        return new static(function() {
             foreach ($this as $item) {
                 yield $item;
             }
@@ -1099,8 +1099,8 @@ 
     {
         $iterables = func_get_args();
 
-        return new static(function () use ($iterables) {
-            $iterators = Collection::make($iterables)->map(function ($iterable) {
+        return new static(function() use ($iterables) {
+            $iterators = Collection::make($iterables)->map(function($iterable) {
                 return $this->makeIterator($iterable);
             })->prepend($this->getIterator());
 
@@ -1125,7 +1125,7 @@ 
             return $this->passthru('pad', func_get_args());
         }
 
-        return new static(function () use ($size, $value) {
+        return new static(function() use ($size, $value) {
             $yielded = 0;
 
             foreach ($this as $index => $item) {
@@ -1208,7 +1208,7 @@ 
      */
     protected function passthru($method, array $params)
     {
-        return new static(function () use ($method, $params) {
+        return new static(function() use ($method, $params) {
             yield from $this->collect()->$method(...$params);
         });
     }

src/Support/Traits/EnumeratesValues.php

Doc Comments +3 added lines, -3 removed lines

@@ -483,7 +483,7 @@ 
      * Filter items by the given key value pair.
      *
      * @param  string  $key
-     * @param  mixed  $operator
+     * @param  string  $operator
      * @param  mixed  $value
      * @return static
      */
@@ -628,7 +628,7 @@ 
     /**
      * Create a collection of all elements that do not pass a given truth test.
      *
-     * @param  callable|mixed  $callback
+     * @param  \Closure  $callback
      * @return static
      */
     public function reject($callback = true)
@@ -783,7 +783,7 @@ 
      * Dynamically access collection proxies.
      *
      * @param  string  $key
-     * @return mixed
+     * @return HigherOrderCollectionProxy
      *
      * @throws \Exception
      */

Spacing +34 added lines, -34 removed lines

@@ -117,13 +117,13 @@ 
     public function containsStrict($key, $value = null)
     {
         if (func_num_args() === 2) {
-            return $this->contains(function ($item) use ($key, $value) {
+            return $this->contains(function($item) use ($key, $value) {
                 return data_get($item, $key) === $value;
             });
         }
 
         if ($this->useAsCallable($key)) {
-            return ! is_null($this->first($key));
+            return !is_null($this->first($key));
         }
 
         foreach ($this as $item) {
@@ -157,7 +157,7 @@ 
     {
         (new static(func_get_args()))
             ->push($this)
-            ->each(function ($item) {
+            ->each(function($item) {
                 VarDumper::dump($item);
             });
 
@@ -189,7 +189,7 @@ 
      */
     public function eachSpread(callable $callback)
     {
-        return $this->each(function ($chunk, $key) use ($callback) {
+        return $this->each(function($chunk, $key) use ($callback) {
             $chunk[] = $key;
 
             return $callback(...$chunk);
@@ -210,7 +210,7 @@ 
             $callback = $this->valueRetriever($key);
 
             foreach ($this as $k => $v) {
-                if (! $callback($v, $k)) {
+                if (!$callback($v, $k)) {
                     return false;
                 }
             }
@@ -241,7 +241,7 @@ 
      */
     public function isNotEmpty()
     {
-        return ! $this->isEmpty();
+        return !$this->isEmpty();
     }
 
     /**
@@ -252,7 +252,7 @@ 
      */
     public function mapSpread(callable $callback)
     {
-        return $this->map(function ($chunk, $key) use ($callback) {
+        return $this->map(function($chunk, $key) use ($callback) {
             $chunk[] = $key;
 
             return $callback(...$chunk);
@@ -293,7 +293,7 @@ 
      */
     public function mapInto($class)
     {
-        return $this->map(function ($value, $key) use ($class) {
+        return $this->map(function($value, $key) use ($class) {
             return new $class($value, $key);
         });
     }
@@ -308,11 +308,11 @@ 
     {
         $callback = $this->valueRetriever($callback);
 
-        return $this->map(function ($value) use ($callback) {
+        return $this->map(function($value) use ($callback) {
             return $callback($value);
-        })->filter(function ($value) {
-            return ! is_null($value);
-        })->reduce(function ($result, $value) {
+        })->filter(function($value) {
+            return !is_null($value);
+        })->reduce(function($result, $value) {
             return is_null($result) || $value < $result ? $value : $result;
         });
     }
@@ -327,9 +327,9 @@ 
     {
         $callback = $this->valueRetriever($callback);
 
-        return $this->filter(function ($value) {
-            return ! is_null($value);
-        })->reduce(function ($result, $item) use ($callback) {
+        return $this->filter(function($value) {
+            return !is_null($value);
+        })->reduce(function($result, $item) use ($callback) {
             $value = $callback($item);
 
             return is_null($result) || $value > $result ? $value : $result;
@@ -387,14 +387,14 @@ 
     public function sum($callback = null)
     {
         if (is_null($callback)) {
-            $callback = function ($value) {
+            $callback = function($value) {
                 return $value;
             };
         } else {
             $callback = $this->valueRetriever($callback);
         }
 
-        return $this->reduce(function ($result, $item) use ($callback) {
+        return $this->reduce(function($result, $item) use ($callback) {
             return $result + $callback($item);
         }, 0);
     }
@@ -452,7 +452,7 @@ 
      */
     public function unless($value, callable $callback, callable $default = null)
     {
-        return $this->when(! $value, $callback, $default);
+        return $this->when(!$value, $callback, $default);
     }
 
     /**
@@ -516,7 +516,7 @@ 
     {
         $values = $this->getArrayableItems($values);
 
-        return $this->filter(function ($item) use ($key, $values, $strict) {
+        return $this->filter(function($item) use ($key, $values, $strict) {
             return in_array(data_get($item, $key), $values, $strict);
         });
     }
@@ -554,7 +554,7 @@ 
      */
     public function whereNotBetween($key, $values)
     {
-        return $this->filter(function ($item) use ($key, $values) {
+        return $this->filter(function($item) use ($key, $values) {
             return data_get($item, $key) < reset($values) || data_get($item, $key) > end($values);
         });
     }
@@ -571,7 +571,7 @@ 
     {
         $values = $this->getArrayableItems($values);
 
-        return $this->reject(function ($item) use ($key, $values, $strict) {
+        return $this->reject(function($item) use ($key, $values, $strict) {
             return in_array(data_get($item, $key), $values, $strict);
         });
     }
@@ -596,7 +596,7 @@ 
      */
     public function whereInstanceOf($type)
     {
-        return $this->filter(function ($value) use ($type) {
+        return $this->filter(function($value) use ($type) {
             return $value instanceof $type;
         });
     }
@@ -635,9 +635,9 @@ 
     {
         $useAsCallable = $this->useAsCallable($callback);
 
-        return $this->filter(function ($value, $key) use ($callback, $useAsCallable) {
+        return $this->filter(function($value, $key) use ($callback, $useAsCallable) {
             return $useAsCallable
-                ? ! $callback($value, $key)
+                ? !$callback($value, $key)
                 : $value != $callback;
         });
     }
@@ -655,7 +655,7 @@ 
 
         $exists = [];
 
-        return $this->reject(function ($item, $key) use ($callback, $strict, &$exists) {
+        return $this->reject(function($item, $key) use ($callback, $strict, &$exists) {
             if (in_array($id = $callback($item, $key), $exists, $strict)) {
                 return true;
             }
@@ -692,7 +692,7 @@ 
      */
     public function toArray()
     {
-        return $this->map(function ($value) {
+        return $this->map(function($value) {
             return $value instanceof Arrayable ? $value->toArray() : $value;
         })->all();
     }
@@ -704,7 +704,7 @@ 
      */
     public function jsonSerialize()
     {
-        return array_map(function ($value) {
+        return array_map(function($value) {
             if ($value instanceof JsonSerializable) {
                 return $value->jsonSerialize();
             } elseif ($value instanceof Jsonable) {
@@ -748,12 +748,12 @@ 
     public function countBy($callback = null)
     {
         if (is_null($callback)) {
-            $callback = function ($value) {
+            $callback = function($value) {
                 return $value;
             };
         }
 
-        return new static($this->groupBy($callback)->map(function ($value) {
+        return new static($this->groupBy($callback)->map(function($value) {
             return $value->count();
         }));
     }
@@ -789,7 +789,7 @@ 
      */
     public function __get($key)
     {
-        if (! in_array($key, static::$proxies)) {
+        if (!in_array($key, static::$proxies)) {
             throw new Exception("Property [{$key}] does not exist on this collection instance.");
         }
 
@@ -843,10 +843,10 @@ 
             $operator = '=';
         }
 
-        return function ($item) use ($key, $operator, $value) {
+        return function($item) use ($key, $operator, $value) {
             $retrieved = data_get($item, $key);
 
-            $strings = array_filter([$retrieved, $value], function ($value) {
+            $strings = array_filter([$retrieved, $value], function($value) {
                 return is_string($value) || (is_object($value) && method_exists($value, '__toString'));
             });
 
@@ -878,7 +878,7 @@ 
      */
     protected function useAsCallable($value)
     {
-        return ! is_string($value) && is_callable($value);
+        return !is_string($value) && is_callable($value);
     }
 
     /**
@@ -893,7 +893,7 @@ 
             return $value;
         }
 
-        return function ($item) use ($value) {
+        return function($item) use ($value) {
             return data_get($item, $value);
         };
     }