nozavroni / csvelte

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/AbstractSniffer.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,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})/", self::PLACEHOLDER_NEWLINE, $matches[0]);
             if (!is_null($delim)) {
                 $ret = str_replace($delim, self::PLACEHOLDER_DELIM, $ret);