Skip to content

Commit 3fb8a2d

Browse files
committed
Generic/DisallowSpaceIndent: check files using only short open echo tag
Includes unit tests.
1 parent ae4f33b commit 3fb8a2d

File tree

5 files changed

+55
-1
lines changed

5 files changed

+55
-1
lines changed

package.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -739,6 +739,8 @@ http://pear.php.net/dtd/package-2.0.xsd">
739739
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.1.inc.fixed" role="test" />
740740
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.2.inc" role="test" />
741741
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.2.inc.fixed" role="test" />
742+
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.3.inc" role="test" />
743+
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.3.inc.fixed" role="test" />
742744
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.js" role="test" />
743745
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.js.fixed" role="test" />
744746
<file baseinstalldir="PHP/CodeSniffer" name="DisallowSpaceIndentUnitTest.php" role="test" />

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,10 @@ class DisallowSpaceIndentSniff implements Sniff
4141
*/
4242
public function register()
4343
{
44-
return [T_OPEN_TAG];
44+
return [
45+
T_OPEN_TAG,
46+
T_OPEN_TAG_WITH_ECHO,
47+
];
4548

4649
}//end register()
4750

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
<?=
2+
"$hello $there";
3+
?>
4+
<html>
5+
<head>
6+
<title>Foo</title>
7+
</head>
8+
<body>
9+
<div>
10+
<div>
11+
<div>
12+
</div>
13+
</div>
14+
</div>
15+
</body>
16+
</html>
17+
18+
<?=
19+
"$hello $there";
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
<?=
2+
"$hello $there";
3+
?>
4+
<html>
5+
<head>
6+
<title>Foo</title>
7+
</head>
8+
<body>
9+
<div>
10+
<div>
11+
<div>
12+
</div>
13+
</div>
14+
</div>
15+
</body>
16+
</html>
17+
18+
<?=
19+
"$hello $there";

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

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,17 @@ public function getErrorList($testFile='DisallowSpaceIndentUnitTest.1.inc')
8686
118 => 1,
8787
];
8888
break;
89+
case 'DisallowSpaceIndentUnitTest.3.inc':
90+
return [
91+
2 => 1,
92+
5 => 1,
93+
10 => 1,
94+
12 => 1,
95+
13 => 1,
96+
14 => 1,
97+
15 => 1,
98+
];
99+
break;
89100
case 'DisallowSpaceIndentUnitTest.js':
90101
return [3 => 1];
91102
break;

0 commit comments

Comments
 (0)