nozavroni / csvelte

src/CSVelte/Dialect.php

Indentation +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 use Traversable;
 use function Noz\collect,
-             Noz\to_array;
+                Noz\to_array;
 
 /**
  * CSV Dialect - Default dialect

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
             if (property_exists($this, $attr)) {
                 // find the appropriate setter...
                 foreach (['set', 'setIs', 'setHas'] as $prefix) {
-                    $setter = $prefix . ucfirst(strtolower($attr));
+                    $setter = $prefix.ucfirst(strtolower($attr));
                     if (method_exists($this, $setter)) {
                         $this->{$setter}($val);
                     }

src/CSVelte/Traits/IsReadable.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
             }
             $buffer .= $byte;
             // Break when a new line is found or the max length - 1 is reached
-            if (array_reduce($eol, function ($carry, $eol) use ($buffer) {
+            if (array_reduce($eol, function($carry, $eol) use ($buffer) {
                     if (!$carry) {
                         $eollen = 0 - strlen($eol);
 

src/CSVelte/Traits/IsWritable.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function writeLine($line, $eol = PHP_EOL)
     {
-        return $this->write($line . $eol);
+        return $this->write($line.$eol);
     }
 
     abstract public function isWritable();

src/CSVelte/IO/StreamResource.php

Spacing +3 added lines, -3 removed lines

@@ -242,7 +242,7 @@ 
         if (!$this->isConnected()) {
             /** @var IOException $e */
             $e          = null;
-            $errhandler = function () use (&$e) {
+            $errhandler = function() use (&$e) {
                 $e = new IOException(sprintf(
                     'Could not open connection for %s using mode %s',
                     $this->getUri(),
@@ -545,7 +545,7 @@ 
 
             return $this;
         }
-        throw new InvalidArgumentException('Context options must be an array, got: ' . gettype($options));
+        throw new InvalidArgumentException('Context options must be an array, got: '.gettype($options));
     }
 
     /**
@@ -569,7 +569,7 @@ 
 
             return $this;
         }
-        throw new InvalidArgumentException('Context parameters must be an array, got: ' . gettype($params));
+        throw new InvalidArgumentException('Context parameters must be an array, got: '.gettype($params));
     }
 
     /**

src/CSVelte/Sniffer.php

Doc Comments +9 added lines, -1 removed lines

@@ -153,7 +153,7 @@ 
      * @param string $data The data to analyze
      * @param string $lineTerminator The line terminator char/sequence
      *
-     * @return array A two-row array containing quotechar, delimchar
+     * @return string[] A two-row array containing quotechar, delimchar
      */
     protected function sniffQuoteAndDelim($data, $lineTerminator)
     {
@@ -163,6 +163,8 @@ 
 
     /**
      * @todo To make this class more oop and test-friendly, implement strategy pattern here with each delim sniffing method implemented in its own strategy class.
+     * @param string|false $data
+     * @param string $lineTerminator
      */
     protected function sniffDelimiter($data, $lineTerminator)
     {
@@ -177,11 +179,17 @@ 
         return current($winners);
     }
 
+    /**
+     * @param string $eols
+     */
     protected function sniffQuotingStyle($delimiter, $eols)
     {
         return Dialect::QUOTE_MINIMAL;
     }
 
+    /**
+     * @param string $eols
+     */
     protected function sniffHeader($delimiter, $eols)
     {
         return true;

Unused Use Statements -3 removed lines

@@ -13,15 +13,12 @@
 namespace CSVelte;
 
 use CSVelte\Contract\Streamable;
-
 use CSVelte\Exception\SnifferException;
 use CSVelte\Sniffer\SniffDelimiterByConsistency;
 use CSVelte\Sniffer\SniffDelimiterByDistribution;
 use CSVelte\Sniffer\SniffLineTerminatorByCount;
 use CSVelte\Sniffer\SniffQuoteAndDelimByAdjacency;
-use Noz\Collection\Collection;
 use function Noz\to_array;
-use RuntimeException;
 
 use function Noz\collect;
 use function Stringy\create as s;

src/CSVelte/Sniffer/SniffLineTerminatorByCount.php

Spacing +3 added lines, -3 removed lines

@@ -35,9 +35,9 @@
         // in this case we really only care about newlines so we pass in a comma as the delim
         $str = $this->replaceQuotedSpecialChars($data, ',');
         $eols = [
-            static::EOL_WINDOWS => "\r\n",  // 0x0D - 0x0A - Windows, DOS OS/2
-            static::EOL_UNIX    => "\n",    // 0x0A -      - Unix, OSX
-            static::EOL_OTHER   => "\r",    // 0x0D -      - Other
+            static::EOL_WINDOWS => "\r\n", // 0x0D - 0x0A - Windows, DOS OS/2
+            static::EOL_UNIX    => "\n", // 0x0A -      - Unix, OSX
+            static::EOL_OTHER   => "\r", // 0x0D -      - Other
         ];
 
         $curCount = 0;

src/CSVelte/Sniffer/SniffQuoteAndDelimByAdjacency.php

Indentation +12 added lines, -12 removed lines

@@ -20,18 +20,18 @@
 
 class SniffQuoteAndDelimByAdjacency extends AbstractSniffer
 {
-     /**
-     * Guess quote and delimiter character(s)
-     *
-     * If there are quoted values within the data, it is often easiest to guess the quote and delimiter characters at
-     * the same time by analyzing their adjacency to one-another. That is to say, in cases where certain values are
-     * wrapped in quotes, it can often be determined what not only that quote character is, but also the delimiter
-     * because it is often on either side of the quote character.
-     *
-     * @param string $data The data to analyze
-     *
-     * @return string[]
-     */
+        /**
+         * Guess quote and delimiter character(s)
+         *
+         * If there are quoted values within the data, it is often easiest to guess the quote and delimiter characters at
+         * the same time by analyzing their adjacency to one-another. That is to say, in cases where certain values are
+         * wrapped in quotes, it can often be determined what not only that quote character is, but also the delimiter
+         * because it is often on either side of the quote character.
+         *
+         * @param string $data The data to analyze
+         *
+         * @return string[]
+         */
     public function sniff($data)
     {
         /**

src/CSVelte/Sniffer/SniffQuoteStyle.php

Doc Comments +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
      *
      * @param string $data The data to check
      *
-     * @return bool
+     * @return integer
      */
     protected function isQuoted($data)
     {

Indentation +11 added lines, -11 removed lines

@@ -19,17 +19,17 @@
 
 class SniffQuoteStyle extends AbstractSniffer
 {
-     /**
-     * Guess quoting style
-     *
-     * The quoting style refers to which types of columns are quoted within a csv dataset. The dialect class defines
-     * four possible quoting styles; all, none, minimal, or non-numeric. This class attempts to determine which of those
-     * four it is by analyzing the content within each quoted value.
-     *
-     * @param string $data The data to analyze
-     *
-     * @return int
-     */
+        /**
+         * Guess quoting style
+         *
+         * The quoting style refers to which types of columns are quoted within a csv dataset. The dialect class defines
+         * four possible quoting styles; all, none, minimal, or non-numeric. This class attempts to determine which of those
+         * four it is by analyzing the content within each quoted value.
+         *
+         * @param string $data The data to analyze
+         *
+         * @return int
+         */
     public function sniff($data)
     {
         $styles = collect([

src/CSVelte/Sniffer/AbstractSniffer.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
         if (is_null($eol)) {
             $eol = "\r\n|\r|\n";
         }
-        return preg_replace_callback('/([\'"])(.*)\1/imsU', function ($matches) use ($delim, $eol) {
+        return preg_replace_callback('/([\'"])(.*)\1/imsU', function($matches) use ($delim, $eol) {
             $ret = preg_replace("/({$eol})/", static::PLACEHOLDER_NEWLINE, $matches[0]);
             if (!is_null($delim)) {
                 $ret = str_replace($delim, static::PLACEHOLDER_DELIM, $ret);