nozavroni / csvelte

src/CSVelte/Collection/AbstractCollection.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 
 use ArrayAccess;
 use ArrayIterator;
-use BadMethodCallException;
 use Countable;
 use InvalidArgumentException;
 use Iterator;

Spacing +11 added lines, -11 removed lines

@@ -165,7 +165,7 @@ 
      *
      * @return mixed
      */
-    public function current ()
+    public function current()
     {
         return current($this->data);
     }
@@ -177,7 +177,7 @@ 
      *
      * @return mixed
      */
-    public function key ()
+    public function key()
     {
         return key($this->data);
     }
@@ -191,7 +191,7 @@ 
      *
      * @return mixed
      */
-    public function next ()
+    public function next()
     {
         $next = next($this->data);
         $key = key($this->data);
@@ -210,7 +210,7 @@ 
      *
      * @return mixed
      */
-    public function rewind ()
+    public function rewind()
     {
         $this->isValid = true;
         return reset($this->data);
@@ -223,7 +223,7 @@ 
      *
      * @return bool True if internal pointer isn't past the end
      */
-    public function valid ()
+    public function valid()
     {
         return $this->isValid;
     }
@@ -292,7 +292,7 @@ 
             return $this->data[$index];
         }
         if ($throw) {
-            throw new OutOfBoundsException(__CLASS__ . " could not find value at index " . $index);
+            throw new OutOfBoundsException(__CLASS__." could not find value at index ".$index);
         }
         return $default;
     }
@@ -332,7 +332,7 @@ 
             unset($this->data[$index]);
         } else {
             if ($throw) {
-                throw new OutOfBoundsException('No value found at given index: ' . $index);
+                throw new OutOfBoundsException('No value found at given index: '.$index);
             }
         }
 
@@ -621,7 +621,7 @@ 
     public function pairs()
     {
         return static::factory(array_map(
-            function ($key, $val) {
+            function($key, $val) {
                 return [$key, $val];
             },
             array_keys($this->data),
@@ -766,7 +766,7 @@ 
                 }
 
                 if (method_exists($val, '__toString')) {
-                    $val = (string)$val;
+                    $val = (string) $val;
                 } else {
                     $val = spl_object_hash($val);
                 }
@@ -815,7 +815,7 @@ 
     protected function assertCorrectInputDataType($data)
     {
         if (!$this->isConsistentDataStructure($data)) {
-            throw new InvalidArgumentException(__CLASS__ . ' expected traversable data, got: ' . gettype ($data));
+            throw new InvalidArgumentException(__CLASS__.' expected traversable data, got: '.gettype($data));
         }
     }
 
@@ -859,7 +859,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;

Doc Comments +5 added lines, -5 removed lines

@@ -282,7 +282,7 @@ 
      * also optionally throw an OutOfBoundsException if no value is found.
      *
      * @param mixed $index The index of the data you want to get
-     * @param mixed $default The default value to return if none available
+     * @param null|integer $default The default value to return if none available
      * @param bool $throw True if you want an exception to be thrown if no data found at $index
      * @throws OutOfBoundsException If $throw is true and $index isn't found
      * @return mixed The data found at $index or failing that, the $default
@@ -547,7 +547,7 @@ 
      * Returns a new collection with $items added.
      *
      * @param array $items Any number of arguments will be pushed onto the
-     * @return mixed The first item in this collection
+     * @return AbstractCollection The first item in this collection
      */
     public function push(...$items)
     {
@@ -560,7 +560,7 @@ 
      *
      * Returns a new collection with $items added.
      *
-     * @return mixed The first item in this collection
+     * @return AbstractCollection The first item in this collection
      */
     public function unshift(...$items)
     {
@@ -616,7 +616,7 @@ 
     /**
      * Iterate over each item that matches criteria in callback
      *
-     * @param Closure|callable $callback A callback to use
+     * @param Closure $callback A callback to use
      * @param object $bindTo The object to bind to
      * @return AbstractCollection
      */
@@ -724,7 +724,7 @@ 
     /**
      * Returns collection in reverse order.
      *
-     * @param null $preserveKeys True if you want to preserve collection's keys
+     * @param boolean $preserveKeys True if you want to preserve collection's keys
      * @return AbstractCollection This collection in reverse order.
      */
     public function reverse($preserveKeys = null)

src/CSVelte/Collection/TabularCollection.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
             return static::factory($values);
         }
         if ($throw) {
-            throw new OutOfBoundsException(__CLASS__ . " could not find column: " . $column);
+            throw new OutOfBoundsException(__CLASS__." could not find column: ".$column);
         }
         return false;
     }
@@ -169,6 +169,6 @@ 
                 return call_user_func_array([$column, $method], $args);
             }
         }
-        throw new BadMethodCallException("Method does not exist: " . __CLASS__ . "::{$method}()");
+        throw new BadMethodCallException("Method does not exist: ".__CLASS__."::{$method}()");
     }
 }
\ No newline at end of file