Skip to content

Commit 40d85d0

Browse files
committed
Replace variable with constant
1 parent ff9aed8 commit 40d85d0

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

src/LineBreaksHelper.php

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44

55
class LineBreaksHelper
66
{
7+
const REGEXP_DELIMITER = '~';
8+
79
/**
810
* Detect line-breaks style in CSV file
911
* @param string $sample
@@ -60,13 +62,12 @@ public static function clearCsvValues($sample, $enclosure, $escapedBy)
6062
* enclosure: |"|, escapedBy: none, regexp: ~"(?>(?>"")|[^"])*"~
6163
* enclosure: |"|, escapedBy: |\|, regexp: ~"(?>(?>\\"|\\\\)|[^"])*"~
6264
*/
63-
$regexpDelimiter = '~';
6465
// @formatter:off
6566
$regexp =
6667
// regexp start
67-
$regexpDelimiter .
68+
self::REGEXP_DELIMITER .
6869
// enclosure start
69-
preg_quote($enclosure, $regexpDelimiter) .
70+
preg_quote($enclosure, self::REGEXP_DELIMITER) .
7071
/*
7172
* Once-only group => if there is a match, do not try other alternatives
7273
* See: https://www.php.net/manual/en/regexp.reference.onlyonce.php
@@ -78,19 +79,19 @@ public static function clearCsvValues($sample, $enclosure, $escapedBy)
7879
// once-only group start
7980
'(?>' .
8081
// escaped enclosure
81-
preg_quote($escapedEnclosure, $regexpDelimiter) .
82+
preg_quote($escapedEnclosure, self::REGEXP_DELIMITER) .
8283
// OR escaped escape char
83-
($escapedEscape ? '|' . preg_quote($escapedEscape, $regexpDelimiter) : '') .
84+
($escapedEscape ? '|' . preg_quote($escapedEscape, self::REGEXP_DELIMITER) : '') .
8485
// group end
8586
')' .
8687
// OR not enclosure
87-
'|[^' . preg_quote($enclosure, $regexpDelimiter) . ']' .
88+
'|[^' . preg_quote($enclosure, self::REGEXP_DELIMITER) . ']' .
8889
// group end
8990
')*' .
9091
// enclosure end
91-
preg_quote($enclosure, $regexpDelimiter) .
92+
preg_quote($enclosure, self::REGEXP_DELIMITER) .
9293
// regexp end
93-
$regexpDelimiter;
94+
self::REGEXP_DELIMITER;
9495
// @formatter:on
9596

9697
return preg_replace($regexp, $doubleEnclosure, $sample);

0 commit comments

Comments
 (0)