Multiple Index Definition |
Analyze, OneFile, All, Simple, CI-checks |
2 |
2 |
Minor |
25 % |
Null Or Boolean Arrays |
All, Analyze |
17 |
1 |
Minor |
0 % |
Randomly Sorted Arrays |
All, Analyze, Level 4, Suggestions |
2 |
1 |
Minor |
41 % |
String Initialization |
All, CompatibilityPHP71 |
22 |
17 |
Minor |
33 % |
Too Many Array Dimensions |
All, Analyze |
11 |
2 |
Minor |
0 % |
Abstract Or Implements |
All, Analyze, LintButWontExec |
1 |
1 |
Major |
4 % |
Ambiguous Static |
All, Analyze |
65 |
48 |
Minor |
64 % |
Ambiguous Visibilities |
All, Analyze |
55 |
36 |
Minor |
70 % |
Avoid Optional Properties |
All, Analyze |
51 |
22 |
Major |
83 % |
Cant Inherit Abstract Method |
All, CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP71, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56 |
1 |
1 |
Critical |
0 % |
Cant Instantiate Class |
All, Analyze |
2 |
2 |
Critical |
28 % |
Child Class Removes Typehint |
All, CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP71, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56, Typechecks |
5 |
2 |
Major |
1 % |
Clone With Non-Object |
All, Analyze, LintButWontExec |
4 |
2 |
Minor |
0 % |
Could Be Abstract Class |
All, Analyze, ClassReview |
7 |
7 |
Minor |
65 % |
Class Could Be Final |
All, Analyze, ClassReview |
154 |
153 |
Minor |
95 % |
Method Could Be Static |
All, Analyze, ClassReview |
527 |
130 |
Minor |
95 % |
Cyclic References |
All, Analyze, ClassReview |
1 |
1 |
Minor |
0 % |
Dependant Abstract Classes |
All, Analyze, ClassReview |
6 |
6 |
Minor |
0 % |
Different Argument Counts |
All, Analyze, ClassReview |
2 |
2 |
Minor |
0 % |
Don't Send $this In Constructor |
All, Analyze |
19 |
19 |
Minor |
39 % |
Don't Unset Properties |
All, Analyze, Top10, php-cs-fixable, CI-checks |
11 |
8 |
Major |
40 % |
Empty Classes |
Analyze, All, Simple |
1 |
1 |
Minor |
58 % |
Class Should Be Final By Ocramius |
All, Analyze |
1 |
1 |
Minor |
42 % |
Hidden Nullable |
All, Analyze, ClassReview |
41 |
22 |
Minor |
0 % |
Incompatible Signature Methods With Covariance |
All, Analyze |
5 |
5 |
Critical |
0 % |
Locally Unused Property |
Dead code, All, Simple |
8 |
7 |
Minor |
76 % |
Assign Default To Properties |
Analyze, ClearPHP, All, Simple, Level 2 |
17 |
13 |
Minor |
49 % |
Make Global A Property |
Analyze, All, Simple |
157 |
44 |
Minor |
33 % |
Method Signature Must Be Compatible |
All, Analyze, LintButWontExec |
2 |
2 |
Critical |
50 % |
No Public Access |
Analyze, All |
3 |
2 |
Minor |
60 % |
Forgotten Visibility |
Analyze, ClearPHP, All, Simple, Level 1, CI-checks |
1 |
1 |
Minor |
42 % |
Parent First |
All, Analyze, Suggestions |
27 |
27 |
Minor |
55 % |
Property Could Be Local |
All, Analyze, ClassReview |
74 |
44 |
Minor |
71 % |
Never Used Properties |
Analyze, All, Simple |
4 |
4 |
Minor |
72 % |
Property Used In One Method Only |
All, Analyze |
51 |
36 |
Minor |
84 % |
Redefined Default |
Analyze, All, Simple, CI-checks |
7 |
7 |
Major |
26 % |
Redefined Private Property |
All, Analyze |
1 |
1 |
Major |
18 % |
Could Use self |
Analyze, All, Simple, Suggestions, Level 3, ClassReview |
4 |
2 |
Minor |
57 % |
Should Use Local Class |
Analyze, ClearPHP, All, Simple |
536 |
115 |
Minor |
94 % |
Static Methods Called From Object |
Analyze, All, Simple, CI-checks |
6 |
4 |
Minor |
38 % |
Wrong Access Style to Property |
All, Analyze, Level 2, ClassReview, CI-checks |
113 |
17 |
Critical |
3 % |
Undefined Classes |
Analyze, All |
558 |
95 |
Major |
91 % |
Undefined Class Constants |
Analyze, All, CI-checks |
27 |
6 |
Major |
53 % |
Undefined Parent |
Analyze, All, Simple |
29 |
18 |
Minor |
63 % |
Undefined Properties |
Analyze, ClearPHP, All, Simple, CI-checks |
6 |
2 |
Major |
58 % |
Undefined static:: Or self:: |
Analyze, All, Simple |
1 |
1 |
Minor |
33 % |
Unresolved Catch |
Dead code, ClearPHP, All |
13 |
9 |
Major |
51 % |
Unresolved Classes |
Analyze, All |
105 |
29 |
Major |
86 % |
Unresolved Instanceof |
Analyze, Dead code, ClearPHP, All, Simple, Top10 |
192 |
51 |
Major |
48 % |
Unused Classes |
Dead code, All, Simple, Analyze |
1 |
1 |
Major |
72 % |
Unused Methods |
Dead code, All, Simple |
463 |
134 |
Minor |
95 % |
Unused Private Methods |
Dead code, OneFile, All, Simple |
2 |
2 |
Minor |
45 % |
Unused Protected Methods |
Dead code, All |
22 |
8 |
Major |
72 % |
Use Class Operator |
All, Analyze, Performances, Simple, CI-checks |
166 |
42 |
Minor |
74 % |
Use Instanceof |
Analyze, All, Simple, CI-checks |
22 |
13 |
Major |
63 % |
Used Once Property |
All, Analyze |
21 |
14 |
Minor |
72 % |
Useless Constructor |
Analyze, All, Simple, Level 3 |
2 |
2 |
Minor |
38 % |
Weak Typing |
All, Analyze |
5 |
4 |
Minor |
41 % |
Undefined Constants |
Analyze, All, Simple, CompatibilityPHP72, CI-checks |
266 |
160 |
Minor |
54 % |
Catch Undefined Variable |
All, Analyze |
7 |
2 |
Minor |
0 % |
Undefined Caught Exceptions |
Dead code, All |
2 |
2 |
None |
50 % |
Forgotten Thrown |
All, Analyze |
2 |
1 |
Major |
30 % |
Uncaught Exceptions |
Analyze, All |
105 |
24 |
Minor |
81 % |
Unthrown Exception |
Analyze, Dead code, ClearPHP, All, Simple |
1 |
1 |
Minor |
43 % |
Useless Catch |
All, Analyze, CI-checks |
1 |
1 |
Minor |
47 % |
Missing Include |
All, Analyze |
13 |
5 |
Critical |
61 % |
Aliases Usage |
Analyze, ClearPHP, OneFile, All, Simple, Level 1, CI-checks |
6 |
3 |
Minor |
42 % |
Use Named Boolean In Argument Definition |
All, Analyze |
66 |
33 |
Minor |
84 % |
Callback Needs Return |
All, Analyze, CI-checks |
1 |
1 |
Major |
51 % |
Could Make A Function |
All, Analyze, Suggestions |
2549 |
177 |
Minor |
75 % |
Empty Function |
Analyze, All, Simple |
6 |
6 |
Minor |
71 % |
Hardcoded Passwords |
Analyze, OneFile, Security, All, Simple, Level 3 |
31 |
9 |
Major |
61 % |
Insufficient Typehint |
All, Analyze, Typechecks |
1 |
1 |
Major |
24 % |
Mismatch Parameter Name |
All, Analyze, CompatibilityPHP80 |
4 |
3 |
Minor |
0 % |
Mismatched Default Arguments |
All, Analyze, Typechecks |
153 |
68 |
Minor |
33 % |
Mismatched Typehint |
All, Analyze, Typechecks |
20 |
14 |
Major |
5 % |
Never Used Parameter |
All, Analyze, Suggestions |
100 |
54 |
Minor |
47 % |
No Boolean As Default |
All, Analyze |
66 |
33 |
Minor |
84 % |
No Return Used |
All, Analyze, Suggestions, Level 4 |
13 |
9 |
Minor |
41 % |
Relay Function |
Analyze, All |
39 |
16 |
Major |
83 % |
Too Many Local Variables |
All, Analyze |
23 |
19 |
Minor |
65 % |
Undefined Functions |
Analyze, All, CI-checks |
4893 |
186 |
Major |
71 % |
Unused Arguments |
Analyze, All, Simple |
64 |
40 |
Major |
89 % |
Unused Functions |
Dead code, All, Simple |
16 |
7 |
Minor |
53 % |
Unused Returned Value |
All, Analyze, OneFile, Dead code, Simple |
20 |
10 |
Minor |
39 % |
Useless Type Check |
All, Dead code, OneFile |
1 |
1 |
Minor |
0 % |
Uses Default Values |
Analyze, All, Simple, CI-checks |
683 |
132 |
Minor |
97 % |
Using Deprecated Method |
All, Analyze |
1 |
1 |
Minor |
0 % |
Methods Without Return |
All, Analyze |
495 |
133 |
Minor |
0 % |
Wrong Number Of Arguments |
Analyze, OneFile, All, Simple, CI-checks |
24 |
14 |
Major |
38 % |
Wrong Optional Parameter |
Analyze, All, Simple, Level 1, CI-checks |
2 |
1 |
Minor |
38 % |
Wrong Type With Call |
All, Analyze, Typechecks, CI-checks |
68 |
29 |
Minor |
0 % |
Already Parents Interface |
Analyze, All, Level 3, Suggestions |
2 |
2 |
Minor |
17 % |
Empty Interfaces |
Analyze, All, Simple |
1 |
1 |
Minor |
38 % |
Interfaces Is Not Implemented |
All, Analyze, ClassReview, LintButWontExec, CI-checks |
5 |
5 |
Minor |
0 % |
Undefined Interfaces |
Analyze, All, CI-checks |
229 |
63 |
Major |
67 % |
Useless Interfaces |
Analyze, ClearPHP, All, Simple, ClassReview, Typechecks |
2 |
2 |
Minor |
57 % |
Hidden Use Expression |
Analyze, OneFile, All, Simple, CI-checks |
5 |
4 |
Minor |
24 % |
Multiple Alias Definitions |
Analyze, All, Simple, CI-checks |
6 |
6 |
Minor |
42 % |
Should Make Alias |
Analyze, OneFile, All, Simple, CI-checks |
132 |
46 |
Minor |
44 % |
Unresolved Use |
Analyze, ClearPHP, All, Simple |
121 |
116 |
Major |
65 % |
Unused Use |
Dead code, ClearPHP, All, Simple |
70 |
70 |
Major |
54 % |
No array_merge() In Loops |
Analyze, Performances, ClearPHP, All, Simple, Level 2, Top10, CI-checks |
6 |
4 |
Major |
55 % |
Logical To in_array |
All, Analyze, Level 2 |
10 |
8 |
Minor |
66 % |
Pre-increment |
Analyze, Performances, All, Simple, Level 4, CI-checks |
34 |
29 |
Minor |
74 % |
strpos() Too Much |
All, Analyze, CI-checks |
5 |
3 |
Minor |
46 % |
array_key_exists() Works On Arrays |
All, CompatibilityPHP74, Analyze |
23 |
12 |
Minor |
0 % |
Avoid mb_dectect_encoding() |
All, Analyze |
2 |
2 |
Minor |
0 % |
Use random_int() |
Analyze, Security, All, CompatibilityPHP71, Simple, Level 2, CI-checks |
16 |
8 |
Major |
59 % |
Detect Current Class |
All, Suggestions, CompatibilityPHP74 |
22 |
13 |
None |
47 % |
Don't Pollute Global Space |
All, Analyze |
283 |
177 |
Minor |
0 % |
Wrong Parameter Type |
Analyze, OneFile, All, Simple, CI-checks |
16 |
8 |
Major |
45 % |
Use === null |
Analyze, OneFile, All, RadwellCodes, Simple, php-cs-fixable, CI-checks |
24 |
11 |
Minor |
52 % |
List With Keys |
CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56, Appcontent, Appinfo, All |
1 |
1 |
Major |
32 % |
Logical Should Use Symbolic Operators |
Analyze, OneFile, ClearPHP, All, Simple, Suggestions, Level 2, Top10, php-cs-fixable, CI-checks |
138 |
138 |
Minor |
42 % |
No Class In Global |
All, Analyze, CI-checks |
247 |
177 |
Minor |
78 % |
PHP Keywords As Names |
Analyze, All, Simple |
253 |
39 |
Major |
89 % |
Scalar Are Not Arrays |
All, Analyze, CompatibilityPHP74, CI-checks |
4 |
2 |
Minor |
0 % |
Should Use Coalesce |
Analyze, All, Simple, Suggestions, Level 3, CI-checks |
77 |
32 |
Major |
71 % |
Too Many Native Calls |
All, Analyze |
2 |
2 |
Minor |
47 % |
Use Nullable Type |
All, Appinfo, CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56 |
162 |
69 |
Major |
21 % |
Avoid Using stdClass |
All, Analyze, OneFile, Simple, Level 4 |
7 |
5 |
Minor |
45 % |
Usort Sorting In PHP 7.0 |
CompatibilityPHP70, All |
1 |
1 |
Major |
46 % |
Wrong Type For Native PHP Function |
All, Analyze, CI-checks |
104 |
39 |
Minor |
0 % |
Always Anchor Regex |
All, Security |
6 |
6 |
Major |
65 % |
Avoid Those Hash Functions |
Security, All |
2 |
2 |
Major |
60 % |
Direct Injection |
Security, All |
11 |
7 |
Major |
41 % |
Indirect Injection |
Security, All |
3 |
3 |
Critical |
21 % |
No Net For Xml Load |
All, Security |
1 |
1 |
Major |
49 % |
Avoid sleep()/usleep() |
Security, All |
2 |
2 |
Minor |
39 % |
Should Use Prepared Statement |
Analyze, Security, All, Simple, CI-checks |
9 |
6 |
Major |
22 % |
Should Use session_regenerateid() |
All, Security |
1 |
1 |
Major |
82 % |
Unserialize Second Arg |
Security, All |
1 |
1 |
Critical |
49 % |
Adding Zero |
Analyze, OneFile, ClearPHP, All, Simple, Level 1, CI-checks |
16 |
14 |
Minor |
33 % |
Altering Foreach Without Reference |
Analyze, ClearPHP, All, Simple, Level 1, CI-checks |
2 |
2 |
Major |
52 % |
Alternative Syntax Consistence |
All, Analyze |
33 |
1 |
Major |
20 % |
Constant Comparison |
All, Analyze |
2 |
2 |
Minor |
0 % |
Array_merge Needs Array Of Arrays |
All, Analyze |
32 |
18 |
Minor |
0 % |
Bail Out Early |
All, Analyze, OneFile, Simple, Level 4 |
19 |
7 |
Minor |
92 % |
Strict Comparison With Booleans |
Analyze, All, Simple, Suggestions, Level 2, CI-checks |
50 |
35 |
Minor |
76 % |
Buried Assignation |
Analyze, All |
62 |
41 |
Minor |
82 % |
Cast To Boolean |
Analyze, OneFile, All, Simple, Level 1 |
5 |
5 |
Minor |
55 % |
Check All Types |
All, Analyze |
21 |
13 |
Major |
68 % |
Check JSON |
All, Analyze, CI-checks |
8 |
2 |
Major |
64 % |
Common Alternatives |
Analyze, All, Simple |
217 |
34 |
Major |
61 % |
Could Be Else |
All, Analyze |
4 |
3 |
Minor |
48 % |
Could Be Static |
Analyze, OneFile, All, ClassReview |
8 |
8 |
Major |
20 % |
Dont Change The Blind Var |
All, Analyze |
16 |
11 |
Minor |
72 % |
Dont Mix ++ |
All, Analyze |
4 |
2 |
Minor |
43 % |
Don't Read And Write In One Expression |
All, Analyze, CompatibilityPHP73, CompatibilityPHP74 |
1 |
1 |
Critical |
38 % |
Double Assignation |
Analyze, All |
4 |
4 |
Major |
55 % |
Double Instructions |
Analyze, All, Simple |
2 |
1 |
Minor |
57 % |
Drop Else After Return |
All, Analyze, OneFile, Simple, Suggestions, Level 4, CI-checks |
10 |
10 |
Minor |
70 % |
Echo With Concat |
Performances, Analyze, All, Simple, Suggestions |
32 |
14 |
Minor |
58 % |
Else If Versus Elseif |
Analyze, All, Simple, php-cs-fixable, Rector, CI-checks |
16 |
14 |
Minor |
65 % |
Empty Instructions |
Dead code, Analyze, All, Simple |
1652 |
48 |
Minor |
43 % |
Empty Try Catch |
Analyze, All, Level 3 |
2 |
2 |
Minor |
47 % |
Exit() Usage |
Analyze, Appinfo, OneFile, ClearPHP, All, CI-checks |
173 |
143 |
Major |
60 % |
Forgotten Whitespace |
Analyze, All, CI-checks |
1 |
1 |
Minor |
15 % |
Global Usage |
Analyze, Appinfo, ClearPHP, All |
184 |
51 |
Minor |
52 % |
Identical Consecutive Expression |
All, Analyze |
3 |
3 |
Minor |
58 % |
If With Same Conditions |
Analyze, All, Simple, CI-checks |
4 |
3 |
Major |
44 % |
Iffectations |
Analyze, All |
184 |
68 |
Minor |
80 % |
Implied If |
Analyze, ClearPHP, All, Simple, CI-checks |
139 |
139 |
Major |
29 % |
Implode() Arguments Order |
All, Analyze, CI-checks |
2 |
2 |
Minor |
0 % |
Inconsistent Elseif |
All, Analyze |
80 |
42 |
Major |
73 % |
Infinite Recursion |
All, Analyze |
3 |
1 |
Major |
0 % |
Long Arguments |
All, Analyze |
160 |
54 |
Minor |
67 % |
Max Level Of Nesting |
All, Analyze |
72 |
50 |
Minor |
0 % |
Merge If Then |
All, Analyze, CI-checks |
99 |
45 |
Minor |
0 % |
Mismatched Ternary Alternatives |
All, Analyze, Suggestions, Level 4 |
6 |
5 |
Major |
62 % |
Mixed Concat And Interpolation |
All, Coding Conventions, Analyze |
3 |
2 |
Minor |
43 % |
Modernize Empty With Expression |
All, Analyze, OneFile, Simple |
86 |
44 |
Minor |
56 % |
Multiple Type Variable |
All, Analyze, Level 4 |
5 |
5 |
Minor |
43 % |
Nested Ifthen |
Analyze, All, RadwellCodes |
332 |
79 |
Major |
87 % |
Nested Ternary |
Analyze, ClearPHP, All, Simple, Level 1, CI-checks |
1 |
1 |
Major |
20 % |
Next Month Trap |
All, Analyze, Top10, CI-checks |
1 |
1 |
Major |
7 % |
Don't Change Incomings |
Analyze, All |
2 |
2 |
Minor |
44 % |
No Choice |
Analyze, All, Simple, Level 2, Top10, CI-checks |
1 |
1 |
Major |
43 % |
No Hardcoded Hash |
Analyze, Security, All, Simple |
1 |
1 |
Critical |
44 % |
No Need For Else |
All, Analyze |
25 |
18 |
Minor |
74 % |
No Parenthesis For Language Construct |
Analyze, ClearPHP, All, RadwellCodes, Simple, Suggestions, Level 2, CI-checks |
69 |
31 |
Minor |
61 % |
Variable Is Not A Condition |
All, Analyze |
463 |
114 |
Minor |
57 % |
@ Operator |
Appinfo, ClearPHP, All, Analyze, Performances, CI-checks |
6 |
3 |
Minor |
66 % |
Not Equal Is Not !== |
All, Analyze, CI-checks |
1 |
1 |
Minor |
0 % |
include_once() Usage |
Analyze, Appinfo, All |
71 |
30 |
Minor |
73 % |
Several Instructions On The Same Line |
Analyze, All |
124 |
32 |
Major |
57 % |
Printf Number Of Arguments |
All, Analyze, CI-checks |
4 |
4 |
Minor |
42 % |
Repeated print() |
Analyze, All, Simple, Suggestions, Level 3, Top10, CI-checks |
11 |
10 |
Major |
49 % |
Repeated Regex |
All, Analyze, Level 1, CI-checks |
3 |
3 |
Minor |
47 % |
Return True False |
Analyze, All, Simple, Level 1, CI-checks |
4 |
4 |
Major |
53 % |
Same Conditions In Condition |
Analyze, All, Simple, CI-checks |
10 |
8 |
Critical |
49 % |
Should Chain Exception |
Analyze, All, Simple, CI-checks |
8 |
7 |
Minor |
36 % |
Should Make Ternary |
All, Analyze, OneFile, Simple, CI-checks |
182 |
68 |
Minor |
85 % |
Preprocessable |
Analyze, All, Rector |
199 |
154 |
Minor |
86 % |
Static Loop |
Analyze, All, Simple, Level 4 |
10 |
10 |
Minor |
65 % |
Strpos()-like Comparison |
Analyze, PHP recommendations, ClearPHP, All, Simple, Level 2, Top10, CI-checks |
4 |
2 |
Major |
54 % |
Switch To Switch |
Analyze, All, RadwellCodes, Simple |
11 |
11 |
Minor |
79 % |
Switch Without Default |
Analyze, ClearPHP, All, Simple, CI-checks |
32 |
21 |
Major |
68 % |
Test Then Cast |
All, Analyze |
5 |
4 |
Major |
32 % |
Timestamp Difference |
Analyze, All, Simple, Level 3, CI-checks |
2 |
2 |
Major |
50 % |
Unconditional Break In Loop |
All, Analyze, Level 3, CI-checks |
2 |
2 |
Major |
46 % |
Unreachable Code |
Dead code, OneFile, ClearPHP, All, Simple, Suggestions, Level 3 |
5 |
2 |
Major |
44 % |
Unused Global |
Analyze, All, Simple |
1 |
1 |
Minor |
18 % |
Use Positive Condition |
All, Analyze, OneFile, Simple |
82 |
45 |
Minor |
81 % |
Useless Casting |
All, PHP recommendations, Analyze, OneFile, RadwellCodes, Simple, Level 4, CI-checks |
5 |
4 |
Minor |
37 % |
Useless Global |
Analyze, OneFile, All, Simple, Level 2 |
13 |
9 |
Minor |
24 % |
Useless Instructions |
Analyze, OneFile, ClearPHP, All, Simple, Level 1, CI-checks |
2 |
2 |
Minor |
58 % |
Useless Parenthesis |
Analyze, All, Simple, CI-checks |
11 |
7 |
Minor |
81 % |
Useless Switch |
Analyze, All, Simple |
1 |
1 |
Major |
30 % |
Should Typecast |
Analyze, OneFile, All, Simple, CI-checks |
6 |
4 |
Minor |
47 % |
String May Hold A Variable |
Analyze, All, Simple |
6 |
5 |
Minor |
58 % |
Assigned Twice |
All, Analyze |
295 |
61 |
Minor |
78 % |
Overwritten Literals |
Analyze, All |
380 |
61 |
Major |
78 % |
Strange Name For Variables |
All, Analyze |
2 |
2 |
Minor |
28 % |
Undefined Variable |
All, Analyze, CI-checks |
78 |
18 |
Minor |
77 % |
Used Once Variables |
Analyze, OneFile, All, Simple, Top10 |
8 |
7 |
Minor |
87 % |
Used Once Variables (In Scope) |
Analyze, ClearPHP, OneFile, All, Simple, Level 4 |
163 |
86 |
Minor |
95 % |
Written Only Variables |
Analyze, OneFile, All, Simple |
132 |
51 |
Minor |
78 % |