Conditions | 4 |
Paths | 4 |
Total Lines | 17 |
Code Lines | 14 |
Lines | 0 |
Ratio | 0 % |
Changes | 2 | ||
Bugs | 0 | Features | 1 |
1 | <?php |
||
23 | public static function makeSubTitle($section) |
||
24 | { |
||
25 | switch ($section) { |
||
26 | case 'seoaspects': |
||
27 | return 'SEO Aspekte'; |
||
28 | break; |
||
|
|||
29 | case 'contentaspects': |
||
30 | return 'Technische Aspekte'; |
||
31 | break; |
||
32 | case 'technicalaspects': |
||
33 | return 'Technische Aspekte'; |
||
34 | break; |
||
35 | default: |
||
36 | return null; |
||
37 | break; |
||
38 | } |
||
39 | } |
||
40 | } |
||
41 |
The break statement is not necessary if it is preceded for example by a return statement:
If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive.