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

Doc Comments +3 added lines

@@ -183,6 +183,9 @@
     return [$data];
 }
 
+/**
+ * @param integer $count
+ */
 function get_range_start_end($range, $count)
 {
     if (Str::contains($range, Sequence::SLICE_DELIM)) {

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 +4 added lines, -4 removed lines

@@ -589,7 +589,7 @@ 
     public function pairs()
     {
         return collect(array_map(
-            function ($key, $val) {
+            function($key, $val) {
                 return [$key, $val];
             },
             array_keys($this->getData()),
@@ -608,11 +608,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;
         });
     }
@@ -899,7 +899,7 @@ 
     public function split($num)
     {
         $count = $this->count();
-        $size = (int)($count / $num);
+        $size = (int) ($count / $num);
         $mod = $count % $num;
         return collect($this->fold(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/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

src/Contracts/Structure/Collectable.php

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      *
      * @param mixed|callable  $value The value to search for
      *
-     * @return mixed
+     * @return boolean
      */
     public function contains($value);