| Null Or Boolean Arrays |
All, Analyze |
1 |
1 |
Minor |
0 % |
| String Initialization |
All, CompatibilityPHP71 |
2 |
2 |
Minor |
33 % |
| Ambiguous Static |
All, Analyze |
37 |
37 |
Minor |
64 % |
| Ambiguous Visibilities |
All, Analyze |
33 |
28 |
Minor |
70 % |
| Avoid option arrays in constructors |
All, Analyze, ClassReview |
5 |
5 |
Minor |
0 % |
| Avoid Optional Properties |
All, Analyze |
19 |
10 |
Major |
83 % |
| Cant Instantiate Class |
All, Analyze |
5 |
3 |
Critical |
28 % |
| Clone With Non-Object |
All, Analyze, LintButWontExec |
1 |
1 |
Minor |
0 % |
| Const Visibility Usage |
All, CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56 |
100 |
12 |
Minor |
16 % |
| Constant Class |
Analyze, All, Simple, CI-checks |
1 |
1 |
Minor |
49 % |
| Could Be Abstract Class |
All, Analyze, ClassReview |
4 |
4 |
Minor |
65 % |
| Class Could Be Final |
All, Analyze, ClassReview |
157 |
157 |
Minor |
95 % |
| Method Could Be Static |
All, Analyze, ClassReview |
84 |
48 |
Minor |
95 % |
| Don't Send $this In Constructor |
All, Analyze |
1 |
1 |
Minor |
39 % |
| Empty Classes |
Analyze, All, Simple |
10 |
10 |
Minor |
58 % |
| Class Should Be Final By Ocramius |
All, Analyze |
7 |
7 |
Minor |
42 % |
| Hidden Nullable |
All, Analyze, ClassReview |
54 |
21 |
Minor |
0 % |
| Incompatible Signature Methods With Covariance |
All, Analyze |
6 |
6 |
Critical |
0 % |
| Locally Unused Property |
Dead code, All, Simple |
21 |
7 |
Minor |
76 % |
| Assign Default To Properties |
Analyze, ClearPHP, All, Simple, Level 2 |
3 |
3 |
Minor |
49 % |
| Make Global A Property |
Analyze, All, Simple |
1 |
1 |
Minor |
33 % |
| Method Signature Must Be Compatible |
All, Analyze, LintButWontExec |
54 |
18 |
Critical |
50 % |
| Non Nullable Getters |
All, Analyze, ClassReview |
38 |
22 |
Minor |
0 % |
| Parent First |
All, Analyze, Suggestions |
3 |
3 |
Minor |
55 % |
| Property Could Be Local |
All, Analyze, ClassReview |
198 |
73 |
Minor |
71 % |
| Never Used Properties |
Analyze, All, Simple |
9 |
5 |
Minor |
72 % |
| Property Used In One Method Only |
All, Analyze |
20 |
12 |
Minor |
84 % |
| Redefined Default |
Analyze, All, Simple, CI-checks |
1 |
1 |
Major |
26 % |
| Could Use self |
Analyze, All, Simple, Suggestions, Level 3, ClassReview |
1 |
1 |
Minor |
57 % |
| Should Use Local Class |
Analyze, ClearPHP, All, Simple |
104 |
56 |
Minor |
94 % |
| Too Many Dereferencing |
All, Analyze |
1 |
1 |
Minor |
0 % |
| Too Many Injections |
All, Analyze, Dismell |
4 |
4 |
Minor |
0 % |
| Undefined Classes |
Analyze, All |
320 |
67 |
Major |
91 % |
| Undefined Class Constants |
Analyze, All, CI-checks |
50 |
12 |
Major |
53 % |
| Undefined ::class |
All, Analyze, CI-checks |
2 |
1 |
Major |
34 % |
| Unresolved Catch |
Dead code, ClearPHP, All |
2 |
2 |
Major |
51 % |
| Unresolved Classes |
Analyze, All |
43 |
23 |
Major |
86 % |
| Unused Classes |
Dead code, All, Simple, Analyze |
2 |
2 |
Major |
72 % |
| Unused Class Constant |
All, Analyze, ClassReview |
3 |
3 |
Minor |
0 % |
| Unused Methods |
Dead code, All, Simple |
207 |
95 |
Minor |
95 % |
| Unused Private Properties |
Dead code, OneFile, All, Simple |
16 |
12 |
Minor |
37 % |
| Unused Protected Methods |
Dead code, All |
6 |
4 |
Major |
72 % |
| Use Instanceof |
Analyze, All, Simple, CI-checks |
6 |
5 |
Major |
63 % |
| Used Once Property |
All, Analyze |
20 |
11 |
Minor |
72 % |
| Undefined Caught Exceptions |
Dead code, All |
2 |
2 |
None |
50 % |
| Forgotten Thrown |
All, Analyze |
1 |
1 |
Major |
30 % |
| Uncaught Exceptions |
Analyze, All |
94 |
50 |
Minor |
81 % |
| Unthrown Exception |
Analyze, Dead code, ClearPHP, All, Simple |
2 |
2 |
Minor |
43 % |
| Useless Catch |
All, Analyze, CI-checks |
2 |
2 |
Minor |
47 % |
| Missing Include |
All, Analyze |
2 |
1 |
Critical |
61 % |
| Use Named Boolean In Argument Definition |
All, Analyze |
10 |
8 |
Minor |
84 % |
| Callback Needs Return |
All, Analyze, CI-checks |
1 |
1 |
Major |
51 % |
| Could Make A Function |
All, Analyze, Suggestions |
31 |
19 |
Minor |
75 % |
| Empty Function |
Analyze, All, Simple |
16 |
5 |
Minor |
71 % |
| Hardcoded Passwords |
Analyze, OneFile, Security, All, Simple, Level 3 |
22 |
5 |
Major |
61 % |
| Insufficient Typehint |
All, Analyze, Typechecks |
6 |
3 |
Major |
24 % |
| Mismatch Parameter And Type |
All, Analyze, Semantics |
2 |
2 |
Minor |
0 % |
| Mismatch Parameter Name |
All, Analyze, CompatibilityPHP80 |
9 |
9 |
Minor |
0 % |
| Mismatched Default Arguments |
All, Analyze, Typechecks |
11 |
9 |
Minor |
33 % |
| Mismatched Typehint |
All, Analyze, Typechecks |
21 |
10 |
Major |
5 % |
| Modified Typed Parameter |
All, Analyze, ClassReview |
4 |
4 |
Minor |
0 % |
| Must Return Methods |
Analyze, All, Simple, Level 2, LintButWontExec, CI-checks |
5 |
4 |
Major |
22 % |
| Never Used Parameter |
All, Analyze, Suggestions |
43 |
37 |
Minor |
47 % |
| No Boolean As Default |
All, Analyze |
10 |
8 |
Minor |
84 % |
| No Return Used |
All, Analyze, Suggestions, Level 4 |
1 |
1 |
Minor |
41 % |
| Relay Function |
Analyze, All |
10 |
6 |
Major |
83 % |
| Too Many Local Variables |
All, Analyze |
5 |
4 |
Minor |
65 % |
| Typehint Must Be Returned |
All, Analyze, LintButWontExec, CI-checks |
5 |
4 |
Major |
10 % |
| Undefined Functions |
Analyze, All, CI-checks |
59 |
35 |
Major |
71 % |
| Unused Arguments |
Analyze, All, Simple |
26 |
8 |
Major |
89 % |
| Useless Referenced Argument |
All, Analyze |
1 |
1 |
Minor |
57 % |
| Useless Type Check |
All, Dead code, OneFile |
3 |
3 |
Minor |
0 % |
| Uses Default Values |
Analyze, All, Simple, CI-checks |
215 |
68 |
Minor |
97 % |
| Using Deprecated Method |
All, Analyze |
1 |
1 |
Minor |
0 % |
| Methods Without Return |
All, Analyze |
13 |
8 |
Minor |
0 % |
| Wrong Number Of Arguments |
Analyze, OneFile, All, Simple, CI-checks |
7 |
6 |
Major |
38 % |
| Wrong Returned Type |
All, Analyze, ClassReview, CI-checks |
48 |
24 |
Minor |
0 % |
| Wrong Type With Call |
All, Analyze, Typechecks, CI-checks |
14 |
7 |
Minor |
0 % |
| Already Parents Interface |
Analyze, All, Level 3, Suggestions |
1 |
1 |
Minor |
17 % |
| Forgotten Interface |
All, Analyze |
8 |
8 |
Minor |
52 % |
| Empty Interfaces |
Analyze, All, Simple |
4 |
4 |
Minor |
38 % |
| Undefined Interfaces |
Analyze, All, CI-checks |
207 |
65 |
Major |
67 % |
| Unused Interfaces |
Dead code, All, Simple, Suggestions, Level 2 |
1 |
1 |
Minor |
25 % |
| Useless Interfaces |
Analyze, ClearPHP, All, Simple, ClassReview, Typechecks |
4 |
4 |
Minor |
57 % |
| Multiple Alias Definitions |
Analyze, All, Simple, CI-checks |
13 |
12 |
Minor |
42 % |
| Should Make Alias |
Analyze, OneFile, All, Simple, CI-checks |
31 |
5 |
Minor |
44 % |
| Unresolved Use |
Analyze, ClearPHP, All, Simple |
46 |
34 |
Major |
65 % |
| Unused Use |
Dead code, ClearPHP, All, Simple |
19 |
18 |
Major |
54 % |
| Use With Fully Qualified Name |
Analyze, Coding Conventions, PHP recommendations, All, Simple |
1 |
1 |
Minor |
16 % |
| No array_merge() In Loops |
Analyze, Performances, ClearPHP, All, Simple, Level 2, Top10, CI-checks |
5 |
5 |
Major |
55 % |
| Logical To in_array |
All, Analyze, Level 2 |
4 |
4 |
Minor |
66 % |
| Pre-increment |
Analyze, Performances, All, Simple, Level 4, CI-checks |
33 |
17 |
Minor |
74 % |
| array_key_exists() Works On Arrays |
All, CompatibilityPHP74, Analyze |
4 |
4 |
Minor |
0 % |
| Assumptions |
All, Analyze |
15 |
14 |
Minor |
0 % |
| Use random_int() |
Analyze, Security, All, CompatibilityPHP71, Simple, Level 2, CI-checks |
2 |
1 |
Major |
59 % |
| Wrong Parameter Type |
Analyze, OneFile, All, Simple, CI-checks |
7 |
5 |
Major |
45 % |
| Use === null |
Analyze, OneFile, All, RadwellCodes, Simple, php-cs-fixable, CI-checks |
1 |
1 |
Minor |
52 % |
| PHP 7.0 Scalar Typehints |
All, CompatibilityPHP53, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56 |
333 |
109 |
Critical |
31 % |
| PHP 7.1 Scalar Typehints |
All, CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56 |
7 |
3 |
Critical |
0 % |
| PHP 7.2 Scalar Typehints |
All, CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP71, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56 |
1 |
1 |
Critical |
0 % |
| Php7 Relaxed Keyword |
CompatibilityPHP53, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56, All, Appinfo |
3 |
3 |
Major |
25 % |
| PHP Keywords As Names |
Analyze, All, Simple |
169 |
27 |
Major |
89 % |
| Scalar Are Not Arrays |
All, Analyze, CompatibilityPHP74, CI-checks |
3 |
2 |
Minor |
0 % |
| Should Use Coalesce |
Analyze, All, Simple, Suggestions, Level 3, CI-checks |
1 |
1 |
Major |
71 % |
| Use Nullable Type |
All, Appinfo, CompatibilityPHP53, CompatibilityPHP70, CompatibilityPHP54, CompatibilityPHP55, CompatibilityPHP56 |
110 |
45 |
Major |
21 % |
| Usort Sorting In PHP 7.0 |
CompatibilityPHP70, All |
1 |
1 |
Major |
46 % |
| Wrong Type For Native PHP Function |
All, Analyze, CI-checks |
32 |
17 |
Minor |
0 % |
| Avoid Those Hash Functions |
Security, All |
3 |
3 |
Major |
60 % |
| Direct Injection |
Security, All |
1 |
1 |
Major |
41 % |
| Don't Echo Error |
All, Analyze, Security, Simple, Level 1, CI-checks |
1 |
1 |
Critical |
32 % |
| No Net For Xml Load |
All, Security |
5 |
5 |
Major |
49 % |
| Avoid sleep()/usleep() |
Security, All |
1 |
1 |
Minor |
39 % |
| Unserialize Second Arg |
Security, All |
1 |
1 |
Critical |
49 % |
| Adding Zero |
Analyze, OneFile, ClearPHP, All, Simple, Level 1, CI-checks |
7 |
3 |
Minor |
33 % |
| Altering Foreach Without Reference |
Analyze, ClearPHP, All, Simple, Level 1, CI-checks |
2 |
2 |
Major |
52 % |
| Constant Comparison |
All, Analyze |
24 |
5 |
Minor |
0 % |
| Array_merge Needs Array Of Arrays |
All, Analyze |
4 |
3 |
Minor |
0 % |
| Strict Comparison With Booleans |
Analyze, All, Simple, Suggestions, Level 2, CI-checks |
9 |
6 |
Minor |
76 % |
| Buried Assignation |
Analyze, All |
8 |
7 |
Minor |
82 % |
| Cast To Boolean |
Analyze, OneFile, All, Simple, Level 1 |
2 |
2 |
Minor |
55 % |
| Catch Overwrite Variable |
Analyze, ClearPHP, All, Simple |
1 |
1 |
Minor |
23 % |
| Common Alternatives |
Analyze, All, Simple |
1 |
1 |
Major |
61 % |
| Could Be Else |
All, Analyze |
3 |
2 |
Minor |
48 % |
| Could Be Static |
Analyze, OneFile, All, ClassReview |
1 |
1 |
Major |
20 % |
| Could Use Short Assignation |
Analyze, OneFile, Performances, All, Simple, CI-checks |
6 |
5 |
Minor |
60 % |
| Dont Change The Blind Var |
All, Analyze |
3 |
3 |
Minor |
72 % |
| Dont Mix ++ |
All, Analyze |
3 |
3 |
Minor |
43 % |
| Double Assignation |
Analyze, All |
1 |
1 |
Major |
55 % |
| Double Instructions |
Analyze, All, Simple |
3 |
3 |
Minor |
57 % |
| Echo With Concat |
Performances, Analyze, All, Simple, Suggestions |
4 |
2 |
Minor |
58 % |
| Empty Blocks |
Analyze, All, Simple, CI-checks |
1 |
1 |
Minor |
57 % |
| Empty Instructions |
Dead code, Analyze, All, Simple |
1 |
1 |
Minor |
43 % |
| Empty Try Catch |
Analyze, All, Level 3 |
2 |
1 |
Minor |
47 % |
| Exit() Usage |
Analyze, Appinfo, OneFile, ClearPHP, All, CI-checks |
4 |
2 |
Major |
60 % |
| Foreach Reference Is Not Modified |
Analyze, All, Simple, CI-checks |
1 |
1 |
Minor |
24 % |
| Global Usage |
Analyze, Appinfo, ClearPHP, All |
1 |
1 |
Minor |
52 % |
| Iffectations |
Analyze, All |
22 |
11 |
Minor |
80 % |
| Implode() Arguments Order |
All, Analyze, CI-checks |
2 |
2 |
Minor |
0 % |
| Inconsistent Elseif |
All, Analyze |
2 |
2 |
Major |
73 % |
| Infinite Recursion |
All, Analyze |
7 |
5 |
Major |
0 % |
| Long Arguments |
All, Analyze |
5 |
5 |
Minor |
67 % |
| Max Level Of Nesting |
All, Analyze |
5 |
3 |
Minor |
0 % |
| Merge If Then |
All, Analyze, CI-checks |
1 |
1 |
Minor |
0 % |
| Mismatched Ternary Alternatives |
All, Analyze, Suggestions, Level 4 |
1 |
1 |
Major |
62 % |
| Modernize Empty With Expression |
All, Analyze, OneFile, Simple |
5 |
3 |
Minor |
56 % |
| Multiple Type Variable |
All, Analyze, Level 4 |
1 |
1 |
Minor |
43 % |
| Nested Ifthen |
Analyze, All, RadwellCodes |
3 |
2 |
Major |
87 % |
| Nested Ternary |
Analyze, ClearPHP, All, Simple, Level 1, CI-checks |
1 |
1 |
Major |
20 % |
| No Direct Usage |
Analyze, All, Simple |
1 |
1 |
Major |
36 % |
| No Hardcoded Hash |
Analyze, Security, All, Simple |
2 |
1 |
Critical |
44 % |
| No Need For Triple Equal |
All, Analyze |
3 |
3 |
Minor |
0 % |
| No Parenthesis For Language Construct |
Analyze, ClearPHP, All, RadwellCodes, Simple, Suggestions, Level 2, CI-checks |
3 |
2 |
Minor |
61 % |
| Avoid Substr() One |
Analyze, Performances, All, CompatibilityPHP71, Simple, Level 2, Suggestions, Top10, CI-checks |
3 |
3 |
Minor |
57 % |
| Variable Is Not A Condition |
All, Analyze |
70 |
39 |
Minor |
57 % |
| @ Operator |
Appinfo, ClearPHP, All, Analyze, Performances, CI-checks |
2 |
1 |
Minor |
66 % |
| include_once() Usage |
Analyze, Appinfo, All |
1 |
1 |
Minor |
73 % |
| Printf Number Of Arguments |
All, Analyze, CI-checks |
4 |
3 |
Minor |
42 % |
| Repeated print() |
Analyze, All, Simple, Suggestions, Level 3, Top10, CI-checks |
2 |
2 |
Major |
49 % |
| Return True False |
Analyze, All, Simple, Level 1, CI-checks |
1 |
1 |
Major |
53 % |
| Should Chain Exception |
Analyze, All, Simple, CI-checks |
1 |
1 |
Minor |
36 % |
| Should Make Ternary |
All, Analyze, OneFile, Simple, CI-checks |
11 |
8 |
Minor |
85 % |
| Preprocessable |
Analyze, All, Rector |
23 |
13 |
Minor |
86 % |
| Static Loop |
Analyze, All, Simple, Level 4 |
8 |
5 |
Minor |
65 % |
| Switch Without Default |
Analyze, ClearPHP, All, Simple, CI-checks |
3 |
3 |
Major |
68 % |
| Timestamp Difference |
Analyze, All, Simple, Level 3, CI-checks |
1 |
1 |
Major |
50 % |
| Unconditional Break In Loop |
All, Analyze, Level 3, CI-checks |
8 |
5 |
Major |
46 % |
| Unpreprocessed Values |
Analyze, OneFile, ClearPHP, All, Simple |
1 |
1 |
Minor |
28 % |
| Unreachable Code |
Dead code, OneFile, ClearPHP, All, Simple, Suggestions, Level 3 |
19 |
5 |
Major |
44 % |
| Unsupported Types With Operators |
All, Analyze, CompatibilityPHP80 |
3 |
1 |
Minor |
0 % |
| Use Positive Condition |
All, Analyze, OneFile, Simple |
1 |
1 |
Minor |
81 % |
| Useless Casting |
All, PHP recommendations, Analyze, OneFile, RadwellCodes, Simple, Level 4, CI-checks |
2 |
2 |
Minor |
37 % |
| Useless Global |
Analyze, OneFile, All, Simple, Level 2 |
1 |
1 |
Minor |
24 % |
| Useless Instructions |
Analyze, OneFile, ClearPHP, All, Simple, Level 1, CI-checks |
6 |
6 |
Minor |
58 % |
| Useless Parenthesis |
Analyze, All, Simple, CI-checks |
15 |
7 |
Minor |
81 % |
| No Real Comparison |
Analyze, All, Simple, Level 2, Top10, CI-checks |
1 |
1 |
Major |
22 % |
| Missing Returntype In Method |
All, Typehints, Analyze, CI-checks |
52 |
27 |
Minor |
0 % |
| Assigned Twice |
All, Analyze |
17 |
5 |
Minor |
78 % |
| Overwritten Literals |
Analyze, All |
19 |
5 |
Major |
78 % |
| Undefined Variable |
All, Analyze, CI-checks |
11 |
5 |
Minor |
77 % |
| Used Once Variables |
Analyze, OneFile, All, Simple, Top10 |
2 |
2 |
Minor |
87 % |
| Used Once Variables (In Scope) |
Analyze, ClearPHP, OneFile, All, Simple, Level 4 |
52 |
18 |
Minor |
95 % |
| Written Only Variables |
Analyze, OneFile, All, Simple |
25 |
10 |
Minor |
78 % |