antonioribeiro / ia-collection

src/Support/Collection.php

Doc Comments +7 added lines, -7 removed lines

@@ -580,7 +580,7 @@ 
      * Filter items by the given key value pair.
      *
      * @param  string  $key
-     * @param  mixed  $operator
+     * @param  string  $operator
      * @param  mixed  $value
      * @return static
      */
@@ -751,7 +751,7 @@ 
      * Get the first item from the collection.
      *
      * @param  callable|null  $callback
-     * @param  mixed  $default
+     * @param  stdClass  $default
      * @return mixed
      */
     public function first(callable $callback = null, $default = null)
@@ -811,7 +811,7 @@ 
     /**
      * Get an item from the collection by key.
      *
-     * @param  mixed  $key
+     * @param  integer  $key
      * @param  mixed  $default
      * @return mixed
      */
@@ -1339,7 +1339,7 @@ 
     /**
      * Get and remove the last item from the collection.
      *
-     * @return mixed
+     * @return string
      */
     public function pop()
     {
@@ -1437,7 +1437,7 @@ 
      * Reduce the collection to a single value.
      *
      * @param  callable  $callback
-     * @param  mixed  $initial
+     * @param  integer  $initial
      * @return mixed
      */
     public function reduce(callable $callback, $initial = null)
@@ -1448,7 +1448,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)
@@ -2061,7 +2061,7 @@ 
      * Dynamically access collection proxies.
      *
      * @param  string  $key
-     * @return mixed
+     * @return HigherOrderCollectionProxy
      *
      * @throws \Exception
      */

Spacing +50 added lines, -50 removed lines

@@ -147,10 +147,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()) {
@@ -178,8 +178,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();
@@ -215,7 +215,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;
         });
 
@@ -223,7 +223,7 @@ 
 
         $highestValue = $sorted->last();
 
-        return $sorted->filter(function ($value) use ($highestValue) {
+        return $sorted->filter(function($value) use ($highestValue) {
             return $value == $highestValue;
         })->sort()->keys()->all();
     }
@@ -284,13 +284,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));
         }
 
         return in_array($key, $this->items, true);
@@ -331,7 +331,7 @@ 
     {
         (new static(func_get_args()))
             ->push($this)
-            ->each(function ($item) {
+            ->each(function($item) {
                 VarDumper::dump($item);
             });
 
@@ -432,7 +432,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);
@@ -453,7 +453,7 @@ 
             $callback = $this->valueRetriever($key);
 
             foreach ($this->items as $k => $v) {
-                if (! $callback($v, $k)) {
+                if (!$callback($v, $k)) {
                     return false;
                 }
             }
@@ -474,7 +474,7 @@ 
     {
         if ($keys instanceof self) {
             $keys = $keys->all();
-        } elseif (! is_array($keys)) {
+        } elseif (!is_array($keys)) {
             $keys = func_get_args();
         }
 
@@ -549,7 +549,7 @@ 
      */
     public function unless($value, callable $callback, callable $default = null)
     {
-        return $this->when(! $value, $callback, $default);
+        return $this->when(!$value, $callback, $default);
     }
 
     /**
@@ -611,10 +611,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'));
             });
 
@@ -662,7 +662,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);
         });
     }
@@ -700,7 +700,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);
         });
     }
@@ -717,7 +717,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);
         });
     }
@@ -742,7 +742,7 @@ 
      */
     public function whereInstanceOf($type)
     {
-        return $this->filter(function ($value) use ($type) {
+        return $this->filter(function($value) use ($type) {
             return $value instanceof $type;
         });
     }
@@ -846,14 +846,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;
                 }
 
@@ -863,7 +863,7 @@ 
 
         $result = new static($results);
 
-        if (! empty($nextGroups)) {
+        if (!empty($nextGroups)) {
             return $result->map->groupBy($nextGroups, $preserveKeys);
         }
 
@@ -906,7 +906,7 @@ 
         $keys = is_array($key) ? $key : func_get_args();
 
         foreach ($keys as $value) {
-            if (! $this->offsetExists($value)) {
+            if (!$this->offsetExists($value)) {
                 return false;
             }
         }
@@ -973,7 +973,7 @@ 
      */
     public function isNotEmpty()
     {
-        return ! $this->isEmpty();
+        return !$this->isEmpty();
     }
 
     /**
@@ -984,7 +984,7 @@ 
      */
     protected function useAsCallable($value)
     {
-        return ! is_string($value) && is_callable($value);
+        return !is_string($value) && is_callable($value);
     }
 
     /**
@@ -1074,7 +1074,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);
@@ -1100,7 +1100,7 @@ 
 
             $value = reset($pair);
 
-            if (! isset($dictionary[$key])) {
+            if (!isset($dictionary[$key])) {
                 $dictionary[$key] = [];
             }
 
@@ -1167,7 +1167,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);
         });
     }
@@ -1182,9 +1182,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;
@@ -1234,11 +1234,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;
         });
     }
@@ -1319,7 +1319,7 @@ 
                 : $this->operatorForWhere(...func_get_args());
 
         foreach ($this->items as $key => $item) {
-            $partitions[(int) ! $callback($item, $key)][$key] = $item;
+            $partitions[(int) !$callback($item, $key)][$key] = $item;
         }
 
         return new static($partitions);
@@ -1455,9 +1455,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;
         });
     }
@@ -1481,7 +1481,7 @@ 
      */
     public function search($value, $strict = false)
     {
-        if (! $this->useAsCallable($value)) {
+        if (!$this->useAsCallable($value)) {
             return array_search($value, $this->items, $strict);
         }
 
@@ -1706,7 +1706,7 @@ 
 
         $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);
     }
@@ -1765,7 +1765,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;
             }
@@ -1807,7 +1807,7 @@ 
             return $value;
         }
 
-        return function ($item) use ($value) {
+        return function($item) use ($value) {
             return data_get($item, $value);
         };
     }
@@ -1823,11 +1823,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);
 
@@ -1853,7 +1853,7 @@ 
      */
     public function toArray()
     {
-        return array_map(function ($value) {
+        return array_map(function($value) {
             return $value instanceof Arrayable ? $value->toArray() : $value;
         }, $this->items);
     }
@@ -1865,7 +1865,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) {
@@ -1929,12 +1929,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();
         }));
     }
@@ -2067,7 +2067,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.");
         }
 

src/Support/Traits/Macroable.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,10 +2,10 @@
 
 namespace IlluminateAgnostic\Collection\Support\Traits;
 
+use BadMethodCallException;
 use Closure;
 use ReflectionClass;
 use ReflectionMethod;
-use BadMethodCallException;
 
 trait Macroable
 {

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
         );
 
         foreach ($methods as $method) {
-            if ($replace || ! static::hasMacro($method->name)) {
+            if ($replace || !static::hasMacro($method->name)) {
                 $method->setAccessible(true);
                 static::macro($method->name, $method->invoke($mixin));
             }
@@ -74,7 +74,7 @@ 
      */
     public static function __callStatic($method, $parameters)
     {
-        if (! static::hasMacro($method)) {
+        if (!static::hasMacro($method)) {
             throw new BadMethodCallException(sprintf(
                 'Method %s::%s does not exist.', static::class, $method
             ));
@@ -98,7 +98,7 @@ 
      */
     public function __call($method, $parameters)
     {
-        if (! static::hasMacro($method)) {
+        if (!static::hasMacro($method)) {
             throw new BadMethodCallException(sprintf(
                 'Method %s::%s does not exist.', static::class, $method
             ));

src/Support/Str.php

Spacing +5 added lines, -5 removed lines

@@ -56,7 +56,7 @@ 
     {
         $languageSpecific = static::languageSpecificCharsArray($language);
 
-        if (! is_null($languageSpecific)) {
+        if (!is_null($languageSpecific)) {
             $value = str_replace($languageSpecific[0], $languageSpecific[1], $value);
         }
 
@@ -249,7 +249,7 @@ 
     {
         preg_match('/^\s*+(?:\S++\s*+){1,'.$words.'}/u', $value, $matches);
 
-        if (! isset($matches[0]) || static::length($value) === static::length($matches[0])) {
+        if (!isset($matches[0]) || static::length($value) === static::length($matches[0])) {
             return $value;
         }
 
@@ -467,7 +467,7 @@ 
             return static::$snakeCache[$key][$delimiter];
         }
 
-        if (! ctype_lower($value)) {
+        if (!ctype_lower($value)) {
             $value = preg_replace('/\s+/u', '', ucwords($value));
 
             $value = static::lower(preg_replace('/(.)(?=[A-Z])/u', '$1'.$delimiter, $value));
@@ -716,7 +716,7 @@ 
     {
         static $languageSpecific;
 
-        if (! isset($languageSpecific)) {
+        if (!isset($languageSpecific)) {
             $languageSpecific = [
                 'bg' => [
                     ['х', 'Х', 'щ', 'Щ', 'ъ', 'Ъ', 'ь', 'Ь'],
@@ -727,7 +727,7 @@ 
                     ['ae', 'oe', 'aa', 'Ae', 'Oe', 'Aa'],
                 ],
                 'de' => [
-                    ['ä',  'ö',  'ü',  'Ä',  'Ö',  'Ü'],
+                    ['ä', 'ö', 'ü', 'Ä', 'Ö', 'Ü'],
                     ['ae', 'oe', 'ue', 'AE', 'OE', 'UE'],
                 ],
                 'ro' => [

src/Support/Arr.php

Spacing +6 added lines, -6 removed lines

@@ -51,7 +51,7 @@ 
         foreach ($array as $values) {
             if ($values instanceof Collection) {
                 $values = $values->all();
-            } elseif (! is_array($values)) {
+            } elseif (!is_array($values)) {
                 continue;
             }
 
@@ -111,7 +111,7 @@ 
         $results = [];
 
         foreach ($array as $key => $value) {
-            if (is_array($value) && ! empty($value)) {
+            if (is_array($value) && !empty($value)) {
                 $results = array_merge($results, static::dot($value, $prepend.$key.'.'));
             } else {
                 $results[$prepend.$key] = $value;
@@ -211,7 +211,7 @@ 
         foreach ($array as $item) {
             $item = $item instanceof Collection ? $item->all() : $item;
 
-            if (! is_array($item)) {
+            if (!is_array($item)) {
                 $result[] = $item;
             } elseif ($depth === 1) {
                 $result = array_merge($result, array_values($item));
@@ -277,7 +277,7 @@ 
      */
     public static function get($array, $key, $default = null)
     {
-        if (! static::accessible($array)) {
+        if (!static::accessible($array)) {
             return value($default);
         }
 
@@ -315,7 +315,7 @@ 
     {
         $keys = (array) $keys;
 
-        if (! $array || $keys === []) {
+        if (!$array || $keys === []) {
             return false;
         }
 
@@ -517,7 +517,7 @@ 
             // If the key doesn't exist at this depth, we will just create an empty array
             // to hold the next value, allowing us to create the arrays to hold final
             // values at the correct depth. Then we'll keep digging into the array.
-            if (! isset($array[$key]) || ! is_array($array[$key])) {
+            if (!isset($array[$key]) || !is_array($array[$key])) {
                 $array[$key] = [];
             }