Inspection of "Indentations fixes" - Ysurac/FlightAirMap - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e17801...597a61 )

by Yannick

created 2017-06-28 09:31 UTC

Status

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

@@ -12,8 +12,12 @@
 block discarded – undo
 $year = filter_input(INPUT_GET,'year',FILTER_SANITIZE_NUMBER_INT);
 $month = filter_input(INPUT_GET,'month',FILTER_SANITIZE_NUMBER_INT);
 $filter = array();
-if ($year != '') $filter = array_merge($filter,array('year' => $year));
-if ($month != '') $filter = array_merge($filter,array('month' => $month));
+if ($year != '') {
+	$filter = array_merge($filter,array('year' => $year));
+}
+if ($month != '') {
+	$filter = array_merge($filter,array('month' => $month));
+}
 $spotter_array = $Spotter->getSpotterDataByPilot($pilot,"0,1", $sort,$filter);
 
 if (!empty($spotter_array))

Please login to merge, or discard this patch.

pilot-statistics-departure-airport-country.php 1 patch

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

@@ -12,8 +12,12 @@
 block discarded – undo
 $year = filter_input(INPUT_GET,'year',FILTER_SANITIZE_NUMBER_INT);
 $month = filter_input(INPUT_GET,'month',FILTER_SANITIZE_NUMBER_INT);
 $filter = array();
-if ($year != '') $filter = array_merge($filter,array('year' => $year));
-if ($month != '') $filter = array_merge($filter,array('month' => $month));
+if ($year != '') {
+	$filter = array_merge($filter,array('year' => $year));
+}
+if ($month != '') {
+	$filter = array_merge($filter,array('month' => $month));
+}
 $spotter_array = $Spotter->getSpotterDataByPilot($pilot,"0,1", $sort,$filter);
 
 if (!empty($spotter_array))

Please login to merge, or discard this patch.

pilot-statistics-arrival-airport-country.php 1 patch

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

@@ -12,8 +12,12 @@
 block discarded – undo
 $year = filter_input(INPUT_GET,'year',FILTER_SANITIZE_NUMBER_INT);
 $month = filter_input(INPUT_GET,'month',FILTER_SANITIZE_NUMBER_INT);
 $filter = array();
-if ($year != '') $filter = array_merge($filter,array('year' => $year));
-if ($month != '') $filter = array_merge($filter,array('month' => $month));
+if ($year != '') {
+	$filter = array_merge($filter,array('year' => $year));
+}
+if ($month != '') {
+	$filter = array_merge($filter,array('month' => $month));
+}
 $spotter_array = $Spotter->getSpotterDataByPilot($pilot,"0,1", $sort,$filter);
 
 if (!empty($spotter_array))

Please login to merge, or discard this patch.

owner-statistics-airline.php 1 patch

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

@@ -12,8 +12,12 @@
 block discarded – undo
 $year = filter_input(INPUT_GET,'year',FILTER_SANITIZE_NUMBER_INT);
 $month = filter_input(INPUT_GET,'month',FILTER_SANITIZE_NUMBER_INT);
 $filter = array();
-if ($year != '') $filter = array_merge($filter,array('year' => $year));
-if ($month != '') $filter = array_merge($filter,array('month' => $month));
+if ($year != '') {
+	$filter = array_merge($filter,array('year' => $year));
+}
+if ($month != '') {
+	$filter = array_merge($filter,array('month' => $month));
+}
 $spotter_array = $Spotter->getSpotterDataByOwner($owner,"0,1", $sort,$filter);
 
 if (!empty($spotter_array))

Please login to merge, or discard this patch.

owner-detailed.php 1 patch

Braces +23 added lines, -8 removed lines patch added patch discarded remove patch

@@ -40,8 +40,12 @@  discard block
 block discarded – undo
 	$year = filter_input(INPUT_GET,'year',FILTER_SANITIZE_NUMBER_INT);
 	$month = filter_input(INPUT_GET,'month',FILTER_SANITIZE_NUMBER_INT);
 	$filter = array();
-	if ($year != '') $filter = array_merge($filter,array('year' => $year));
-	if ($month != '') $filter = array_merge($filter,array('month' => $month));
+	if ($year != '') {
+		$filter = array_merge($filter,array('year' => $year));
+	}
+	if ($month != '') {
+		$filter = array_merge($filter,array('month' => $month));
+	}
 	if ($sort != '') 
 	{
 		$spotter_array = $Spotter->getSpotterDataByOwner($owner,$limit_start.",".$absolute_difference, $sort,$filter);
@@ -59,8 +63,12 @@  discard block
 block discarded – undo
 	{
 		$title = sprintf(_("Detailed View for %s"),$spotter_array[0]['aircraft_owner']);
 		//$ident = $spotter_array[0]['ident'];
-		if (isset($spotter_array[0]['latitude'])) $latitude = $spotter_array[0]['latitude'];
-		if (isset($spotter_array[0]['longitude'])) $longitude = $spotter_array[0]['longitude'];
+		if (isset($spotter_array[0]['latitude'])) {
+			$latitude = $spotter_array[0]['latitude'];
+		}
+		if (isset($spotter_array[0]['longitude'])) {
+			$longitude = $spotter_array[0]['longitude'];
+		}
 		require_once('header.php');
 		/*
 		if (isset($globalArchive) && $globalArchive) {
@@ -115,9 +123,14 @@  discard block
 block discarded – undo
 		print '<h1>'.$spotter_array[0]['aircraft_owner'].'</h1>';
 		//print '<div><span class="label">'._("Owner").'</span>'.$spotter_array[0]['aircraft_owner'].'</div>';
 		$Stats = new Stats();
-		if ($year == '' && $month == '') $flights = $Stats->getStatsOwner($owner);
-		else $flights = 0;
-		if ($flights == 0) $flights = $Spotter->countFlightsByOwner($owner,$filter);
+		if ($year == '' && $month == '') {
+			$flights = $Stats->getStatsOwner($owner);
+		} else {
+			$flights = 0;
+		}
+		if ($flights == 0) {
+			$flights = $Spotter->countFlightsByOwner($owner,$filter);
+		}
 		print '<div><span class="label">'._("Flights").'</span>'.$flights.'</div>';
 		$aircraft_type = count($Spotter->countAllAircraftTypesByOwner($owner,$filter));
 		print '<div><span class="label">'._("Aircrafts type").'</span>'.$aircraft_type.'</div>';
@@ -128,7 +141,9 @@  discard block
 block discarded – undo
 		$airlines = count($Spotter->countAllAirlinesByOwner($owner,$filter));
 		print '<div><span class="label">'._("Airlines").'</span>'.$airlines.'</div>';
 		$duration = $Spotter->getFlightDurationByOwner($owner,$filter);
-		if ($duration != '0') print '<div><span class="label">'._("Total flights spotted duration").'</span>'.$duration.'</div>';
+		if ($duration != '0') {
+			print '<div><span class="label">'._("Total flights spotted duration").'</span>'.$duration.'</div>';
+		}
 		print '</div>';
 	
 		include('owner-sub-menu.php');

Please login to merge, or discard this patch.

require/class.SpotterArchive.php 1 patch

Braces +37 added lines, -13 removed lines patch added patch discarded remove patch

@@ -26,7 +26,9 @@  discard block
 block discarded – undo
 	if (isset($filter[0]['source'])) {
 		$filters = array_merge($filters,$filter);
 	}
-	if (is_array($globalFilter)) $filter = array_merge($filter,$globalFilter);
+	if (is_array($globalFilter)) {
+		$filter = array_merge($filter,$globalFilter);
+	}
 	$filter_query_join = '';
 	$filter_query_where = '';
 	foreach($filters as $flt) {
@@ -99,8 +101,11 @@  discard block
 block discarded – undo
 	    }
 	    $filter_query_join .= " INNER JOIN (SELECT flightaware_id FROM spotter_archive_output".preg_replace('/^ AND/',' WHERE',$filter_query_date).") sd ON sd.flightaware_id = spotter_archive_output.flightaware_id";
 	}
-	if ($filter_query_where == '' && $where) $filter_query_where = ' WHERE';
-	elseif ($filter_query_where != '' && $and) $filter_query_where .= ' AND';
+	if ($filter_query_where == '' && $where) {
+		$filter_query_where = ' WHERE';
+	} elseif ($filter_query_where != '' && $and) {
+		$filter_query_where .= ' AND';
+	}
 	if ($filter_query_where != '') {
 		$filter_query_where = preg_replace('/^ AND/',' WHERE',$filter_query_where);
 	}
@@ -114,10 +119,17 @@  discard block
 block discarded – undo
 		if ($over_country == '') {
 			$Spotter = new Spotter($this->db);
 			$data_country = $Spotter->getCountryFromLatitudeLongitude($latitude,$longitude);
-			if (!empty($data_country)) $country = $data_country['iso2'];
-			else $country = '';
-		} else $country = $over_country;
-		if ($airline_type === NULL) $airline_type ='';
+			if (!empty($data_country)) {
+				$country = $data_country['iso2'];
+			} else {
+				$country = '';
+			}
+		} else {
+			$country = $over_country;
+		}
+		if ($airline_type === NULL) {
+			$airline_type ='';
+		}
 	
 		//if ($country == '') echo "\n".'************ UNKNOW COUNTRY ****************'."\n";
 		//else echo "\n".'*/*/*/*/*/*/*/ Country : '.$country.' */*/*/*/*/*/*/*/*/'."\n";
@@ -620,7 +632,9 @@  discard block
 block discarded – undo
 		    $additional_query .= "(spotter_archive_output.pilot_name like '%".$q_item."%') OR ";
 		    $additional_query .= "(spotter_archive_output.ident like '%".$q_item."%') OR ";
 		    $translate = $Translation->ident2icao($q_item);
-		    if ($translate != $q_item) $additional_query .= "(spotter_archive_output.ident like '%".$translate."%') OR ";
+		    if ($translate != $q_item) {
+		    	$additional_query .= "(spotter_archive_output.ident like '%".$translate."%') OR ";
+		    }
 		    $additional_query .= "(spotter_archive_output.highlight like '%".$q_item."%')";
 		    $additional_query .= ")";
 		}
@@ -838,7 +852,9 @@  discard block
 block discarded – undo
 		date_default_timezone_set($globalTimezone);
 		$datetime = new DateTime();
 		$offset = $datetime->format('P');
-	    } else $offset = '+00:00';
+	    } else {
+	    	$offset = '+00:00';
+	    }
 
 
 	    if ($date_array[1] != "")
@@ -1114,9 +1130,13 @@  discard block
 block discarded – undo
 				$query .= "AND date < CURRENT_TIMESTAMP AT TIME ZONE 'UTC' - INTERVAL '".$olderthanmonths." MONTHS'";
 			}
 		}
-                if ($sincedate != '') $query .= "AND date > '".$sincedate."' ";
+                if ($sincedate != '') {
+                	$query .= "AND date > '".$sincedate."' ";
+                }
 	$query .= "GROUP BY c.name, c.iso3, c.iso2 ORDER BY nb DESC";
-	if ($limit) $query .= " LIMIT 0,10";
+	if ($limit) {
+		$query .= " LIMIT 0,10";
+	}
       
 	
 	$sth = $this->db->prepare($query);
@@ -1160,9 +1180,13 @@  discard block
 block discarded – undo
 				$query .= "AND s.date < CURRENT_TIMESTAMP AT TIME ZONE 'UTC' - INTERVAL '".$olderthanmonths." MONTHS'";
 			}
 		}
-                if ($sincedate != '') $query .= "AND s.date > '".$sincedate."' ";
+                if ($sincedate != '') {
+                	$query .= "AND s.date > '".$sincedate."' ";
+                }
 	$query .= "GROUP BY o.airline_icao,c.name, c.iso3, c.iso2 ORDER BY nb DESC";
-	if ($limit) $query .= " LIMIT 0,10";
+	if ($limit) {
+		$query .= " LIMIT 0,10";
+	}
       
 	
 	$sth = $this->db->prepare($query);

Please login to merge, or discard this patch.

route-statistics-manufacturer.php 1 patch

Braces +3 added lines, -1 removed lines patch added patch discarded remove patch

@@ -14,7 +14,9 @@
 block discarded – undo
 $arrival_airport = filter_input(INPUT_GET,'arrival_airport',FILTER_SANITIZE_STRING);
 if (isset($_GET['departure_airport']) && isset($_GET['arrival_airport'])) {
 	$spotter_array = $Spotter->getSpotterDataByRoute($departure_airport, $arrival_airport, "0,1", $sort);
-} else $spotter_array = array();  
+} else {
+	$spotter_array = array();
+}
 
 if (!empty($spotter_array))
 {

Please login to merge, or discard this patch.

pilot-sub-menu.php 1 patch

Braces +70 added lines, -10 removed lines patch added patch discarded remove patch

@@ -3,15 +3,39 @@  discard block
 block discarded – undo
 </span>
 <div class="sub-menu sub-menu-container">
 	<ul class="nav nav-pills">
-		<li><a href="<?php print $globalURL; ?>/pilot/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>" <?php if (strtolower($current_page) == "pilot-detailed"){ print 'class="active"'; } ?>><?php echo _("Detailed"); ?></a></li>
+		<li><a href="<?php print $globalURL; ?>/pilot/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>" <?php if (strtolower($current_page) == "pilot-detailed"){ print 'class="active"'; } ?>><?php echo _("Detailed"); ?></a></li>
 		<li class="dropdown">
 		    <a class="dropdown-toggle <?php if(strtolower($current_page) == "pilot-statistics-aircraft" || strtolower($current_page) == "pilot-statistics-registration" || strtolower($current_page) == "pilot-statistics-manufacturer"){ print 'active'; } ?>" data-toggle="dropdown" href="#">
 		      <?php echo _("Aircraft"); ?> <span class="caret"></span>
 		    </a>
 		    <ul class="dropdown-menu" role="menu">
-		      <li><a href="<?php print $globalURL; ?>/pilot/statistics/aircraft/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Aircraft Type"); ?></a></li>
-					<li><a href="<?php print $globalURL; ?>/pilot/statistics/registration/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Registration"); ?></a></li>
-					<li><a href="<?php print $globalURL; ?>/pilot/statistics/manufacturer/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Manufacturer"); ?></a></li>
+		      <li><a href="<?php print $globalURL; ?>/pilot/statistics/aircraft/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>"><?php echo _("Aircraft Type"); ?></a></li>
+					<li><a href="<?php print $globalURL; ?>/pilot/statistics/registration/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>"><?php echo _("Registration"); ?></a></li>
+					<li><a href="<?php print $globalURL; ?>/pilot/statistics/manufacturer/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>"><?php echo _("Manufacturer"); ?></a></li>
 		    </ul>
 		</li>
 		<li class="dropdown">
@@ -19,13 +43,49 @@  discard block
 block discarded – undo
 		      <?php echo _("Airport"); ?> <span class="caret"></span>
 		    </a>
 		    <ul class="dropdown-menu" role="menu">
-		      <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Departure Airport"); ?></a></li>
-		      <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Departure Airport by Country"); ?></a></li>
-			  <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Arrival Airport"); ?></a></li>
-			  <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Arrival Airport by Country"); ?></a></li>
+		      <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>"><?php echo _("Departure Airport"); ?></a></li>
+		      <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>"><?php echo _("Departure Airport by Country"); ?></a></li>
+			  <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>"><?php echo _("Arrival Airport"); ?></a></li>
+			  <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>"><?php echo _("Arrival Airport by Country"); ?></a></li>
 		    </ul>
 		</li>
-		<li><a href="<?php print $globalURL; ?>/pilot/statistics/route/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>" <?php if (strtolower($current_page) == "pilot-statistics-route"){ print 'class="active"'; } ?>><?php echo _("Route"); ?></a></li>
-		<li><a href="<?php print $globalURL; ?>/pilot/statistics/time/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>" <?php if (strtolower($current_page) == "pilot-statistics-time"){ print 'class="active"'; } ?>><?php echo _("Time"); ?></a></li>
+		<li><a href="<?php print $globalURL; ?>/pilot/statistics/route/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>" <?php if (strtolower($current_page) == "pilot-statistics-route"){ print 'class="active"'; } ?>><?php echo _("Route"); ?></a></li>
+		<li><a href="<?php print $globalURL; ?>/pilot/statistics/time/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
+	echo '/'.$year;
+}
+?><?php if (isset($month) && $month != '') {
+	echo '/'.$month;
+}
+?>" <?php if (strtolower($current_page) == "pilot-statistics-time"){ print 'class="active"'; } ?>><?php echo _("Time"); ?></a></li>
 	</ul>
 </div>
\ No newline at end of file

Please login to merge, or discard this patch.

route-statistics-registration.php 1 patch

Braces +3 added lines, -1 removed lines patch added patch discarded remove patch

@@ -14,7 +14,9 @@
 block discarded – undo
 $arrival_airport = filter_input(INPUT_GET,'arrival_airport',FILTER_SANITIZE_STRING);
 if (isset($_GET['departure_airport']) && isset($_GET['arrival_airport'])) {
 	$spotter_array = $Spotter->getSpotterDataByRoute($departure_airport, $arrival_airport, "0,1", $sort);
-} else $spotter_array = array();
+} else {
+	$spotter_array = array();
+}
   
 if (!empty($spotter_array))
 {

Please login to merge, or discard this patch.

		@@ -12,8 +12,12 @@
		block discarded – undo
12	12	$year = filter_input(INPUT_GET,'year',FILTER_SANITIZE_NUMBER_INT);
13	13	$month = filter_input(INPUT_GET,'month',FILTER_SANITIZE_NUMBER_INT);
14	14	$filter = array();
15		-if ($year != '') $filter = array_merge($filter,array('year' => $year));
16		-if ($month != '') $filter = array_merge($filter,array('month' => $month));
	15	+if ($year != '') {
	16	+ $filter = array_merge($filter,array('year' => $year));
	17	+}
	18	+if ($month != '') {
	19	+ $filter = array_merge($filter,array('month' => $month));
	20	+}
17	21	$spotter_array = $Spotter->getSpotterDataByPilot($pilot,"0,1", $sort,$filter);
18	22
19	23	if (!empty($spotter_array))

		@@ -40,8 +40,12 @@ discard block
		block discarded – undo
40	40	$year = filter_input(INPUT_GET,'year',FILTER_SANITIZE_NUMBER_INT);
41	41	$month = filter_input(INPUT_GET,'month',FILTER_SANITIZE_NUMBER_INT);
42	42	$filter = array();
43		- if ($year != '') $filter = array_merge($filter,array('year' => $year));
44		- if ($month != '') $filter = array_merge($filter,array('month' => $month));
	43	+ if ($year != '') {
	44	+ $filter = array_merge($filter,array('year' => $year));
	45	+ }
	46	+ if ($month != '') {
	47	+ $filter = array_merge($filter,array('month' => $month));
	48	+ }
45	49	if ($sort != '')
46	50	{
47	51	$spotter_array = $Spotter->getSpotterDataByOwner($owner,$limit_start.",".$absolute_difference, $sort,$filter);
		@@ -59,8 +63,12 @@ discard block
		block discarded – undo
59	63	{
60	64	$title = sprintf(_("Detailed View for %s"),$spotter_array[0]['aircraft_owner']);
61	65	//$ident = $spotter_array[0]['ident'];
62		- if (isset($spotter_array[0]['latitude'])) $latitude = $spotter_array[0]['latitude'];
63		- if (isset($spotter_array[0]['longitude'])) $longitude = $spotter_array[0]['longitude'];
	66	+ if (isset($spotter_array[0]['latitude'])) {
	67	+ $latitude = $spotter_array[0]['latitude'];
	68	+ }
	69	+ if (isset($spotter_array[0]['longitude'])) {
	70	+ $longitude = $spotter_array[0]['longitude'];
	71	+ }
64	72	require_once('header.php');
65	73	/*
66	74	if (isset($globalArchive) && $globalArchive) {
		@@ -115,9 +123,14 @@ discard block
		block discarded – undo
115	123	print '<h1>'.$spotter_array[0]['aircraft_owner'].'</h1>';
116	124	//print '<div><span class="label">'._("Owner").'</span>'.$spotter_array[0]['aircraft_owner'].'</div>';
117	125	$Stats = new Stats();
118		- if ($year == '' && $month == '') $flights = $Stats->getStatsOwner($owner);
119		- else $flights = 0;
120		- if ($flights == 0) $flights = $Spotter->countFlightsByOwner($owner,$filter);
	126	+ if ($year == '' && $month == '') {
	127	+ $flights = $Stats->getStatsOwner($owner);
	128	+ } else {
	129	+ $flights = 0;
	130	+ }
	131	+ if ($flights == 0) {
	132	+ $flights = $Spotter->countFlightsByOwner($owner,$filter);
	133	+ }
121	134	print '<div><span class="label">'._("Flights").'</span>'.$flights.'</div>';
122	135	$aircraft_type = count($Spotter->countAllAircraftTypesByOwner($owner,$filter));
123	136	print '<div><span class="label">'._("Aircrafts type").'</span>'.$aircraft_type.'</div>';
		@@ -128,7 +141,9 @@ discard block
		block discarded – undo
128	141	$airlines = count($Spotter->countAllAirlinesByOwner($owner,$filter));
129	142	print '<div><span class="label">'._("Airlines").'</span>'.$airlines.'</div>';
130	143	$duration = $Spotter->getFlightDurationByOwner($owner,$filter);
131		- if ($duration != '0') print '<div><span class="label">'._("Total flights spotted duration").'</span>'.$duration.'</div>';
	144	+ if ($duration != '0') {
	145	+ print '<div><span class="label">'._("Total flights spotted duration").'</span>'.$duration.'</div>';
	146	+ }
132	147	print '</div>';
133	148
134	149	include('owner-sub-menu.php');

		@@ -26,7 +26,9 @@ discard block
		block discarded – undo
26	26	if (isset($filter[0]['source'])) {
27	27	$filters = array_merge($filters,$filter);
28	28	}
29		- if (is_array($globalFilter)) $filter = array_merge($filter,$globalFilter);
	29	+ if (is_array($globalFilter)) {
	30	+ $filter = array_merge($filter,$globalFilter);
	31	+ }
30	32	$filter_query_join = '';
31	33	$filter_query_where = '';
32	34	foreach($filters as $flt) {
		@@ -99,8 +101,11 @@ discard block
		block discarded – undo
99	101	}
100	102	$filter_query_join .= " INNER JOIN (SELECT flightaware_id FROM spotter_archive_output".preg_replace('/^ AND/',' WHERE',$filter_query_date).") sd ON sd.flightaware_id = spotter_archive_output.flightaware_id";
101	103	}
102		- if ($filter_query_where == '' && $where) $filter_query_where = ' WHERE';
103		- elseif ($filter_query_where != '' && $and) $filter_query_where .= ' AND';
	104	+ if ($filter_query_where == '' && $where) {
	105	+ $filter_query_where = ' WHERE';
	106	+ } elseif ($filter_query_where != '' && $and) {
	107	+ $filter_query_where .= ' AND';
	108	+ }
104	109	if ($filter_query_where != '') {
105	110	$filter_query_where = preg_replace('/^ AND/',' WHERE',$filter_query_where);
106	111	}
		@@ -114,10 +119,17 @@ discard block
		block discarded – undo
114	119	if ($over_country == '') {
115	120	$Spotter = new Spotter($this->db);
116	121	$data_country = $Spotter->getCountryFromLatitudeLongitude($latitude,$longitude);
117		- if (!empty($data_country)) $country = $data_country['iso2'];
118		- else $country = '';
119		- } else $country = $over_country;
120		- if ($airline_type === NULL) $airline_type ='';
	122	+ if (!empty($data_country)) {
	123	+ $country = $data_country['iso2'];
	124	+ } else {
	125	+ $country = '';
	126	+ }
	127	+ } else {
	128	+ $country = $over_country;
	129	+ }
	130	+ if ($airline_type === NULL) {
	131	+ $airline_type ='';
	132	+ }
121	133
122	134	//if ($country == '') echo "\n".'********** UNKNOW COUNTRY **************'."\n";
123	135	//else echo "\n".'/////// Country : '.$country.' /////////'."\n";
		@@ -620,7 +632,9 @@ discard block
		block discarded – undo
620	632	$additional_query .= "(spotter_archive_output.pilot_name like '%".$q_item."%') OR ";
621	633	$additional_query .= "(spotter_archive_output.ident like '%".$q_item."%') OR ";
622	634	$translate = $Translation->ident2icao($q_item);
623		- if ($translate != $q_item) $additional_query .= "(spotter_archive_output.ident like '%".$translate."%') OR ";
	635	+ if ($translate != $q_item) {
	636	+ $additional_query .= "(spotter_archive_output.ident like '%".$translate."%') OR ";
	637	+ }
624	638	$additional_query .= "(spotter_archive_output.highlight like '%".$q_item."%')";
625	639	$additional_query .= ")";
626	640	}
		@@ -838,7 +852,9 @@ discard block
		block discarded – undo
838	852	date_default_timezone_set($globalTimezone);
839	853	$datetime = new DateTime();
840	854	$offset = $datetime->format('P');
841		- } else $offset = '+00:00';
	855	+ } else {
	856	+ $offset = '+00:00';
	857	+ }
842	858
843	859
844	860	if ($date_array[1] != "")
		@@ -1114,9 +1130,13 @@ discard block
		block discarded – undo
1114	1130	$query .= "AND date < CURRENT_TIMESTAMP AT TIME ZONE 'UTC' - INTERVAL '".$olderthanmonths." MONTHS'";
1115	1131	}
1116	1132	}
1117		- if ($sincedate != '') $query .= "AND date > '".$sincedate."' ";
	1133	+ if ($sincedate != '') {
	1134	+ $query .= "AND date > '".$sincedate."' ";
	1135	+ }
1118	1136	$query .= "GROUP BY c.name, c.iso3, c.iso2 ORDER BY nb DESC";
1119		- if ($limit) $query .= " LIMIT 0,10";
	1137	+ if ($limit) {
	1138	+ $query .= " LIMIT 0,10";
	1139	+ }
1120	1140
1121	1141
1122	1142	$sth = $this->db->prepare($query);
		@@ -1160,9 +1180,13 @@ discard block
		block discarded – undo
1160	1180	$query .= "AND s.date < CURRENT_TIMESTAMP AT TIME ZONE 'UTC' - INTERVAL '".$olderthanmonths." MONTHS'";
1161	1181	}
1162	1182	}
1163		- if ($sincedate != '') $query .= "AND s.date > '".$sincedate."' ";
	1183	+ if ($sincedate != '') {
	1184	+ $query .= "AND s.date > '".$sincedate."' ";
	1185	+ }
1164	1186	$query .= "GROUP BY o.airline_icao,c.name, c.iso3, c.iso2 ORDER BY nb DESC";
1165		- if ($limit) $query .= " LIMIT 0,10";
	1187	+ if ($limit) {
	1188	+ $query .= " LIMIT 0,10";
	1189	+ }
1166	1190
1167	1191
1168	1192	$sth = $this->db->prepare($query);

		@@ -14,7 +14,9 @@
		block discarded – undo
14	14	$arrival_airport = filter_input(INPUT_GET,'arrival_airport',FILTER_SANITIZE_STRING);
15	15	if (isset($_GET['departure_airport']) && isset($_GET['arrival_airport'])) {
16	16	$spotter_array = $Spotter->getSpotterDataByRoute($departure_airport, $arrival_airport, "0,1", $sort);
17		-} else $spotter_array = array();
	17	+} else {
	18	+ $spotter_array = array();
	19	+}
18	20
19	21	if (!empty($spotter_array))
20	22	{

		@@ -3,15 +3,39 @@ discard block
		block discarded – undo
3	3	</span>
4	4	<div class="sub-menu sub-menu-container">
5	5	<ul class="nav nav-pills">
6		- <li><a href="<?php print $globalURL; ?>/pilot/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>" <?php if (strtolower($current_page) == "pilot-detailed"){ print 'class="active"'; } ?>><?php echo _("Detailed"); ?></a></li>
	6	+ <li><a href="<?php print $globalURL; ?>/pilot/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	7	+ echo '/'.$year;
	8	+}
	9	+?><?php if (isset($month) && $month != '') {
	10	+ echo '/'.$month;
	11	+}
	12	+?>" <?php if (strtolower($current_page) == "pilot-detailed"){ print 'class="active"'; } ?>><?php echo _("Detailed"); ?></a></li>
7	13	<li class="dropdown">
8	14	<a class="dropdown-toggle <?php if(strtolower($current_page) == "pilot-statistics-aircraft" \|\| strtolower($current_page) == "pilot-statistics-registration" \|\| strtolower($current_page) == "pilot-statistics-manufacturer"){ print 'active'; } ?>" data-toggle="dropdown" href="#">
9	15	<?php echo _("Aircraft"); ?> <span class="caret"></span>
10	16	</a>
11	17	<ul class="dropdown-menu" role="menu">
12		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/aircraft/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Aircraft Type"); ?></a></li>
13		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/registration/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Registration"); ?></a></li>
14		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/manufacturer/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Manufacturer"); ?></a></li>
	18	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/aircraft/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	19	+ echo '/'.$year;
	20	+}
	21	+?><?php if (isset($month) && $month != '') {
	22	+ echo '/'.$month;
	23	+}
	24	+?>"><?php echo _("Aircraft Type"); ?></a></li>
	25	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/registration/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	26	+ echo '/'.$year;
	27	+}
	28	+?><?php if (isset($month) && $month != '') {
	29	+ echo '/'.$month;
	30	+}
	31	+?>"><?php echo _("Registration"); ?></a></li>
	32	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/manufacturer/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	33	+ echo '/'.$year;
	34	+}
	35	+?><?php if (isset($month) && $month != '') {
	36	+ echo '/'.$month;
	37	+}
	38	+?>"><?php echo _("Manufacturer"); ?></a></li>
15	39	</ul>
16	40	</li>
17	41	<li class="dropdown">
		@@ -19,13 +43,49 @@ discard block
		block discarded – undo
19	43	<?php echo _("Airport"); ?> <span class="caret"></span>
20	44	</a>
21	45	<ul class="dropdown-menu" role="menu">
22		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Departure Airport"); ?></a></li>
23		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Departure Airport by Country"); ?></a></li>
24		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Arrival Airport"); ?></a></li>
25		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>"><?php echo _("Arrival Airport by Country"); ?></a></li>
	46	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	47	+ echo '/'.$year;
	48	+}
	49	+?><?php if (isset($month) && $month != '') {
	50	+ echo '/'.$month;
	51	+}
	52	+?>"><?php echo _("Departure Airport"); ?></a></li>
	53	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/departure-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	54	+ echo '/'.$year;
	55	+}
	56	+?><?php if (isset($month) && $month != '') {
	57	+ echo '/'.$month;
	58	+}
	59	+?>"><?php echo _("Departure Airport by Country"); ?></a></li>
	60	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	61	+ echo '/'.$year;
	62	+}
	63	+?><?php if (isset($month) && $month != '') {
	64	+ echo '/'.$month;
	65	+}
	66	+?>"><?php echo _("Arrival Airport"); ?></a></li>
	67	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/arrival-airport-country/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	68	+ echo '/'.$year;
	69	+}
	70	+?><?php if (isset($month) && $month != '') {
	71	+ echo '/'.$month;
	72	+}
	73	+?>"><?php echo _("Arrival Airport by Country"); ?></a></li>
26	74	</ul>
27	75	</li>
28		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/route/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>" <?php if (strtolower($current_page) == "pilot-statistics-route"){ print 'class="active"'; } ?>><?php echo _("Route"); ?></a></li>
29		- <li><a href="<?php print $globalURL; ?>/pilot/statistics/time/<?php print $pilot; ?><?php if (isset($year) && $year != '') echo '/'.$year; ?><?php if (isset($month) && $month != '') echo '/'.$month; ?>" <?php if (strtolower($current_page) == "pilot-statistics-time"){ print 'class="active"'; } ?>><?php echo _("Time"); ?></a></li>
	76	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/route/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	77	+ echo '/'.$year;
	78	+}
	79	+?><?php if (isset($month) && $month != '') {
	80	+ echo '/'.$month;
	81	+}
	82	+?>" <?php if (strtolower($current_page) == "pilot-statistics-route"){ print 'class="active"'; } ?>><?php echo _("Route"); ?></a></li>
	83	+ <li><a href="<?php print $globalURL; ?>/pilot/statistics/time/<?php print $pilot; ?><?php if (isset($year) && $year != '') {
	84	+ echo '/'.$year;
	85	+}
	86	+?><?php if (isset($month) && $month != '') {
	87	+ echo '/'.$month;
	88	+}
	89	+?>" <?php if (strtolower($current_page) == "pilot-statistics-time"){ print 'class="active"'; } ?>><?php echo _("Time"); ?></a></li>
30	90	</ul>
31	91	</div>
32	92	\ No newline at end of file

Ysurac / FlightAirMap

Push — master ( e17801...597a61 )

Status

Category

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

Braces +23 added lines, -8 removed lines patch added patch discarded remove patch

Braces +37 added lines, -13 removed lines patch added patch discarded remove patch

Braces +3 added lines, -1 removed lines patch added patch discarded remove patch

Braces +70 added lines, -10 removed lines patch added patch discarded remove patch

Braces +3 added lines, -1 removed lines patch added patch discarded remove patch