Cleaned up code - some adjustments to selected files

The following adjustments were made:
- Files should end with a newline (PHP:S113)
- Control structures should use curly braces (PHP:S121)
- Sections of code should not be commented out (PHP:S125)
- Empty statements should be removed (PHP:S1116)
- Lines should not end with trailing whitespaces (PHP:S1131)
- Unused function parameters should be removed (PHP:S1172)
- Unused local variables should be removed (PHP:S1481)
- PHP keywords and constants "true", "false", "null" should be lower
case (PHP:S1781)
- Method visibility should be explicitly declared (PHP:S1784)
- Method arguments with default values should be last (PHP:S1788)
- "elseif" keyword should be used in place of "else if" keywords
(PHP:S1793)
- "require_once" and "include_once" should be used instead of "require"
and "include" (PHP:2003)
- "&&" and "||" should be used (PHP:S2010)
- Jump statements should not be redundant (PHP:S3626)

Author: DanProgs

lib/results/metricsDashboard.php

@@ -30,7 +30,7 @@
 list($gui->tplan_metrics,$gui->show_platforms, $platforms) = getMetrics($db,$_SESSION['currentUser'],$args,$result_cfg, $labels);
 
 
-if(count($gui->tplan_metrics) > 0)
+if(!empty($gui->tplan_metrics))
 {
   $statusSetForDisplay = $result_cfg['status_label_for_exec_ui'];
   $gui->warning_msg = '';