nozavroni / csvelte

src/CSVelte/Traits/IsWritable.php

Spacing +1 added lines, -1 removed lines

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

src/CSVelte/Reader.php

Spacing +3 added lines, -3 removed lines

@@ -283,7 +283,7 @@ 
      */
     protected function replaceQuotedSpecialChars($data, $delim, $quo, $eol)
     {
-        return preg_replace_callback('/(['. preg_quote($quo, '/') . '])(.*)\1/imsU', function($matches) use ($delim, $eol) {
+        return preg_replace_callback('/(['.preg_quote($quo, '/').'])(.*)\1/imsU', function($matches) use ($delim, $eol) {
             $ret = str_replace($eol, self::PLACEHOLDER_NEWLINE, $matches[0]);
             $ret = str_replace($delim, self::PLACEHOLDER_DELIM, $ret);
             return $ret;
@@ -335,7 +335,7 @@ 
      */
     protected function unEscape($str, $esc, $quo)
     {
-        return str_replace($esc . $quo, $quo, $str);
+        return str_replace($esc.$quo, $quo, $str);
     }
 
     /**
@@ -476,7 +476,7 @@ 
      */
     public function toArray()
     {
-        return array_map(function($row){
+        return array_map(function($row) {
             return $row->toArray();
         }, iterator_to_array($this));
     }

src/CSVelte/Taster.php

Spacing +16 added lines, -16 removed lines

@@ -215,16 +215,16 @@ 
     {
         $str = $this->removeQuotedStrings($this->sample);
         $eols = [
-            self::EOL_WINDOWS => "\r\n",  // 0x0D - 0x0A - Windows, DOS OS/2
-            self::EOL_UNIX    => "\n",    // 0x0A -      - Unix, OSX
-            self::EOL_TRS80   => "\r",    // 0x0D -      - Apple ][, TRS80
+            self::EOL_WINDOWS => "\r\n", // 0x0D - 0x0A - Windows, DOS OS/2
+            self::EOL_UNIX    => "\n", // 0x0A -      - Unix, OSX
+            self::EOL_TRS80   => "\r", // 0x0D -      - Apple ][, TRS80
         ];
 
         $curCount = 0;
         // @todo This should return a default maybe?
         $curEol = PHP_EOL;
-        foreach($eols as $k => $eol) {
-            if( ($count = substr_count($str, $eol)) > $curCount) {
+        foreach ($eols as $k => $eol) {
+            if (($count = substr_count($str, $eol)) > $curCount) {
                 $curCount = $count;
                 $curEol = $eol;
             }
@@ -256,12 +256,12 @@ 
         $patterns = [];
         // delim can be anything but line breaks, quotes, alphanumeric, underscore, backslash, or any type of spaces
         $antidelims = implode(array("\r", "\n", "\w", preg_quote('"', '/'), preg_quote("'", '/')/*, preg_quote('\\', '/')*/, preg_quote(chr(self::SPACE), '/')));
-        $delim = '(?P<delim>[^' . $antidelims . '])';
+        $delim = '(?P<delim>[^'.$antidelims.'])';
         $quote = '(?P<quoteChar>"|\'|`)'; // @todo I think MS Excel uses some strange encoding for fancy open/close quotes
-        $patterns[] = '/' . $delim . ' ?' . $quote . '.*?\2\1/ms'; // ,"something", - anything but whitespace or quotes followed by a possible space followed by a quote followed by anything followed by same quote, followed by same anything but whitespace
-        $patterns[] = '/(?:^|\n)' . $quote . '.*?\1' . $delim . ' ?/ms'; // 'something', - beginning of line or line break, followed by quote followed by anything followed by quote followed by anything but whitespace or quotes
-        $patterns[] = '/' . $delim . ' ?' . $quote . '.*?\2(?:^|\n)/ms'; // ,'something' - anything but whitespace or quote followed by possible space followed by quote followed by anything followed by quote, followed by end of line
-        $patterns[] = '/(?:^|\n)' . $quote . '.*?\2(?:$|\n)/ms'; // 'something' - beginning of line followed by quote followed by anything followed by quote followed by same quote followed by end of line
+        $patterns[] = '/'.$delim.' ?'.$quote.'.*?\2\1/ms'; // ,"something", - anything but whitespace or quotes followed by a possible space followed by a quote followed by anything followed by same quote, followed by same anything but whitespace
+        $patterns[] = '/(?:^|\n)'.$quote.'.*?\1'.$delim.' ?/ms'; // 'something', - beginning of line or line break, followed by quote followed by anything followed by quote followed by anything but whitespace or quotes
+        $patterns[] = '/'.$delim.' ?'.$quote.'.*?\2(?:^|\n)/ms'; // ,'something' - anything but whitespace or quote followed by possible space followed by quote followed by anything followed by quote, followed by end of line
+        $patterns[] = '/(?:^|\n)'.$quote.'.*?\2(?:$|\n)/ms'; // 'something' - beginning of line followed by quote followed by anything followed by quote followed by same quote followed by end of line
         foreach ($patterns as $pattern) {
             // @todo I had to add the error suppression char here because it was
             //     causing undefined offset errors with certain data sets. strange...
@@ -530,7 +530,7 @@ 
             if ($types->contains(self::DATA_NONNUMERIC)) {
                 // allow for a SMALL amount of error here
                 $counts = collect([self::DATA_SPECIAL => 0, self::DATA_NONNUMERIC => 0]);
-                $freq->get('quoted')->walk(function ($type) use (&$counts) {
+                $freq->get('quoted')->walk(function($type) use (&$counts) {
                     $counts->increment($type);
                 });
                 // @todo is all this even necessary? seems unnecessary to me...
@@ -651,13 +651,13 @@ 
                 $day = '[0-3]?[0-9]';
                 $sep = '[\/\.\-]?';
                 $time = '([0-2]?[0-9](:[0-5][0-9]){1,2}(am|pm)?|[01]?[0-9](am|pm))';
-                $date = '(' . $month . $sep . $day . $sep . $year . '|' . $day . $sep . $month . $sep . $year . '|' . $year . $sep . $month . $sep . $day . ')';
+                $date = '('.$month.$sep.$day.$sep.$year.'|'.$day.$sep.$month.$sep.$year.'|'.$year.$sep.$month.$sep.$day.')';
                 $dt = new DateTime($data);
-                $dt->setTime(0,0,0);
+                $dt->setTime(0, 0, 0);
                 $now = new DateTime();
-                $now->setTime(0,0,0);
+                $now->setTime(0, 0, 0);
                 $diff = $dt->diff($now);
-                $diffDays = (integer) $diff->format( "%R%a" );
+                $diffDays = (integer) $diff->format("%R%a");
                 if ($diffDays === 0) {
                     // then this is most likely a time string...
                     if (preg_match("/^{$time}$/i", $data)) {
@@ -666,7 +666,7 @@ 
                 }
                 if (preg_match("/^{$date}$/i", $data)) {
                     return self::TYPE_DATE;
-                } elseif(preg_match("/^{$date} {$time}$/i")) {
+                } elseif (preg_match("/^{$date} {$time}$/i")) {
                     return self::TYPE_DATETIME;
                 }
             } catch (\Exception $e) {

Indentation +25 added lines, -25 removed lines

@@ -292,20 +292,20 @@ 
         throw new TasterException("quoteChar and delimiter cannot be determined", TasterException::ERR_QUOTE_AND_DELIM);
     }
 
-     /**
-      * Take a list of likely delimiter characters and find the one that occurs
-      * the most consistent amount of times within the provided data.
-      *
-      * @param string $eol The character(s) used for newlines
-      * @return string One of four Flavor::QUOTING_* constants
-      * @see Flavor for possible quote style constants
-      * @todo Refactor this method--It needs more thorough testing against a wider
-      *     variety of CSV data to be sure it works reliably. And I'm sure there
-      *     are many performance and logic improvements that could be made. This
-      *     is essentially a first draft.
-      * @todo Can't use replaceQuotedSpecialChars rather than removeQuotedStrings
-      *     because the former requires u to know the delimiter
-      */
+        /**
+         * Take a list of likely delimiter characters and find the one that occurs
+         * the most consistent amount of times within the provided data.
+         *
+         * @param string $eol The character(s) used for newlines
+         * @return string One of four Flavor::QUOTING_* constants
+         * @see Flavor for possible quote style constants
+         * @todo Refactor this method--It needs more thorough testing against a wider
+         *     variety of CSV data to be sure it works reliably. And I'm sure there
+         *     are many performance and logic improvements that could be made. This
+         *     is essentially a first draft.
+         * @todo Can't use replaceQuotedSpecialChars rather than removeQuotedStrings
+         *     because the former requires u to know the delimiter
+         */
     protected function lickDelimiter($eol = "\n")
     {
         $frequencies = [];
@@ -382,16 +382,16 @@ 
              *     which one has the best distribution, return that one.
              */
 
-             $decision = $dups->get($max);
-             try {
-                 return $this->guessDelimByDistribution($decision, $eol);
-             } catch (TasterException $e) {
-                 // if somehow we STILL can't come to a consensus, then fall back to a
-                 // "preferred delimiters" list...
-                 foreach ($this->delims as $key => $val) {
+                $decision = $dups->get($max);
+                try {
+                    return $this->guessDelimByDistribution($decision, $eol);
+                } catch (TasterException $e) {
+                    // if somehow we STILL can't come to a consensus, then fall back to a
+                    // "preferred delimiters" list...
+                    foreach ($this->delims as $key => $val) {
                     if ($delim = array_search($val, $decision)) return $delim;
-                 }
-             }
+                    }
+                }
         }
         return $delims
             ->sort()
@@ -450,8 +450,8 @@ 
             })->map(function($dists) {
                 return $dists->average();
             })->sort()
-              ->reverse()
-              ->getKeyAtPosition(0)];
+                ->reverse()
+                ->getKeyAtPosition(0)];
         } catch (Exception $e) {
             throw new TasterException("delimiter cannot be determined by distribution", TasterException::ERR_DELIMITER);
         }

src/CSVelte/Table/AbstractRow.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
             } elseif ($fields instanceof Iterator) {
                 $fields = iterator_to_array($fields);
             } else {
-                throw new InvalidArgumentException(__CLASS__ . " requires an array, got: " . gettype($fields));
+                throw new InvalidArgumentException(__CLASS__." requires an array, got: ".gettype($fields));
             }
         }
         $this->fields = collect(array_values($fields));

src/CSVelte/IO/BufferStream.php

Indentation +10 added lines, -10 removed lines

@@ -267,16 +267,16 @@
         return strlen($data);
     }
 
-     /**
-      * Seekability accessor.
-      *
-      * Despite the fact that any class that implements this interface must also
-      * define methods such as seek, that is no guarantee that an
-      * object will necessarily be seekable. This method should tell the user
-      * whether a stream is, in fact, seekable.
-      *
-      * @return boolean True if seekable, false otherwise
-      */
+        /**
+         * Seekability accessor.
+         *
+         * Despite the fact that any class that implements this interface must also
+         * define methods such as seek, that is no guarantee that an
+         * object will necessarily be seekable. This method should tell the user
+         * whether a stream is, in fact, seekable.
+         *
+         * @return boolean True if seekable, false otherwise
+         */
     public function isSeekable()
     {
         return $this->seekable;

src/CSVelte/Collection.php

Spacing +9 added lines, -9 removed lines

@@ -194,7 +194,7 @@ 
     public function toArray()
     {
         $data = [];
-        foreach($this->data as $key => $val) {
+        foreach ($this->data as $key => $val) {
             $data[$key] = (is_object($val) && method_exists($val, 'toArray')) ? $val->toArray() : $val;
         }
         return $data;
@@ -369,7 +369,7 @@ 
             if ($i === $pos) return $key;
             $i++;
         }
-        throw new OutOfBoundsException("Collection data does not contain a key at given position: " . $pos);
+        throw new OutOfBoundsException("Collection data does not contain a key at given position: ".$pos);
     }
 
     /**
@@ -554,7 +554,7 @@ 
             return $this->data[$key];
         } else {
             if ($throwExc) {
-                throw new OutOfBoundsException("Collection data does not contain value for given key: " . $key);
+                throw new OutOfBoundsException("Collection data does not contain value for given key: ".$key);
             }
         }
         return $default;
@@ -899,7 +899,7 @@ 
     public function pairs($alt = false)
     {
         return new self(array_map(
-            function ($key, $val) use ($alt) {
+            function($key, $val) use ($alt) {
                 if ($alt) {
                     return [$key => $val];
                 } else {
@@ -981,7 +981,7 @@ 
         if (false !== ($condRet = $this->if2DMapInternalMethod(__METHOD__))) {
             return $condRet;
         }
-        $strvals = $this->map(function($val){
+        $strvals = $this->map(function($val) {
             return (string) $val;
         });
         $this->assertNumericValues();
@@ -1095,7 +1095,7 @@ 
         $this->assertIsTabular();
         return $this->sort(function($a, $b) use ($key, $cmp) {
             if (!isset($a[$key]) || !isset($b[$key])) {
-                throw new RuntimeException('Cannot order collection by non-existant key: ' . $key);
+                throw new RuntimeException('Cannot order collection by non-existant key: '.$key);
             }
             if (is_null($cmp)) {
                 return strcasecmp($a[$key], $b[$key]);
@@ -1142,7 +1142,7 @@ 
      */
     public function is2D()
     {
-        return !$this->contains(function($val){
+        return !$this->contains(function($val) {
             return !is_array($val);
         });
         return false;
@@ -1212,7 +1212,7 @@ 
 
     protected function assertNumericValues()
     {
-        if ($this->contains(function($val){
+        if ($this->contains(function($val) {
             return !is_numeric($val);
         })) {
             // can't average non-numeric data
@@ -1228,6 +1228,6 @@ 
         if (is_null($data) || is_array($data) || $data instanceof Iterator) {
             return;
         }
-        throw new InvalidArgumentException("Invalid type for collection data: " . gettype($data));
+        throw new InvalidArgumentException("Invalid type for collection data: ".gettype($data));
     }
 }

src/CSVelte/Contract/Streamable.php

Indentation +10 added lines, -10 removed lines

@@ -267,16 +267,16 @@
         return strlen($data);
     }
 
-     /**
-      * Seekability accessor.
-      *
-      * Despite the fact that any class that implements this interface must also
-      * define methods such as seek, that is no guarantee that an
-      * object will necessarily be seekable. This method should tell the user
-      * whether a stream is, in fact, seekable.
-      *
-      * @return boolean True if seekable, false otherwise
-      */
+        /**
+         * Seekability accessor.
+         *
+         * Despite the fact that any class that implements this interface must also
+         * define methods such as seek, that is no guarantee that an
+         * object will necessarily be seekable. This method should tell the user
+         * whether a stream is, in fact, seekable.
+         *
+         * @return boolean True if seekable, false otherwise
+         */
     public function isSeekable()
     {
         return $this->seekable;

src/CSVelte/IO/IteratorStream.php

Indentation +10 added lines, -10 removed lines

@@ -267,16 +267,16 @@
         return strlen($data);
     }
 
-     /**
-      * Seekability accessor.
-      *
-      * Despite the fact that any class that implements this interface must also
-      * define methods such as seek, that is no guarantee that an
-      * object will necessarily be seekable. This method should tell the user
-      * whether a stream is, in fact, seekable.
-      *
-      * @return boolean True if seekable, false otherwise
-      */
+        /**
+         * Seekability accessor.
+         *
+         * Despite the fact that any class that implements this interface must also
+         * define methods such as seek, that is no guarantee that an
+         * object will necessarily be seekable. This method should tell the user
+         * whether a stream is, in fact, seekable.
+         *
+         * @return boolean True if seekable, false otherwise
+         */
     public function isSeekable()
     {
         return $this->seekable;

src/CSVelte/IO/Resource.php

Spacing +3 added lines, -3 removed lines

@@ -239,7 +239,7 @@ 
     {
         if (!$this->isConnected()) {
             $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(),
@@ -555,7 +555,7 @@ 
             $this->updateContext();
             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));
     }
 
     /**
@@ -575,7 +575,7 @@ 
             $this->updateContext();
             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));
     }
 
     /**

Indentation +7 added lines, -7 removed lines

@@ -185,10 +185,10 @@ 
 
         // ok we're opening a new stream resource handle
         $this->setUri($uri)
-             ->setMode($mode)
-             ->setLazy($lazy)
-             ->setUseIncludePath($use_include_path)
-             ->setContext($context_options, $context_params);
+                ->setMode($mode)
+                ->setLazy($lazy)
+                ->setUseIncludePath($use_include_path)
+                ->setContext($context_options, $context_params);
         if (!$this->isLazy()) {
             $this->connect();
         }
@@ -351,9 +351,9 @@ 
 
         $this->flag = '';
         $this->setBaseMode($base)
-             ->setIsPlus($plus == '+')
-             ->setIsText($flag == 't')
-             ->setIsBinary($flag == 'b');
+                ->setIsPlus($plus == '+')
+                ->setIsText($flag == 't')
+                ->setIsBinary($flag == 'b');
 
         return $this;
     }