Completed
Branch BUG/required-message-fields (8f9492)
by
unknown
10:53 queued 20s
created
src/Standards/Squiz/Tests/Classes/DuplicatePropertyUnitTest.php 1 patch
Indentation   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -15,38 +15,38 @@
 block discarded – undo
15 15
 {
16 16
 
17 17
 
18
-    /**
19
-     * Returns the lines where errors should occur.
20
-     *
21
-     * The key of the array should represent the line number and the value
22
-     * should represent the number of errors that should occur on that line.
23
-     *
24
-     * @return array<int, int>
25
-     */
26
-    public function getErrorList()
27
-    {
28
-        return [
29
-            4  => 1,
30
-            8  => 1,
31
-            28 => 1,
32
-        ];
33
-
34
-    }//end getErrorList()
35
-
36
-
37
-    /**
38
-     * Returns the lines where warnings should occur.
39
-     *
40
-     * The key of the array should represent the line number and the value
41
-     * should represent the number of warnings that should occur on that line.
42
-     *
43
-     * @return array<int, int>
44
-     */
45
-    public function getWarningList()
46
-    {
47
-        return [];
48
-
49
-    }//end getWarningList()
18
+	/**
19
+	 * Returns the lines where errors should occur.
20
+	 *
21
+	 * The key of the array should represent the line number and the value
22
+	 * should represent the number of errors that should occur on that line.
23
+	 *
24
+	 * @return array<int, int>
25
+	 */
26
+	public function getErrorList()
27
+	{
28
+		return [
29
+			4  => 1,
30
+			8  => 1,
31
+			28 => 1,
32
+		];
33
+
34
+	}//end getErrorList()
35
+
36
+
37
+	/**
38
+	 * Returns the lines where warnings should occur.
39
+	 *
40
+	 * The key of the array should represent the line number and the value
41
+	 * should represent the number of warnings that should occur on that line.
42
+	 *
43
+	 * @return array<int, int>
44
+	 */
45
+	public function getWarningList()
46
+	{
47
+		return [];
48
+
49
+	}//end getWarningList()
50 50
 
51 51
 
52 52
 }//end class
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Classes/ClassDeclarationUnitTest.inc 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 {
57 57
 }//end class
58 58
 
59
-class ClassOne implements ClassFour   ,ClassFive,  ClassSix
59
+class ClassOne implements ClassFour, ClassFive, ClassSix
60 60
 {
61 61
 }//end class
62 62
 
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 
83 83
 } // phpcs:enable Standard.Category.Sniff
84 84
 
85
-class File implements \Zend_Auth_Storage_Interface,\Zend_Auth_Storage,  \Zend_Foo
85
+class File implements \Zend_Auth_Storage_Interface, \Zend_Auth_Storage, \Zend_Foo
86 86
 {
87 87
 }
88 88
 
@@ -107,4 +107,4 @@  discard block
 block discarded – undo
107 107
 class IncorrectCodeBeforeClosingBrace
108 108
 {
109 109
 
110
-echo phpinfo();}
110
+echo phpinfo(); }
Please login to merge, or discard this patch.
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -24,10 +24,10 @@  discard block
 block discarded – undo
24 24
 class IncorrectBracePlacementWithImplements implements correctClassDeclaration {}
25 25
 
26 26
 
27
-    class IncorrectIndentedClass
28
-    {
27
+	class IncorrectIndentedClass
28
+	{
29 29
 
30
-    }//end class
30
+	}//end class
31 31
 
32 32
 
33 33
 // Incorrect code placement for opening/closing brace.
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
 
48 48
 echo phpinfo(); }
49 49
 
50
-    class IncorrectIndentedClass
50
+	class IncorrectIndentedClass
51 51
 {
52 52
 
53 53
 }
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 
97 97
 class MyClass2
98 98
 {
99
-    var $x;
99
+	var $x;
100 100
 }
101 101
 
102 102
 
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Classes/SelfMemberReferenceUnitTest.inc 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 
14 14
     public function SelfMemberReferenceUnitTestExample()
15 15
     {
16
-        $testResults =& $this->testResults;
16
+        $testResults = & $this->testResults;
17 17
 
18 18
 
19 19
         // Correct call to self.
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
                         MyClass::test($value);
58 58
                     };
59 59
 
60
-        $array = array(1,2,3);
60
+        $array = array(1, 2, 3);
61 61
         array_walk($array, $callback);
62 62
     }
63 63
 }
Please login to merge, or discard this patch.
Indentation   +92 added lines, -92 removed lines patch added patch discarded remove patch
@@ -5,54 +5,54 @@  discard block
 block discarded – undo
5 5
 {
6 6
 
7 7
 
8
-    private $testCount = 0;
8
+	private $testCount = 0;
9 9
 
10 10
 
11
-    private $testResults = array();
11
+	private $testResults = array();
12 12
 
13 13
 
14
-    public function SelfMemberReferenceUnitTestExample()
15
-    {
16
-        $testResults =& $this->testResults;
14
+	public function SelfMemberReferenceUnitTestExample()
15
+	{
16
+		$testResults =& $this->testResults;
17 17
 
18 18
 
19
-        // Correct call to self.
20
-        $testResults[] = self::selfMemberReferenceUnitTestFunction();
21
-        $testResults[] = parent::selfMemberReferenceUnitTestFunction();
19
+		// Correct call to self.
20
+		$testResults[] = self::selfMemberReferenceUnitTestFunction();
21
+		$testResults[] = parent::selfMemberReferenceUnitTestFunction();
22 22
 
23
-        // Incorrect case.
24
-        $testResults[] = Self::selfMemberReferenceUnitTestFunction();
25
-        $testResults[] = SELF::selfMemberReferenceUnitTestFunction();
26
-        $testResults[] = SelfMemberReferenceUnitTestExample::selfMemberReferenceUnitTestFunction();
23
+		// Incorrect case.
24
+		$testResults[] = Self::selfMemberReferenceUnitTestFunction();
25
+		$testResults[] = SELF::selfMemberReferenceUnitTestFunction();
26
+		$testResults[] = SelfMemberReferenceUnitTestExample::selfMemberReferenceUnitTestFunction();
27 27
 
28 28
 
29
-        // Incorrect spacing.
30
-        $testResults[] = self ::selfMemberReferenceUnitTestFunction();
31
-        $testResults[] = self::  selfMemberReferenceUnitTestFunction();
32
-        $testResults[] = self  :: selfMemberReferenceUnitTestFunction();
29
+		// Incorrect spacing.
30
+		$testResults[] = self ::selfMemberReferenceUnitTestFunction();
31
+		$testResults[] = self::  selfMemberReferenceUnitTestFunction();
32
+		$testResults[] = self  :: selfMemberReferenceUnitTestFunction();
33 33
 
34
-        // Remove ALL the newlines
35
-        $testResults[] = self
34
+		// Remove ALL the newlines
35
+		$testResults[] = self
36 36
 
37 37
 
38 38
 
39 39
 
40
-            ::
40
+			::
41 41
 
42 42
 
43 43
 
44 44
 
45
-            selfMemberReferenceUnitTestFunction();
45
+			selfMemberReferenceUnitTestFunction();
46 46
 
47
-    }
47
+	}
48 48
 
49 49
 
50
-    function selfMemberReferenceUnitTestFunction()
51
-    {
52
-        $this->testCount = $this->testCount + 1;
53
-        return $this->testCount;
50
+	function selfMemberReferenceUnitTestFunction()
51
+	{
52
+		$this->testCount = $this->testCount + 1;
53
+		return $this->testCount;
54 54
 
55
-    }
55
+	}
56 56
 
57 57
 
58 58
 }
@@ -60,115 +60,115 @@  discard block
 block discarded – undo
60 60
 
61 61
 class MyClass {
62 62
 
63
-    public static function test($value) {
64
-        echo "$value\n";
65
-    }
63
+	public static function test($value) {
64
+		echo "$value\n";
65
+	}
66 66
 
67
-    public static function walk() {
68
-        $callback = function($value, $key) {
69
-                        // This is valid because you can't use self:: in a closure.
70
-                        MyClass::test($value);
71
-                    };
67
+	public static function walk() {
68
+		$callback = function($value, $key) {
69
+						// This is valid because you can't use self:: in a closure.
70
+						MyClass::test($value);
71
+					};
72 72
 
73
-        $array = array(1,2,3);
74
-        array_walk($array, $callback);
75
-    }
73
+		$array = array(1,2,3);
74
+		array_walk($array, $callback);
75
+	}
76 76
 }
77 77
 
78 78
 MyClass::walk();
79 79
 
80 80
 class Controller
81 81
 {
82
-    public function Action()
83
-    {
84
-        Doctrine\Common\Util\Debug::dump();
85
-    }
82
+	public function Action()
83
+	{
84
+		Doctrine\Common\Util\Debug::dump();
85
+	}
86 86
 }
87 87
 
88 88
 class Foo
89 89
 {
90
-    public static function bar()
91
-    {
92
-        \Foo::baz();
93
-    }
90
+	public static function bar()
91
+	{
92
+		\Foo::baz();
93
+	}
94 94
 }
95 95
 
96 96
 namespace TYPO3\CMS\Reports;
97 97
 
98 98
 class Status {
99
-    const NOTICE = -2;
100
-    const INFO = -1;
101
-    const OK = 0;
102
-    const WARNING = 1;
103
-    const ERROR = 2;
99
+	const NOTICE = -2;
100
+	const INFO = -1;
101
+	const OK = 0;
102
+	const WARNING = 1;
103
+	const ERROR = 2;
104 104
 }
105 105
 
106 106
 namespace TYPO3\CMS\Reports\Report\Status;
107 107
 
108 108
 class Status implements \TYPO3\CMS\Reports\ReportInterface {
109
-    public function getHighestSeverity(array $statusCollection) {
110
-        $highestSeverity = \TYPO3\CMS\Reports\Status::NOTICE;
111
-    }
109
+	public function getHighestSeverity(array $statusCollection) {
110
+		$highestSeverity = \TYPO3\CMS\Reports\Status::NOTICE;
111
+	}
112 112
 }
113 113
 
114 114
 namespace Foo;
115 115
 
116 116
 class Bar {
117 117
 
118
-    function myFunction()
119
-    {
120
-        \Foo\Whatever::something();
121
-        \Foo\Bar::something();
122
-    }
118
+	function myFunction()
119
+	{
120
+		\Foo\Whatever::something();
121
+		\Foo\Bar::something();
122
+	}
123 123
 }
124 124
 
125 125
 namespace Foo\Bar;
126 126
 
127 127
 class Baz {
128 128
 
129
-    function myFunction()
130
-    {
131
-        \Foo\Bar\Whatever::something();
132
-        \Foo\Bar\Baz::something();
133
-    }
129
+	function myFunction()
130
+	{
131
+		\Foo\Bar\Whatever::something();
132
+		\Foo\Bar\Baz::something();
133
+	}
134 134
 }
135 135
 
136 136
 class Nested_Anon_Class {
137
-    public function getAnonymousClass() {
138
-        // Spacing/comments should not cause false negatives for the NotUsed error.
139
-        Nested_Anon_Class      ::       $prop;
140
-        Nested_Anon_Class
141
-        /* some comment */
142
-
143
-        ::
144
-
145
-        // phpcs:ignore Standard.Category.SniffName -- for reasons.
146
-        Bar();
147
-
148
-        // Anonymous class is a different scope.
149
-        return new class() {
150
-            public function nested_function() {
151
-                Nested_Anon_Class::$prop;
152
-                Nested_Anon_Class::BAR;
153
-            }
154
-        };
155
-    }
137
+	public function getAnonymousClass() {
138
+		// Spacing/comments should not cause false negatives for the NotUsed error.
139
+		Nested_Anon_Class      ::       $prop;
140
+		Nested_Anon_Class
141
+		/* some comment */
142
+
143
+		::
144
+
145
+		// phpcs:ignore Standard.Category.SniffName -- for reasons.
146
+		Bar();
147
+
148
+		// Anonymous class is a different scope.
149
+		return new class() {
150
+			public function nested_function() {
151
+				Nested_Anon_Class::$prop;
152
+				Nested_Anon_Class::BAR;
153
+			}
154
+		};
155
+	}
156 156
 }
157 157
 
158 158
 // Test dealing with scoped namespaces.
159 159
 namespace Foo\Baz {
160
-    class BarFoo {
161
-        public function foo() {
162
-            echo Foo\Baz\BarFoo::$prop;
163
-        }
164
-    }
160
+	class BarFoo {
161
+		public function foo() {
162
+			echo Foo\Baz\BarFoo::$prop;
163
+		}
164
+	}
165 165
 }
166 166
 
167 167
 // Prevent false negative when namespace has whitespace/comments.
168 168
 namespace Foo /*comment*/ \ Bah {
169
-    class BarFoo {
170
-        public function foo() {
171
-            echo Foo \ /*comment*/ Bah\BarFoo::$prop;
172
-        }
173
-    }
169
+	class BarFoo {
170
+		public function foo() {
171
+			echo Foo \ /*comment*/ Bah\BarFoo::$prop;
172
+		}
173
+	}
174 174
 }
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Operators/ComparisonOperatorUsageUnitTest.php 3 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@
 block discarded – undo
25 25
      *
26 26
      * @return array<int, int>
27 27
      */
28
-    public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
28
+    public function getErrorList($testFile = 'ComparisonOperatorUsageUnitTest.inc')
29 29
     {
30 30
         switch ($testFile) {
31 31
         case 'ComparisonOperatorUsageUnitTest.inc':
Please login to merge, or discard this patch.
Indentation   +77 added lines, -77 removed lines patch added patch discarded remove patch
@@ -15,87 +15,87 @@
 block discarded – undo
15 15
 {
16 16
 
17 17
 
18
-    /**
19
-     * Returns the lines where errors should occur.
20
-     *
21
-     * The key of the array should represent the line number and the value
22
-     * should represent the number of errors that should occur on that line.
23
-     *
24
-     * @param string $testFile The name of the file being tested.
25
-     *
26
-     * @return array<int, int>
27
-     */
28
-    public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
29
-    {
30
-        switch ($testFile) {
31
-        case 'ComparisonOperatorUsageUnitTest.inc':
32
-            return [
33
-                6   => 1,
34
-                7   => 1,
35
-                10  => 1,
36
-                11  => 1,
37
-                18  => 1,
38
-                19  => 1,
39
-                22  => 1,
40
-                23  => 1,
41
-                29  => 2,
42
-                32  => 2,
43
-                38  => 4,
44
-                47  => 2,
45
-                69  => 1,
46
-                72  => 1,
47
-                75  => 1,
48
-                78  => 1,
49
-                80  => 1,
50
-                82  => 1,
51
-                83  => 1,
52
-                89  => 1,
53
-                92  => 1,
54
-                100 => 1,
55
-                106 => 1,
56
-                112 => 1,
57
-                123 => 1,
58
-                127 => 1,
59
-                131 => 1,
60
-                135 => 1,
61
-            ];
62
-            break;
63
-        case 'ComparisonOperatorUsageUnitTest.js':
64
-            return [
65
-                5  => 1,
66
-                6  => 1,
67
-                17 => 1,
68
-                18 => 1,
69
-                28 => 2,
70
-                40 => 1,
71
-                47 => 1,
72
-                52 => 1,
73
-                63 => 1,
74
-                67 => 1,
75
-                71 => 1,
76
-            ];
77
-            break;
78
-        default:
79
-            return [];
80
-            break;
81
-        }//end switch
18
+	/**
19
+	 * Returns the lines where errors should occur.
20
+	 *
21
+	 * The key of the array should represent the line number and the value
22
+	 * should represent the number of errors that should occur on that line.
23
+	 *
24
+	 * @param string $testFile The name of the file being tested.
25
+	 *
26
+	 * @return array<int, int>
27
+	 */
28
+	public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
29
+	{
30
+		switch ($testFile) {
31
+		case 'ComparisonOperatorUsageUnitTest.inc':
32
+			return [
33
+				6   => 1,
34
+				7   => 1,
35
+				10  => 1,
36
+				11  => 1,
37
+				18  => 1,
38
+				19  => 1,
39
+				22  => 1,
40
+				23  => 1,
41
+				29  => 2,
42
+				32  => 2,
43
+				38  => 4,
44
+				47  => 2,
45
+				69  => 1,
46
+				72  => 1,
47
+				75  => 1,
48
+				78  => 1,
49
+				80  => 1,
50
+				82  => 1,
51
+				83  => 1,
52
+				89  => 1,
53
+				92  => 1,
54
+				100 => 1,
55
+				106 => 1,
56
+				112 => 1,
57
+				123 => 1,
58
+				127 => 1,
59
+				131 => 1,
60
+				135 => 1,
61
+			];
62
+			break;
63
+		case 'ComparisonOperatorUsageUnitTest.js':
64
+			return [
65
+				5  => 1,
66
+				6  => 1,
67
+				17 => 1,
68
+				18 => 1,
69
+				28 => 2,
70
+				40 => 1,
71
+				47 => 1,
72
+				52 => 1,
73
+				63 => 1,
74
+				67 => 1,
75
+				71 => 1,
76
+			];
77
+			break;
78
+		default:
79
+			return [];
80
+			break;
81
+		}//end switch
82 82
 
83
-    }//end getErrorList()
83
+	}//end getErrorList()
84 84
 
85 85
 
86
-    /**
87
-     * Returns the lines where warnings should occur.
88
-     *
89
-     * The key of the array should represent the line number and the value
90
-     * should represent the number of warnings that should occur on that line.
91
-     *
92
-     * @return array<int, int>
93
-     */
94
-    public function getWarningList()
95
-    {
96
-        return [];
86
+	/**
87
+	 * Returns the lines where warnings should occur.
88
+	 *
89
+	 * The key of the array should represent the line number and the value
90
+	 * should represent the number of warnings that should occur on that line.
91
+	 *
92
+	 * @return array<int, int>
93
+	 */
94
+	public function getWarningList()
95
+	{
96
+		return [];
97 97
 
98
-    }//end getWarningList()
98
+	}//end getWarningList()
99 99
 
100 100
 
101 101
 }//end class
Please login to merge, or discard this patch.
Switch Indentation   +50 added lines, -50 removed lines patch added patch discarded remove patch
@@ -28,56 +28,56 @@
 block discarded – undo
28 28
     public function getErrorList($testFile='ComparisonOperatorUsageUnitTest.inc')
29 29
     {
30 30
         switch ($testFile) {
31
-        case 'ComparisonOperatorUsageUnitTest.inc':
32
-            return [
33
-                6   => 1,
34
-                7   => 1,
35
-                10  => 1,
36
-                11  => 1,
37
-                18  => 1,
38
-                19  => 1,
39
-                22  => 1,
40
-                23  => 1,
41
-                29  => 2,
42
-                32  => 2,
43
-                38  => 4,
44
-                47  => 2,
45
-                69  => 1,
46
-                72  => 1,
47
-                75  => 1,
48
-                78  => 1,
49
-                80  => 1,
50
-                82  => 1,
51
-                83  => 1,
52
-                89  => 1,
53
-                92  => 1,
54
-                100 => 1,
55
-                106 => 1,
56
-                112 => 1,
57
-                123 => 1,
58
-                127 => 1,
59
-                131 => 1,
60
-                135 => 1,
61
-            ];
62
-            break;
63
-        case 'ComparisonOperatorUsageUnitTest.js':
64
-            return [
65
-                5  => 1,
66
-                6  => 1,
67
-                17 => 1,
68
-                18 => 1,
69
-                28 => 2,
70
-                40 => 1,
71
-                47 => 1,
72
-                52 => 1,
73
-                63 => 1,
74
-                67 => 1,
75
-                71 => 1,
76
-            ];
77
-            break;
78
-        default:
79
-            return [];
80
-            break;
31
+        	case 'ComparisonOperatorUsageUnitTest.inc':
32
+            	return [
33
+                	6   => 1,
34
+                	7   => 1,
35
+                	10  => 1,
36
+                	11  => 1,
37
+                	18  => 1,
38
+                	19  => 1,
39
+                	22  => 1,
40
+                	23  => 1,
41
+                	29  => 2,
42
+                	32  => 2,
43
+                	38  => 4,
44
+                	47  => 2,
45
+                	69  => 1,
46
+                	72  => 1,
47
+                	75  => 1,
48
+                	78  => 1,
49
+                	80  => 1,
50
+                	82  => 1,
51
+                	83  => 1,
52
+                	89  => 1,
53
+                	92  => 1,
54
+                	100 => 1,
55
+                	106 => 1,
56
+                	112 => 1,
57
+                	123 => 1,
58
+                	127 => 1,
59
+                	131 => 1,
60
+                	135 => 1,
61
+            	];
62
+            	break;
63
+        	case 'ComparisonOperatorUsageUnitTest.js':
64
+            	return [
65
+                	5  => 1,
66
+                	6  => 1,
67
+                	17 => 1,
68
+                	18 => 1,
69
+                	28 => 2,
70
+                	40 => 1,
71
+                	47 => 1,
72
+                	52 => 1,
73
+                	63 => 1,
74
+                	67 => 1,
75
+                	71 => 1,
76
+            	];
77
+            	break;
78
+        	default:
79
+            	return [];
80
+            	break;
81 81
         }//end switch
82 82
 
83 83
     }//end getErrorList()
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Operators/ValidLogicalOperatorsUnitTest.php 1 patch
Indentation   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -15,38 +15,38 @@
 block discarded – undo
15 15
 {
16 16
 
17 17
 
18
-    /**
19
-     * Returns the lines where errors should occur.
20
-     *
21
-     * The key of the array should represent the line number and the value
22
-     * should represent the number of errors that should occur on that line.
23
-     *
24
-     * @return array<int, int>
25
-     */
26
-    public function getErrorList()
27
-    {
28
-        return [
29
-            5  => 1,
30
-            11 => 1,
31
-            17 => 2,
32
-        ];
33
-
34
-    }//end getErrorList()
35
-
36
-
37
-    /**
38
-     * Returns the lines where warnings should occur.
39
-     *
40
-     * The key of the array should represent the line number and the value
41
-     * should represent the number of warnings that should occur on that line.
42
-     *
43
-     * @return array<int, int>
44
-     */
45
-    public function getWarningList()
46
-    {
47
-        return [];
48
-
49
-    }//end getWarningList()
18
+	/**
19
+	 * Returns the lines where errors should occur.
20
+	 *
21
+	 * The key of the array should represent the line number and the value
22
+	 * should represent the number of errors that should occur on that line.
23
+	 *
24
+	 * @return array<int, int>
25
+	 */
26
+	public function getErrorList()
27
+	{
28
+		return [
29
+			5  => 1,
30
+			11 => 1,
31
+			17 => 2,
32
+		];
33
+
34
+	}//end getErrorList()
35
+
36
+
37
+	/**
38
+	 * Returns the lines where warnings should occur.
39
+	 *
40
+	 * The key of the array should represent the line number and the value
41
+	 * should represent the number of warnings that should occur on that line.
42
+	 *
43
+	 * @return array<int, int>
44
+	 */
45
+	public function getWarningList()
46
+	{
47
+		return [];
48
+
49
+	}//end getWarningList()
50 50
 
51 51
 
52 52
 }//end class
Please login to merge, or discard this patch.
src/Standards/Squiz/Tests/Operators/ComparisonOperatorUsageUnitTest.inc 2 patches
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -52,8 +52,8 @@  discard block
 block discarded – undo
52 52
 
53 53
 // Without brackets around inline IF condition.
54 54
 $var1 === TRUE
55
-    ? $var2 = 0
56
-    : $var2 = 1;
55
+	? $var2 = 0
56
+	: $var2 = 1;
57 57
 
58 58
 $var1 === TRUE ? $var2 = 0 : $var2 = 1;
59 59
 ?>
@@ -61,13 +61,13 @@  discard block
 block discarded – undo
61 61
 $var1 === TRUE ? $var2 = 0 : $var2 = 1;
62 62
 
63 63
 if ($var2 === TRUE) {
64
-    $var1 === TRUE ? $var2 = 0 : $var2 = 1;
64
+	$var1 === TRUE ? $var2 = 0 : $var2 = 1;
65 65
 }
66 66
 $var1 === TRUE ? $var2 = 0 : $var2 = 1;
67 67
 
68 68
 $var1
69
-    ? $var2 = 0
70
-    : $var2 = 1;
69
+	? $var2 = 0
70
+	: $var2 = 1;
71 71
 
72 72
 $var1 ? $var2 = 0 : $var2 = 1;
73 73
 
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 $var1 ? $var2 = 0 : $var2 = 1;
76 76
 
77 77
 if ($var2 === TRUE) {
78
-    $var1 ? $var2 = 0 : $var2 = 1;
78
+	$var1 ? $var2 = 0 : $var2 = 1;
79 79
 }
80 80
 $var1 ? $var2 = 0 : $var2 = 1;
81 81
 
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 }
9 9
 
10 10
 if ($value) {
11
-} else if (!$value) {
11
+} else if ( ! $value) {
12 12
 }
13 13
 
14 14
 if (is_array($array) === TRUE) {
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
 }
21 21
 
22 22
 if (is_array($array)) {
23
-} else if (!myFunction($value)) {
23
+} else if ( ! myFunction($value)) {
24 24
 }
25 25
 
26 26
 if ($value === TRUE || $other === FALSE) {
@@ -29,13 +29,13 @@  discard block
 block discarded – undo
29 29
 if ($value == TRUE || $other == FALSE) {
30 30
 }
31 31
 
32
-if ($value || !$other) {
32
+if ($value || ! $other) {
33 33
 }
34 34
 
35 35
 if ($one === TRUE || $two === TRUE || $three === FALSE || $four === TRUE) {
36 36
 }
37 37
 
38
-if ($one || $two || !$three || $four) {
38
+if ($one || $two || ! $three || $four) {
39 39
 }
40 40
 
41 41
 if ($var instanceof PHP_CodeSniffer) {
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
 $var1 ? $var2 = 0 : $var2 = 1;
81 81
 
82 82
 if ($value) {
83
-} elseif (!$value) {
83
+} elseif ( ! $value) {
84 84
 }
85 85
 
86 86
 if (false === ($parent instanceof Foo) && ($parent instanceof Bar) === false) {
@@ -97,11 +97,11 @@  discard block
 block discarded – undo
97 97
 
98 98
 do {
99 99
 
100
-} while ($var1);
100
+}while ($var1);
101 101
 
102 102
 do {
103 103
 
104
-} while ($var1 === TRUE);
104
+}while ($var1 === TRUE);
105 105
 
106 106
 for ($var1 = 10; $var1; $var1--) {
107 107
 }
Please login to merge, or discard this patch.
src/Standards/Squiz/Sniffs/NamingConventions/ValidFunctionNameSniff.php 1 patch
Indentation   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -17,38 +17,38 @@
 block discarded – undo
17 17
 {
18 18
 
19 19
 
20
-    /**
21
-     * Processes the tokens outside the scope.
22
-     *
23
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being processed.
24
-     * @param int                         $stackPtr  The position where this token was
25
-     *                                               found.
26
-     *
27
-     * @return void
28
-     */
29
-    protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
30
-    {
31
-        $functionName = $phpcsFile->getDeclarationName($stackPtr);
32
-        if ($functionName === null) {
33
-            return;
34
-        }
35
-
36
-        $errorData = [$functionName];
37
-
38
-        // Does this function claim to be magical?
39
-        if (preg_match('|^__[^_]|', $functionName) !== 0) {
40
-            $error = 'Function name "%s" is invalid; only PHP magic methods should be prefixed with a double underscore';
41
-            $phpcsFile->addError($error, $stackPtr, 'DoubleUnderscore', $errorData);
42
-
43
-            $functionName = ltrim($functionName, '_');
44
-        }
45
-
46
-        if (Common::isCamelCaps($functionName, false, true, false) === false) {
47
-            $error = 'Function name "%s" is not in camel caps format';
48
-            $phpcsFile->addError($error, $stackPtr, 'NotCamelCaps', $errorData);
49
-        }
50
-
51
-    }//end processTokenOutsideScope()
20
+	/**
21
+	 * Processes the tokens outside the scope.
22
+	 *
23
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being processed.
24
+	 * @param int                         $stackPtr  The position where this token was
25
+	 *                                               found.
26
+	 *
27
+	 * @return void
28
+	 */
29
+	protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
30
+	{
31
+		$functionName = $phpcsFile->getDeclarationName($stackPtr);
32
+		if ($functionName === null) {
33
+			return;
34
+		}
35
+
36
+		$errorData = [$functionName];
37
+
38
+		// Does this function claim to be magical?
39
+		if (preg_match('|^__[^_]|', $functionName) !== 0) {
40
+			$error = 'Function name "%s" is invalid; only PHP magic methods should be prefixed with a double underscore';
41
+			$phpcsFile->addError($error, $stackPtr, 'DoubleUnderscore', $errorData);
42
+
43
+			$functionName = ltrim($functionName, '_');
44
+		}
45
+
46
+		if (Common::isCamelCaps($functionName, false, true, false) === false) {
47
+			$error = 'Function name "%s" is not in camel caps format';
48
+			$phpcsFile->addError($error, $stackPtr, 'NotCamelCaps', $errorData);
49
+		}
50
+
51
+	}//end processTokenOutsideScope()
52 52
 
53 53
 
54 54
 }//end class
Please login to merge, or discard this patch.
src/Standards/Squiz/Sniffs/PHP/DisallowBooleanStatementSniff.php 1 patch
Indentation   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -17,43 +17,43 @@
 block discarded – undo
17 17
 {
18 18
 
19 19
 
20
-    /**
21
-     * Returns an array of tokens this test wants to listen for.
22
-     *
23
-     * @return array
24
-     */
25
-    public function register()
26
-    {
27
-        return Tokens::$booleanOperators;
28
-
29
-    }//end register()
30
-
31
-
32
-    /**
33
-     * Processes this test, when one of its tokens is encountered.
34
-     *
35
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
36
-     * @param int                         $stackPtr  The position of the current token
37
-     *                                               in the stack passed in $tokens.
38
-     *
39
-     * @return void
40
-     */
41
-    public function process(File $phpcsFile, $stackPtr)
42
-    {
43
-        $tokens = $phpcsFile->getTokens();
44
-        if (isset($tokens[$stackPtr]['nested_parenthesis']) === true) {
45
-            foreach ($tokens[$stackPtr]['nested_parenthesis'] as $open => $close) {
46
-                if (isset($tokens[$open]['parenthesis_owner']) === true) {
47
-                    // Any owner means we are not just a simple statement.
48
-                    return;
49
-                }
50
-            }
51
-        }
52
-
53
-        $error = 'Boolean operators are not allowed outside of control structure conditions';
54
-        $phpcsFile->addError($error, $stackPtr, 'Found');
55
-
56
-    }//end process()
20
+	/**
21
+	 * Returns an array of tokens this test wants to listen for.
22
+	 *
23
+	 * @return array
24
+	 */
25
+	public function register()
26
+	{
27
+		return Tokens::$booleanOperators;
28
+
29
+	}//end register()
30
+
31
+
32
+	/**
33
+	 * Processes this test, when one of its tokens is encountered.
34
+	 *
35
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
36
+	 * @param int                         $stackPtr  The position of the current token
37
+	 *                                               in the stack passed in $tokens.
38
+	 *
39
+	 * @return void
40
+	 */
41
+	public function process(File $phpcsFile, $stackPtr)
42
+	{
43
+		$tokens = $phpcsFile->getTokens();
44
+		if (isset($tokens[$stackPtr]['nested_parenthesis']) === true) {
45
+			foreach ($tokens[$stackPtr]['nested_parenthesis'] as $open => $close) {
46
+				if (isset($tokens[$open]['parenthesis_owner']) === true) {
47
+					// Any owner means we are not just a simple statement.
48
+					return;
49
+				}
50
+			}
51
+		}
52
+
53
+		$error = 'Boolean operators are not allowed outside of control structure conditions';
54
+		$phpcsFile->addError($error, $stackPtr, 'Found');
55
+
56
+	}//end process()
57 57
 
58 58
 
59 59
 }//end class
Please login to merge, or discard this patch.
php_codesniffer/src/Standards/Squiz/Sniffs/PHP/GlobalKeywordSniff.php 1 patch
Indentation   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -16,38 +16,38 @@
 block discarded – undo
16 16
 {
17 17
 
18 18
 
19
-    /**
20
-     * Returns an array of tokens this test wants to listen for.
21
-     *
22
-     * @return array
23
-     */
24
-    public function register()
25
-    {
26
-        return [T_GLOBAL];
27
-
28
-    }//end register()
29
-
30
-
31
-    /**
32
-     * Processes this test, when one of its tokens is encountered.
33
-     *
34
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
35
-     * @param int                         $stackPtr  The position of the current token in the
36
-     *                                               stack passed in $tokens.
37
-     *
38
-     * @return void
39
-     */
40
-    public function process(File $phpcsFile, $stackPtr)
41
-    {
42
-        $tokens = $phpcsFile->getTokens();
43
-
44
-        $nextVar = $tokens[$phpcsFile->findNext([T_VARIABLE], $stackPtr)];
45
-        $varName = str_replace('$', '', $nextVar['content']);
46
-        $error   = 'Use of the "global" keyword is forbidden; use "$GLOBALS[\'%s\']" instead';
47
-        $data    = [$varName];
48
-        $phpcsFile->addError($error, $stackPtr, 'NotAllowed', $data);
49
-
50
-    }//end process()
19
+	/**
20
+	 * Returns an array of tokens this test wants to listen for.
21
+	 *
22
+	 * @return array
23
+	 */
24
+	public function register()
25
+	{
26
+		return [T_GLOBAL];
27
+
28
+	}//end register()
29
+
30
+
31
+	/**
32
+	 * Processes this test, when one of its tokens is encountered.
33
+	 *
34
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
35
+	 * @param int                         $stackPtr  The position of the current token in the
36
+	 *                                               stack passed in $tokens.
37
+	 *
38
+	 * @return void
39
+	 */
40
+	public function process(File $phpcsFile, $stackPtr)
41
+	{
42
+		$tokens = $phpcsFile->getTokens();
43
+
44
+		$nextVar = $tokens[$phpcsFile->findNext([T_VARIABLE], $stackPtr)];
45
+		$varName = str_replace('$', '', $nextVar['content']);
46
+		$error   = 'Use of the "global" keyword is forbidden; use "$GLOBALS[\'%s\']" instead';
47
+		$data    = [$varName];
48
+		$phpcsFile->addError($error, $stackPtr, 'NotAllowed', $data);
49
+
50
+	}//end process()
51 51
 
52 52
 
53 53
 }//end class
Please login to merge, or discard this patch.