@@ -165,7 +165,7 @@ |
||
| 165 | 165 | $top = substr($this->buffer, 0, $start); |
| 166 | 166 | $data = substr($this->buffer, $start, $length); |
| 167 | 167 | $bottom = substr($this->buffer, $start + $length); |
| 168 | - $this->buffer = $top . $bottom; |
|
| 168 | + $this->buffer = $top.$bottom; |
|
| 169 | 169 | |
| 170 | 170 | return $data; |
| 171 | 171 | } |
@@ -104,7 +104,7 @@ |
||
| 104 | 104 | if (property_exists($this, $attr)) { |
| 105 | 105 | // find the appropriate setter... |
| 106 | 106 | foreach (['set', 'setIs', 'setHas'] as $prefix) { |
| 107 | - $setter = $prefix . ucfirst(strtolower($attr)); |
|
| 107 | + $setter = $prefix.ucfirst(strtolower($attr)); |
|
| 108 | 108 | if (method_exists($this, $setter)) { |
| 109 | 109 | $this->{$setter}($val); |
| 110 | 110 | } |
@@ -51,7 +51,7 @@ |
||
| 51 | 51 | } |
| 52 | 52 | $buffer .= $byte; |
| 53 | 53 | // Break when a new line is found or the max length - 1 is reached |
| 54 | - if (array_reduce($eol, function ($carry, $eol) use ($buffer) { |
|
| 54 | + if (array_reduce($eol, function($carry, $eol) use ($buffer) { |
|
| 55 | 55 | if (!$carry) { |
| 56 | 56 | $eollen = 0 - strlen($eol); |
| 57 | 57 | |
@@ -35,7 +35,7 @@ |
||
| 35 | 35 | */ |
| 36 | 36 | public function writeLine($line, $eol = PHP_EOL) |
| 37 | 37 | { |
| 38 | - return $this->write($line . $eol); |
|
| 38 | + return $this->write($line.$eol); |
|
| 39 | 39 | } |
| 40 | 40 | |
| 41 | 41 | abstract public function isWritable(); |
@@ -242,7 +242,7 @@ discard block |
||
| 242 | 242 | if (!$this->isConnected()) { |
| 243 | 243 | /** @var IOException $e */ |
| 244 | 244 | $e = null; |
| 245 | - $errhandler = function () use (&$e) { |
|
| 245 | + $errhandler = function() use (&$e) { |
|
| 246 | 246 | $e = new IOException(sprintf( |
| 247 | 247 | 'Could not open connection for %s using mode %s', |
| 248 | 248 | $this->getUri(), |
@@ -545,7 +545,7 @@ discard block |
||
| 545 | 545 | |
| 546 | 546 | return $this; |
| 547 | 547 | } |
| 548 | - throw new InvalidArgumentException('Context options must be an array, got: ' . gettype($options)); |
|
| 548 | + throw new InvalidArgumentException('Context options must be an array, got: '.gettype($options)); |
|
| 549 | 549 | } |
| 550 | 550 | |
| 551 | 551 | /** |
@@ -569,7 +569,7 @@ discard block |
||
| 569 | 569 | |
| 570 | 570 | return $this; |
| 571 | 571 | } |
| 572 | - throw new InvalidArgumentException('Context parameters must be an array, got: ' . gettype($params)); |
|
| 572 | + throw new InvalidArgumentException('Context parameters must be an array, got: '.gettype($params)); |
|
| 573 | 573 | } |
| 574 | 574 | |
| 575 | 575 | /** |
@@ -153,7 +153,7 @@ discard block |
||
| 153 | 153 | * @param string $data The data to analyze |
| 154 | 154 | * @param string $lineTerminator The line terminator char/sequence |
| 155 | 155 | * |
| 156 | - * @return array A two-row array containing quotechar, delimchar |
|
| 156 | + * @return string[] A two-row array containing quotechar, delimchar |
|
| 157 | 157 | */ |
| 158 | 158 | protected function sniffQuoteAndDelim($data, $lineTerminator) |
| 159 | 159 | { |
@@ -163,6 +163,8 @@ discard block |
||
| 163 | 163 | |
| 164 | 164 | /** |
| 165 | 165 | * @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. |
| 166 | + * @param string|false $data |
|
| 167 | + * @param string $lineTerminator |
|
| 166 | 168 | */ |
| 167 | 169 | protected function sniffDelimiter($data, $lineTerminator) |
| 168 | 170 | { |
@@ -177,11 +179,17 @@ discard block |
||
| 177 | 179 | return current($winners); |
| 178 | 180 | } |
| 179 | 181 | |
| 182 | + /** |
|
| 183 | + * @param string $eols |
|
| 184 | + */ |
|
| 180 | 185 | protected function sniffQuotingStyle($delimiter, $eols) |
| 181 | 186 | { |
| 182 | 187 | return Dialect::QUOTE_MINIMAL; |
| 183 | 188 | } |
| 184 | 189 | |
| 190 | + /** |
|
| 191 | + * @param string $eols |
|
| 192 | + */ |
|
| 185 | 193 | protected function sniffHeader($delimiter, $eols) |
| 186 | 194 | { |
| 187 | 195 | return true; |
@@ -35,9 +35,9 @@ |
||
| 35 | 35 | // in this case we really only care about newlines so we pass in a comma as the delim |
| 36 | 36 | $str = $this->replaceQuotedSpecialChars($data, ','); |
| 37 | 37 | $eols = [ |
| 38 | - static::EOL_WINDOWS => "\r\n", // 0x0D - 0x0A - Windows, DOS OS/2 |
|
| 39 | - static::EOL_UNIX => "\n", // 0x0A - - Unix, OSX |
|
| 40 | - static::EOL_OTHER => "\r", // 0x0D - - Other |
|
| 38 | + static::EOL_WINDOWS => "\r\n", // 0x0D - 0x0A - Windows, DOS OS/2 |
|
| 39 | + static::EOL_UNIX => "\n", // 0x0A - - Unix, OSX |
|
| 40 | + static::EOL_OTHER => "\r", // 0x0D - - Other |
|
| 41 | 41 | ]; |
| 42 | 42 | |
| 43 | 43 | $curCount = 0; |
@@ -91,7 +91,7 @@ |
||
| 91 | 91 | * |
| 92 | 92 | * @param string $data The data to check |
| 93 | 93 | * |
| 94 | - * @return bool |
|
| 94 | + * @return integer |
|
| 95 | 95 | */ |
| 96 | 96 | protected function isQuoted($data) |
| 97 | 97 | { |
@@ -64,7 +64,7 @@ |
||
| 64 | 64 | if (is_null($eol)) { |
| 65 | 65 | $eol = "\r\n|\r|\n"; |
| 66 | 66 | } |
| 67 | - return preg_replace_callback('/([\'"])(.*)\1/imsU', function ($matches) use ($delim, $eol) { |
|
| 67 | + return preg_replace_callback('/([\'"])(.*)\1/imsU', function($matches) use ($delim, $eol) { |
|
| 68 | 68 | $ret = preg_replace("/({$eol})/", static::PLACEHOLDER_NEWLINE, $matches[0]); |
| 69 | 69 | if (!is_null($delim)) { |
| 70 | 70 | $ret = str_replace($delim, static::PLACEHOLDER_DELIM, $ret); |