nozavroni / collections

src/Traits/IsArrayable.php

Doc Comments +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     /**
      * Get object as array.
      *
-     * @return array This object as an array
+     * @return callable This object as an array
      */
     public function toArray()
     {

src/functions.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

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)

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

@@ -80,7 +80,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);
 
@@ -92,7 +92,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);
 
@@ -101,7 +101,7 @@ 
      *
      * @param string|int $index Delete item at this index
      *
-     * @return mixed
+     * @return CollectionInterface
      */
     public function delete($index);
 
@@ -136,7 +136,7 @@ 
      * @param mixed|callable  $value The value to search for
      * @param int|string|null $index The index to search for
      *
-     * @return mixed
+     * @return boolean
      */
     public function contains($value, $index = null);
 
@@ -214,7 +214,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 every($nth, $offset = null);
 
@@ -300,7 +300,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, -4 removed lines

@@ -9,18 +9,16 @@
  */
 namespace Noz\Collection;
 
-use ArrayAccess;
 use ArrayIterator;
 use Countable;
 use InvalidArgumentException;
 use Iterator;
 use Noz\Contracts\ArrayableInterface;
 use Noz\Contracts\CollectionInterface;
-use OutOfBoundsException;
-use Traversable;
-
 use Noz\Traits\IsArrayable;
 use Noz\Traits\IsImmutable;
+use OutOfBoundsException;
+use Traversable;
 
 use function
     Noz\is_traversable,

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

Spacing +4 added lines, -4 removed lines

@@ -324,7 +324,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 {
@@ -603,7 +603,7 @@ 
     public function pairs()
     {
         return collect(array_map(
-            function ($key, $val) {
+            function($key, $val) {
                 return [$key, $val];
             },
             array_keys($this->getData()),
@@ -622,11 +622,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;
         });
     }