nozavroni / collections

src/functions.php

Spacing +3 added lines, -3 removed lines

@@ -171,7 +171,7 @@ 
 {
     $type = gettype($data);
     if ($meta) {
-        switch($type) {
+        switch ($type) {
             case 'object':
                 $class = get_class($data);
                 return "{$type} <{$class}>";
@@ -180,7 +180,7 @@ 
                 return "{$type} <{$restype}>";
         }
     } else {
-        switch($type) {
+        switch ($type) {
             case 'object':
                 return get_class($data);
             case 'resource':
@@ -270,7 +270,7 @@ 
     } else {
         echo $label . "\n" . implode(
                 array_map(
-                    function () {
+                    function() {
                         return '-';
                     },
                     str_split($label)

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
  */
 namespace Noz;
 
-use Closure;
 use InvalidArgumentException;
 use Iterator;
 use Noz\Collection\Collection;

Indentation -4 removed lines

@@ -52,10 +52,8 @@ 
 
 /**
  * Underscore function.
-
  * This function is meant to work sort of like jQuery's "$()". It is a contextual catch-all type function. It works
  * as a short-hand alias for invoke, collect, and with.
-
  * @param callable|mixed    $in
  * @param mixed ...         $_
  *
@@ -133,10 +131,8 @@ 
  *
  * Accepts any kind of data and converts it to an array. If strict mode is on, only data that returns true from
  * is_arrayable() will be converted to an array. Anything else will cause an InvalidArgumentException to be thrown.
-
  * @param mixed $data   Data to convert to array
  * @param bool  $strict Whether to use strict mode
-
  * @return array
  *
  * @throws InvalidArgumentException

src/Contracts/CollectionInterface.php

Indentation -1 removed lines

@@ -491,7 +491,6 @@
      *
      * Reduces this collection to one value by passing value, key, and the return value from the previous iteration
      * until only one value remains. Iteration begins from the last item in the collection and moves up.
-
      * @param callable $callback The callback function
      * @param          $initial  The initial "carry" value
      *

Doc Comments +5 added lines, -5 removed lines

@@ -62,7 +62,7 @@ 
      * @param string|int $index Set value at this index
      * @param mixed      $value Set this value at index
      *
-     * @return mixed
+     * @return CollectionInterface
      */
     public function set($index, $value);
 
@@ -73,7 +73,7 @@ 
      *
      * @param string|int $index Delete item at this index
      *
-     * @return mixed
+     * @return CollectionInterface
      */
     public function delete($index);
 
@@ -108,7 +108,7 @@ 
      * @param string|int $index Add value at this index
      * @param mixed      $value Add this value at index
      *
-     * @return mixed
+     * @return CollectionInterface
      */
     public function add($index, $value);
 
@@ -164,7 +164,7 @@ 
      * @param int  $nth    The number of items between cycles
      * @param null $offset An optional offset to begin from
      *
-     * @return mixed
+     * @return CollectionInterface
      */
     public function nth($nth, $offset = null);
 
@@ -228,7 +228,7 @@ 
      *
      * @param int $num Number of items to return
      *
-     * @return mixed|CollectionInterface
+     * @return CollectionInterface
      */
     public function random($num);
 

src/Collection/Collection.php

Unused Use Statements +2 added lines, -3 removed lines

@@ -14,13 +14,12 @@
 use InvalidArgumentException;
 use Iterator;
 use Noz\Contracts\Arrayable;
-use Noz\Contracts\Invokable;
 use Noz\Contracts\CollectionInterface;
+use Noz\Contracts\Invokable;
+use Noz\Traits\IsArrayable;
 use OutOfBoundsException;
 use Traversable;
 
-use Noz\Traits\IsArrayable;
-
 use function
     Noz\is_traversable,
     Noz\typeof,

Indentation +1 added lines, -3 removed lines

@@ -997,7 +997,7 @@ 
             }
         });
         $args = $args->prepend($this->getData())
-                     ->prepend(null);
+                        ->prepend(null);
 
         return collect(
 //            array_map(
@@ -1185,11 +1185,9 @@ 
 
     /**
      * Get the number of times each item occurs in the collection.
-
      * This method will return a NumericCollection where keys are the
      * values and values are the number of times that value occurs in
      * the original collection.
-
      * @return CollectionInterface
      */
     public function counts()

Spacing +5 added lines, -5 removed lines

@@ -347,7 +347,7 @@ 
      */
     public function contains($value, $index = null)
     {
-        return (bool) $this->first(function ($val, $key) use ($value, $index) {
+        return (bool) $this->first(function($val, $key) use ($value, $index) {
             if (is_callable($value)) {
                 $found = $value($val, $key);
             } else {
@@ -626,7 +626,7 @@ 
     public function pairs()
     {
         return collect(array_map(
-            function ($key, $val) {
+            function($key, $val) {
                 return [$key, $val];
             },
             array_keys($this->getData()),
@@ -645,11 +645,11 @@ 
     public function duplicates()
     {
         $dups = [];
-        $this->walk(function ($val, $key) use (&$dups) {
+        $this->walk(function($val, $key) use (&$dups) {
             $dups[$val][] = $key;
         });
 
-        return collect($dups)->filter(function ($val) {
+        return collect($dups)->filter(function($val) {
             return count($val) > 1;
         });
     }
@@ -936,7 +936,7 @@ 
     public function split($num)
     {
         $count = $this->count();
-        $size = (int)($count / $num);
+        $size = (int) ($count / $num);
         $mod = $count % $num;
         return collect($this->foldRight(function($chunks, $val, $key, $iter) use ($num, $size, $mod) {
             $chunk_count = count($chunks);

src/Collection/Dict.php

Unused Use Statements +2 added lines, -9 removed lines

@@ -9,18 +9,11 @@
  */
 namespace Noz\Collection;
 
-use BadMethodCallException;
-
 use Countable;
-use Noz\Contracts\Structure\Dictable;
-use Traversable;
-use SplFixedArray;
-
-use Noz\Contracts\Structure\Sequenceable;
-use Noz\Contracts\Immutable;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Dictable;
 use Noz\Traits\IsImmutable;
 
 use function Noz\to_array;

src/Collection/Lst.php

Unused Use Statements +3 added lines, -9 removed lines

@@ -9,19 +9,13 @@
  */
 namespace Noz\Collection;
 
-use BadMethodCallException;
-
 use Countable;
-use Noz\Contracts\Structure\Listable;
-use Traversable;
-use SplDoublyLinkedList;
-
-use Noz\Contracts\Structure\Sequenceable;
-use Noz\Contracts\Immutable;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Listable;
 use Noz\Traits\IsImmutable;
+use SplDoublyLinkedList;
 
 use function Noz\to_array;
 use function Noz\is_traversable;

src/Collection/Queue.php

Unused Use Statements +2 added lines, -9 removed lines

@@ -9,18 +9,11 @@
  */
 namespace Noz\Collection;
 
-use BadMethodCallException;
-
 use Countable;
-use Noz\Contracts\Structure\Listable;
-use Traversable;
-use SplFixedArray;
-
-use Noz\Contracts\Structure\Sequenceable;
-use Noz\Contracts\Immutable;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Listable;
 use Noz\Traits\IsImmutable;
 
 use function Noz\to_array;

src/Collection/Sequence.php

Doc Comments +3 added lines, -3 removed lines

@@ -134,7 +134,7 @@ 
      * You may optionally pass in a callback which will determine if each of the items within the collection are empty.
      * If all items in the collection are empty according to this callback, this method will return true.
      * @param callable $callback The callback
-     * @return bool
+     * @return boolean|null
      */
     public function isEmpty(callable $callback = null)
     {
@@ -157,7 +157,7 @@ 
      * If callback is provided, this method will return true if all items in collection cause callback to return true.
      * Otherwise, it will return true if all items in the collection have a truthy value.
      * @param callable|null $callback The callback
-     * @return bool
+     * @return boolean|null
      */
     public function every(callable $callback = null)
     {
@@ -168,7 +168,7 @@ 
      * Does every item return false?
      * This method is the exact opposite of "all".
      * @param callable|null $callback The callback
-     * @return bool
+     * @return boolean|null
      */
     public function none(callable $callback = null)
     {

Unused Use Statements +4 added lines, -7 removed lines

@@ -10,17 +10,14 @@
 namespace Noz\Collection;
 
 use BadMethodCallException;
-
 use Countable;
-use Traversable;
-use SplFixedArray;
-
-use Noz\Contracts\Structure\Sequenceable;
-use Noz\Contracts\Immutable;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Sequenceable;
 use Noz\Traits\IsImmutable;
+use SplFixedArray;
+use Traversable;
 
 use function Noz\to_array;
 use function Noz\is_traversable;

Braces +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
      * @param mixed $item Item to prepend to collection
      * @return $this
      */public function prepend($item)
-{
+    {
     // TODO: Implement prepend() method.
 }
 

src/Collection/Set.php

Unused Use Statements +2 added lines, -8 removed lines

@@ -9,17 +9,11 @@
  */
 namespace Noz\Collection;
 
-use BadMethodCallException;
-
 use Countable;
-use Traversable;
-use SplFixedArray;
-
-use Noz\Contracts\Structure\Settable;
-use Noz\Contracts\Immutable;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Settable;
 use Noz\Traits\IsImmutable;
 
 use function Noz\to_array;

src/Contracts/Comparable.php

Indentation -2 removed lines

@@ -110,10 +110,8 @@
  *
  * Accepts any kind of data and converts it to an array. If strict mode is on, only data that returns true from
  * is_arrayable() will be converted to an array. Anything else will cause an InvalidArgumentException to be thrown.
-
  * @param mixed $data   Data to convert to array
  * @param bool  $strict Whether to use strict mode
-
  * @return array
  *
  * @throws InvalidArgumentException