nozavroni / collections

src/Collection/AbstractCollection.php

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

@@ -16,9 +16,8 @@
 use Iterator;
 use Noz\Contracts\ArrayableInterface;
 use Noz\Contracts\CollectionInterface;
-use OutOfBoundsException;
-
 use Noz\Traits\IsArrayable;
+use OutOfBoundsException;
 
 use function
     Noz\is_traversable,

Spacing +5 added lines, -5 removed lines

@@ -457,7 +457,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 {
@@ -762,7 +762,7 @@ 
                 }
             }
             // if row contains an array it isn't tabular
-            if (array_reduce($row, function ($carry, $item) {
+            if (array_reduce($row, function($carry, $item) {
                 return is_array($item) && $carry;
             }, true)) {
                 return false;
@@ -893,7 +893,7 @@ 
     public function pairs()
     {
         return collect(array_map(
-            function ($key, $val) {
+            function($key, $val) {
                 return [$key, $val];
             },
             array_keys($this->data),
@@ -912,11 +912,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;
         });
     }

src/Contracts/CollectionInterface.php

Doc Comments +6 added lines, -6 removed lines

@@ -70,7 +70,7 @@ 
      * @param string|int $index Set value at this index
      * @param mixed      $value Set this value at index
      *
-     * @return mixed
+     * @return \Noz\Collection\AbstractCollection
      */
     public function set($index, $value);
 
@@ -82,7 +82,7 @@ 
      * @param string|int $index Add value at this index
      * @param mixed      $value Add this value at index
      *
-     * @return mixed
+     * @return \Noz\Collection\AbstractCollection
      */
     public function add($index, $value);
 
@@ -91,7 +91,7 @@ 
      *
      * @param string|int $index Delete item at this index
      *
-     * @return mixed
+     * @return \Noz\Collection\AbstractCollection
      */
     public function delete($index);
 
@@ -137,7 +137,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);
 
@@ -244,7 +244,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);
 
@@ -330,7 +330,7 @@ 
      *
      * @param int $num Number of items to return
      *
-     * @return mixed|CollectionInterface
+     * @return CollectionInterface
      */
     public function random($num);
 

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
      *

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)