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;

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/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);