Skip to content

Commit b4ad583

Browse files
committed
Squiz/ForLoopDeclaration: add missing fixed files
The sniff contains fixers, but didn't have `fixed` files.
1 parent b53f64e commit b4ad583

File tree

3 files changed

+90
-0
lines changed

3 files changed

+90
-0
lines changed

package.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1457,7 +1457,9 @@ http://pear.php.net/dtd/package-2.0.xsd">
14571457
<file baseinstalldir="PHP/CodeSniffer" name="ForEachLoopDeclarationUnitTest.inc.fixed" role="test" />
14581458
<file baseinstalldir="PHP/CodeSniffer" name="ForEachLoopDeclarationUnitTest.php" role="test" />
14591459
<file baseinstalldir="PHP/CodeSniffer" name="ForLoopDeclarationUnitTest.inc" role="test" />
1460+
<file baseinstalldir="PHP/CodeSniffer" name="ForLoopDeclarationUnitTest.inc.fixed" role="test" />
14601461
<file baseinstalldir="PHP/CodeSniffer" name="ForLoopDeclarationUnitTest.js" role="test" />
1462+
<file baseinstalldir="PHP/CodeSniffer" name="ForLoopDeclarationUnitTest.js.fixed" role="test" />
14611463
<file baseinstalldir="PHP/CodeSniffer" name="ForLoopDeclarationUnitTest.php" role="test" />
14621464
<file baseinstalldir="PHP/CodeSniffer" name="InlineIfDeclarationUnitTest.inc" role="test" />
14631465
<file baseinstalldir="PHP/CodeSniffer" name="InlineIfDeclarationUnitTest.inc.fixed" role="test" />
Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
<?php
2+
3+
// Valid.
4+
for ($i = 0; $i < 10; $i++) {
5+
}
6+
7+
// Invalid.
8+
for ($i = 0; $i < 10; $i++) {
9+
}
10+
11+
for ($i = 0; $i < 10; $i++) {
12+
}
13+
14+
for ($i = 0; $i < 10; $i++) {
15+
}
16+
17+
for ($i = 0; $i < 10; $i++) {
18+
}
19+
20+
// The works.
21+
for ($i = 0; $i < 10; $i++) {
22+
}
23+
24+
for ($i = 0; $i < 10;) {
25+
}
26+
27+
for ($i = 0; $i < 10;) {
28+
}
29+
30+
for ($i = 0;; $i++) {
31+
}
32+
for ($i = 0;; $i++) {
33+
}
34+
35+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesAfterOpen 1
36+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesBeforeClose 1
37+
for ( $i = 0; $i < 10; $i++ ) {}
38+
for ( $i = 0; $i < 10; $i++ ) {}
39+
for ( $i = 0; $i < 10; $i++ ) {}
40+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesAfterOpen 0
41+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesBeforeClose 0
Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
// Valid.
2+
for (var i = 0; i < 10; i++) {
3+
}
4+
5+
// Invalid.
6+
for (i = 0; i < 10; i++) {
7+
}
8+
9+
for (i = 0; i < 10; i++) {
10+
}
11+
12+
for (var i = 0; i < 10; i++) {
13+
}
14+
15+
for (i = 0; i < 10; i++) {
16+
}
17+
18+
// The works.
19+
for (var i = 0; i < 10; i++) {
20+
}
21+
22+
this.formats = {};
23+
dfx.inherits('ContentFormat', 'Widget');
24+
25+
for (var widgetid in this.loadedContents) {
26+
if (dfx.isset(widget) === true) {
27+
widget.loadAutoSaveCWidgetStore.setData('activeScreen', null);widget.getContents(this.loadedContents[widgetid], function() {self.widgetLoaded(widget.id);});
28+
}
29+
}
30+
31+
for (var i = 0; i < 10;) {
32+
}
33+
for (var i = 0; i < 10;) {
34+
}
35+
36+
for (var i = 0;; i++) {
37+
}
38+
for (var i = 0;; i++) {
39+
}
40+
41+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesAfterOpen 1
42+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesBeforeClose 1
43+
for ( var i = 0; i < 10; i++ ) {}
44+
for ( var i = 0; i < 10; i++ ) {}
45+
for ( var i = 0; i < 10; i++ ) {}
46+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesAfterOpen 0
47+
// phpcs:set Squiz.ControlStructures.ForLoopDeclaration requiredSpacesBeforeClose 0

0 commit comments

Comments
 (0)