Skip to content

Commit 831ff34

Browse files
committed
Merge branch 'feature/generic-disallowtabindent-bugfix-php73-flexible-heredoc-nowdoc' of https://github.com/jrfnl/PHP_CodeSniffer
2 parents 19179ab + 9cc8d43 commit 831ff34

File tree

5 files changed

+45
-5
lines changed

5 files changed

+45
-5
lines changed

package.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -804,6 +804,8 @@ http://pear.php.net/dtd/package-2.0.xsd">
804804
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.1.inc.fixed" role="test" />
805805
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.2.inc" role="test" />
806806
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.2.inc.fixed" role="test" />
807+
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.3.inc" role="test" />
808+
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.3.inc.fixed" role="test" />
807809
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.js" role="test" />
808810
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.js.fixed" role="test" />
809811
<file baseinstalldir="PHP/CodeSniffer" name="DisallowTabIndentUnitTest.php" role="test" />

src/Standards/Generic/Sniffs/WhiteSpace/DisallowTabIndentSniff.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,8 @@ public function process(File $phpcsFile, $stackPtr)
7676
T_DOC_COMMENT_WHITESPACE => true,
7777
T_DOC_COMMENT_STRING => true,
7878
T_COMMENT => true,
79+
T_END_HEREDOC => true,
80+
T_END_NOWDOC => true,
7981
];
8082

8183
for ($i = 0; $i < $phpcsFile->numTokens; $i++) {
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
<?php
2+
3+
$heredoc = <<<"END"
4+
a
5+
b
6+
c
7+
END;
8+
9+
$nowdoc = <<<'END'
10+
a
11+
b
12+
c
13+
END;
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
<?php
2+
3+
$heredoc = <<<"END"
4+
a
5+
b
6+
c
7+
END;
8+
9+
$nowdoc = <<<'END'
10+
a
11+
b
12+
c
13+
END;

src/Standards/Generic/Tests/WhiteSpace/DisallowTabIndentUnitTest.php

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public function getErrorList($testFile='')
8383
92 => 1,
8484
93 => 1,
8585
];
86-
break;
86+
8787
case 'DisallowTabIndentUnitTest.2.inc':
8888
return [
8989
6 => 1,
@@ -96,23 +96,33 @@ public function getErrorList($testFile='')
9696
13 => 1,
9797
19 => 1,
9898
];
99-
break;
99+
100+
case 'DisallowTabIndentUnitTest.3.inc':
101+
if (\PHP_VERSION_ID >= 70300) {
102+
return [
103+
7 => 1,
104+
13 => 1,
105+
];
106+
}
107+
108+
// PHP 7.2 or lower: PHP version which doesn't support flexible heredocs/nowdocs yet.
109+
return [];
110+
100111
case 'DisallowTabIndentUnitTest.js':
101112
return [
102113
3 => 1,
103114
5 => 1,
104115
6 => 1,
105116
];
106-
break;
117+
107118
case 'DisallowTabIndentUnitTest.css':
108119
return [
109120
1 => 1,
110121
2 => 1,
111122
];
112-
break;
123+
113124
default:
114125
return [];
115-
break;
116126
}//end switch
117127

118128
}//end getErrorList()

0 commit comments

Comments
 (0)