Novactive / NovaCollection

src/Selector/Range.php

Doc Comments +6 added lines, -2 removed lines

@@ -21,13 +21,17 @@ 
     /**
      * Explode the selector to get ranges.
      *
-     * @param $selector
+     * @param string $selector
      *
      * @return Collection
      */
     protected function getRanges($selector)
     {
         return Factory::create(explode(';', $selector))->map(
+
+            /**
+             * @param string $range
+             */
             function ($range) {
                 if ($separator = $this->dotDotSeparator($range) != false) {
                     return Factory::create([substr($range, 0, $separator), substr($range, $separator + 2)]);
@@ -73,7 +77,7 @@ 
 
     /**
      * @param Collection $parameters
-     * @param Collection $collectionl
+     * @param Collection $collection
      */
     public function convert(Collection $parameters, Collection $collection)
     {

Spacing +7 added lines, -7 removed lines

@@ -28,14 +28,14 @@ 
     protected function getRanges($selector)
     {
         return Factory::create(explode(';', $selector))->map(
-            function ($range) {
+            function($range) {
                 if ($separator = $this->dotDotSeparator($range) != false) {
-                    return Factory::create([substr($range, 0, $separator), substr($range, $separator + 2)]);
+                    return Factory::create([ substr($range, 0, $separator), substr($range, $separator + 2) ]);
                 }
                 if (($separator = $this->columnSeparator($range) != false) ||
                     ($separator = $this->dashSeparator($range) != false)
                 ) {
-                    return Factory::create([substr($range, 0, $separator), substr($range, $separator + 1)]);
+                    return Factory::create([ substr($range, 0, $separator), substr($range, $separator + 1) ]);
                 }
                 if ($separator = $this->commaSeparator($range) != false) {
                     return Factory::create(
@@ -47,7 +47,7 @@ 
                 }
 
                 // just a number here
-                return Factory::create([$range, $range]);
+                return Factory::create([ $range, $range ]);
             }
         );
     }
@@ -60,7 +60,7 @@ 
     public function supports(Collection $parameters)
     {
         return $parameters->assert(
-            function ($param) {
+            function($param) {
                 if (is_array($param) && count($param) == 2) {
                     return true;
                 }
@@ -79,7 +79,7 @@ 
     {
         $newCollection = Factory::create();
         $selector      = $parameters->map(
-            function ($param) {
+            function($param) {
                 if ((is_array($param) && count($param) == 2)) {
                     return implode(':', $param);
                 }
@@ -89,7 +89,7 @@ 
         )->implode(';');
 
         return $this->getRanges($selector)->reduce(
-            function (Collection $accumulator, Collection $range) use ($collection) {
+            function(Collection $accumulator, Collection $range) use ($collection) {
                 $from = $range->first();
                 $to   = $range->last();
                 if ($to >= $from) {