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)

Indentation -7 removed lines

@@ -55,10 +55,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 ...         $_
  *
@@ -136,10 +134,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
@@ -245,12 +241,9 @@ 
  * Provided with the requested offset, whether it be a string, an integer (positive or negative), or some type of
  * object, this function will normalize it to a positive integer offset or, failing that, it will throw an exception.
  * A negative offset will require either the traversable that is being indexed or its total count in order to normalize
-
  * @param int|mixed $offset The offset to normalize
  * @param int|array|traversable $count  Either the traversable count, or the traversable itself.
-
  * @return int
-
  * @throws RuntimeException If offset cannot be normalized
  * @throws InvalidArgumentException If offset is negative and count is not provided
  */

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

@@ -9,24 +9,15 @@
  */
 namespace Noz;
 
-use Closure;
 use Illuminate\Support\Str;
 use InvalidArgumentException;
 use Iterator;
+use Noz\Contracts\CollectionInterface;
 use Noz\Immutable\Collection;
 use Noz\Immutable\Sequence;
-use Noz\Contracts\CollectionInterface;
 use RuntimeException;
-use Serializable;
 use Traversable;
 
-use Symfony\Component\Serializer\Serializer;
-use Symfony\Component\Serializer\Encoder\XmlEncoder;
-use Symfony\Component\Serializer\Encoder\JsonEncoder;
-use Symfony\Component\Serializer\Encoder\YamlEncoder;
-use Symfony\Component\Serializer\Encoder\CsvEncoder;
-use Symfony\Component\Serializer\Normalizer\ObjectNormalizer;
-
 /**
  * Collection factory.
  *

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/Traits/IsContainer.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
     public function contains($value, $index = null)
     {
-        return (bool) $this->fold(function ($carry, $val, $key) use ($value, $index) {
+        return (bool) $this->fold(function($carry, $val, $key) use ($value, $index) {
             if ($carry) {
                 return $carry;
             }

src/Traits/IsArrayable.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
      *

src/Immutable/Collection.php

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

@@ -9,21 +9,18 @@
  */
 namespace Noz\Collection;
 
-use InvalidArgumentException;
-use OutOfBoundsException;
-
 use ArrayIterator;
 use Countable;
+use InvalidArgumentException;
 use Iterator;
-use Traversable;
-
 use Noz\Contracts\Arrayable;
-use Noz\Contracts\Invokable;
 use Noz\Contracts\CollectionInterface;
-
+use Noz\Contracts\Invokable;
 use Noz\Traits\IsArrayable;
 use Noz\Traits\IsContainer;
 use Noz\Traits\IsSerializable;
+use OutOfBoundsException;
+use Traversable;
 
 use function
     Noz\is_traversable,

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/Immutable/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/Immutable/LList.php

Doc Comments +3 added lines, -3 removed lines

@@ -205,7 +205,7 @@ 
      *
      * @param callable|null $funk The callback
      *
-     * @return bool
+     * @return boolean|null
      */
     public function every(callable $funk = null)
     {
@@ -224,7 +224,7 @@ 
      *
      * @param callable|null $callback The callback
      *
-     * @return bool
+     * @return boolean|null
      */
     public function none(callable $callback = null)
     {
@@ -282,7 +282,7 @@ 
 
     /**
      * @param callable|null $folder
-     * @param null          $initial
+     * @param boolean          $initial
      *
      * @return null
      */

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

@@ -9,21 +9,18 @@
  */
 namespace Noz\Immutable;
 
-use InvalidArgumentException;
-
 use Countable;
-use Traversable;
-use Serializable;
-use SplDoublyLinkedList;
-
-use Noz\Contracts\Structure\Listable;
-use Noz\Contracts\Immutable;
+use InvalidArgumentException;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Listable;
 use Noz\Traits\IsArrayable;
-use Noz\Traits\IsImmutable;
 use Noz\Traits\IsContainer;
+use Noz\Traits\IsImmutable;
+use Serializable;
+use SplDoublyLinkedList;
+use Traversable;
 
 use function
     Noz\is_traversable;

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         } else {
             $dll = new SplDoublyLinkedList($data);
             $dll->setIteratorMode(SplDoublyLinkedList::IT_MODE_KEEP);
-            foreach($data as $key => $val) {
+            foreach ($data as $key => $val) {
                 $dll->push($val);
             }
         }

src/Immutable/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/Immutable/Sequence.php

Doc Comments +2 added lines, -2 removed lines

@@ -164,7 +164,7 @@ 
     /**
      * Return the key of the current element.
      *
-     * @return mixed|null
+     * @return integer
      */
     public function key()
     {
@@ -376,7 +376,7 @@ 
      * Fold (reduce) sequence into a single value.
      *
      * @param callable $funk    A callback function
-     * @param mixed    $initial Initial value for accumulator
+     * @param boolean    $initial Initial value for accumulator
      *
      * @return mixed
      */

Unused Use Statements +6 added lines, -11 removed lines

@@ -9,27 +9,22 @@
  */
 namespace Noz\Immutable;
 
-use BadMethodCallException;
-use RuntimeException;
-
-use Iterator;
 use ArrayAccess;
+use BadMethodCallException;
 use Countable;
-use Serializable;
-use SplFixedArray;
-use Traversable;
-
 use Illuminate\Support\Str;
-
+use Iterator;
 use Noz\Contracts\Arrayable;
 use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
 use Noz\Contracts\Structure\Sequenceable;
-
+use Noz\Traits\IsArrayable;
 use Noz\Traits\IsContainer;
 use Noz\Traits\IsImmutable;
-use Noz\Traits\IsArrayable;
 use Noz\Traits\IsSerializable;
+use RuntimeException;
+use SplFixedArray;
+use Traversable;
 
 use function
     Noz\to_array,

Indentation +6 added lines, -10 removed lines

@@ -83,16 +83,12 @@ 
 
     /**
      * Invoke sequence.
-
      * A sequence is invokable as if it were a function. This allows some pretty useful functionality such as negative
      * indexing, sub-sequence selection, etc. Basically, any way you invoke a sequence, you're going to get back either
      * a single value from the sequence or a subset of it.
-
      * @internal param mixed $funk Either a numerical offset (positive or negative), a range string (start:end), or a
      * callback to be used as a filter.
-
      * @return mixed
-
      * @todo Put all the slice logic into a helper function or several
      */
     public function __invoke()
@@ -360,12 +356,12 @@ 
      *
      * @return Sequence
      */
-     public function prepend($item)
-     {
-         $arr = $this->getData();
-         array_unshift($arr, $item);
-         return new static($arr);
-     }
+        public function prepend($item)
+        {
+            $arr = $this->getData();
+            array_unshift($arr, $item);
+            return new static($arr);
+        }
 
     /**
      * Append item to collection.

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
             $count = $this->count();
             if (count($args)) {
                 // if there are more args...
-                $length =  array_shift($args);
+                $length = array_shift($args);
             }
             if (Str::contains($offset, static::SLICE_DELIM)) {
                 list($start, $length) = get_range_start_end($offset, $count);
@@ -413,7 +413,7 @@ 
     public function isEmpty(callable $funk = null)
     {
         if (!is_null($funk)) {
-            return $this->fold(function ($carry, $val) use ($funk) {
+            return $this->fold(function($carry, $val) use ($funk) {
                 return $carry && $funk($val);
             }, true);
         }