nitpicks

Author: freekmurze

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes to `laravel-collection-macros` will be documented in this file
 
+## 2.6.0 - 2017-08-22
+
+- Added `tail`, `eachCons`, `sliceBefore` and `chunkBy`
+
 ## 2.5.0 - 2017-08-03
 - Added customization options for `sectionBy`
 

README.md

@@ -446,30 +446,30 @@ Extract the tail from a collection. So everything except the first element.
 It's a shorthand for `slice(1)->values()`, but nevertheless very handy.
 
 ```php
-collect([1, 2, 3))->tail(); // return Collection([2, 3])
+collect([1, 2, 3))->tail(); // return collect([2, 3])
 ```
 
 ### `eachCons`
 
-Get the following consecutive neighbours in a collection from a given chunk size
+Get the following consecutive neighbours in a collection from a given chunk size.
 
 ```php
-collect([1, 2, 3, 4])->eachCons(2); // return Collection([[1, 2], [2, 3], [3, 4]])
+collect([1, 2, 3, 4])->eachCons(2); // return collect([[1, 2], [2, 3], [3, 4]])
 ```
 
 ### `sliceBefore`
 
-Slice the values out from a collection before the given callback is true
+Slice the values out from a collection before the given callback is true.
 
 ```php
 collect([20, 51, 10, 50, 66])->sliceBefore(function($item) {
     return $item > 50;
-}); // return Collection([[20],[51, 10]])
+}); // return collect([[20],[51, 10]])
 ```
 
 ### `chunkBy`
 
-Chunks the values from a collection into groups as long the given callback is true
+Chunks the values from a collection into groups as long the given callback is true.
 
 ```php
 collect(['A', 'A', 'B', 'A'])->chunkBy(function($item) {

src/macros.php

@@ -418,10 +418,10 @@
      */
     Collection::macro('eachCons', function ($chunkSize) {
         if ($this->count() < $chunkSize) {
-            return new self;
+            return new static();
         }
 
-        return (new self([$this->take($chunkSize)->values()]))
+        return (new static([$this->take($chunkSize)->values()]))
             ->merge($this->tail()->eachCons($chunkSize));
     });
 }
@@ -434,20 +434,19 @@
      */
     Collection::macro('sliceBefore', function ($callback) {
         if ($this->isEmpty()) {
-            return new self;
+            return new static();
         }
 
-        $sliced = new self([
-            new self([$this->first()]),
+        $sliced = new static([
+            new static([$this->first()]),
         ]);
 
-        return $this->eachCons(2)->reduce(function ($sliced, $prevAndCurr) use ($callback) {
-            list($previousItem, $item) = $prevAndCurr;
-            if ($callback($item, $previousItem)) {
-                $sliced->push(new self([$item]));
-            } else {
-                $sliced->last()->push($item);
-            }
+        return $this->eachCons(2)->reduce(function ($sliced, $previousAndCurrent) use ($callback) {
+            list($previousItem, $item) = $previousAndCurrent;
+
+            $callback($item, $previousItem)
+                ? $sliced->push(new static([$item]))
+                : $sliced->last()->push($item);
 
             return $sliced;
         }, $sliced);