Ysurac / FlightAirMap

aircraft.php

Braces +9 added lines, -3 removed lines

@@ -16,7 +16,9 @@ 
 	print '<h1>'._("Aircrafts Types").'</h1>';
 
 	$aircraft_types = $Stats->getAllAircraftTypes();
-	if (empty($aircraft_types) || $aircraft_types[0]['aircraft_manufacturer'] == '') $aircraft_types = $Spotter->getAllAircraftTypes();
+	if (empty($aircraft_types) || $aircraft_types[0]['aircraft_manufacturer'] == '') {
+		$aircraft_types = $Spotter->getAllAircraftTypes();
+	}
 	$previous = null;
 	print '<div class="alphabet-legend">';
 	foreach($aircraft_types as $value) {
@@ -24,7 +26,9 @@ 
 		$firstLetter = substr($value['aircraft_manufacturer'], 0, 1);
 		if($previous !== $firstLetter)
 		{
-			if ($previous !== null) print ' | ';
+			if ($previous !== null) {
+				print ' | ';
+			}
 			print '<a href="#'.$firstLetter.'">'.$firstLetter.'</a>';
 		}
 		$previous = $firstLetter;
@@ -38,7 +42,9 @@ 
 		{
 			if($previous !== $firstLetter)
 			{
-				if ($previous !== null) print '</div>';
+				if ($previous !== null) {
+					print '</div>';
+				}
 				print '<a name="'.$firstLetter.'"></a><h4 class="alphabet-header">'.$firstLetter.'</h4><div class="alphabet">';
 			}
 			$previous = $firstLetter;

table-output.php

Braces +45 added lines, -17 removed lines

@@ -339,7 +339,9 @@ 
 	if (isset($globalTimezone))
 	{
 		date_default_timezone_set($globalTimezone);
-	} else date_default_timezone_set('UTC');
+	} else {
+		date_default_timezone_set('UTC');
+	}
 	if ($showSpecial === true)
 	{
 		print '<tr class="special">'."\n";
@@ -358,8 +360,9 @@ 
 			print '<td class="aircraft_thumbnail">'."\n";
 			
 			if ($spotter_item['image_source'] == 'planespotters') {
-				if ($spotter_item['image_source_website'] != '') $image_src = $spotter_item['image_source_website'];
-				else {
+				if ($spotter_item['image_source_website'] != '') {
+					$image_src = $spotter_item['image_source_website'];
+				} else {
 					$planespotter_url_array = explode("_", $spotter_item['image']);
 					$planespotter_id = str_replace(".jpg", "", $planespotter_url_array[1]);
                                         $image_src = 'https://www.planespotters.net/Aviation_Photos/photo.show?id='.$planespotter_id;
@@ -372,7 +375,9 @@ 
                         } else {
                     		if ($spotter_item['image_source'] == 'wikimedia' || $spotter_item['image_source'] == 'devianart' || $spotter_item['image_source'] == 'flickr') {
                     		$image_thumbnail = preg_replace("/^http:/i","https:",$spotter_item['image_thumbnail']);
-                    		} else 	$image_thumbnail = $spotter_item['image_thumbnail'];
+                    		} else {
+                    			$image_thumbnail = $spotter_item['image_thumbnail'];
+                    		}
 				if (isset($spotter_item['airline_name'])) {
 					print '<img src="'.$image_thumbnail.'" class="img-rounded" data-toggle="popover" title="'.$spotter_item['registration'].' - '.$spotter_item['airline_name'].'" alt="'.$spotter_item['registration'].' - '.$spotter_item['airline_name'].'" data-content="'._("Registration:").' '.$spotter_item['registration'].'<br />'._("Airline:").' '.$spotter_item['airline_name'].'" data-html="true" width="100px" />'."\n".'<div class="thumbnail-copyright">&copy; '.$spotter_item['image_copyright'].'</div>';
 				} else {
@@ -395,8 +400,9 @@ 
 			print '<td class="aircraft_thumbnail">'."\n";
 			//print '<a href="'.$globalURL.'/flightid/'.$spotter_item['spotter_id'].'"><img src="'.$spotter_item['image_thumbnail'].'" alt="Click to see more information about this flight" title="Click to see more information about this flight" width="100px" /></a>';
 			if ($spotter_item['image_source'] == 'planespotters') {
-				if ($spotter_item['image_source_website'] != '') $image_src = $spotter_item['image_source_website'];
-				else {
+				if ($spotter_item['image_source_website'] != '') {
+					$image_src = $spotter_item['image_source_website'];
+				} else {
 					$planespotter_url_array = explode("_", $spotter_array[0]['image']);
 					$planespotter_id = str_replace(".jpg", "", $planespotter_url_array[1]);
                                         $image_src = 'https://www.planespotters.net/Aviation_Photos/photo.show?id='.$planespotter_id;
@@ -413,7 +419,9 @@ 
 			} else {
                     		if ($spotter_item['image_source'] == 'wikimedia' || $spotter_item['image_source'] == 'devianart' || $spotter_item['image_source'] == 'flickr') {
                     		$image_thumbnail = preg_replace("/^http:/i","https:",$spotter_item['image_thumbnail']);
-                    		} else 	$image_thumbnail = $spotter_item['image_thumbnail'];
+                    		} else {
+                    			$image_thumbnail = $spotter_item['image_thumbnail'];
+                    		}
 				if (!isset($spotter_item['airline_name']) && isset($spotter_item['aircraft_name'])) {
 					print '<a href="'.$globalURL.'/flightid/'.$spotter_item['spotter_id'].'"><img src="'.$image_thumbnail.'" class="img-rounded" data-toggle="popover" title="'.$spotter_item['registration'].' - '.$spotter_item['aircraft_type'].' - '._("Not available").'" alt="'.$spotter_item['registration'].' - '.$spotter_item['aircraft_type'].' - '._("Not available").'" data-content="'._("Registration:").' '.$spotter_item['registration'].'<br />'._("Aircraft:").' '.$spotter_item['aircraft_name'].' ('.$spotter_item['aircraft_type'].')<br />'._("Airline:").' '._("Not available").'<br />'._("Squawk:").' '.$spotter_item['squawk'].'" data-html="true" width="100px" /></a>'."\n".'<div class="thumbnail-copyright">&copy; '.$spotter_item['image_copyright'].'</div>';
 				} elseif (!isset($spotter_item['aircraft_name']) && isset($spotter_item['airline_name'])) {
@@ -494,15 +502,21 @@ 
 	if (isset($spotter_item['departure_airport_time']) && isset($spotter_item['real_departure_airport_time'])) {
 		if ($spotter_item['departure_airport_time'] > 2460) {
 			$departure_airport_time = date('H:m',$spotter_item['departure_airport_time']);
-		} else $departure_airport_time = substr($spotter_item['departure_airport_time'],0,-2).':'.substr($spotter_item['departure_airport_time'],-2);
+		} else {
+			$departure_airport_time = substr($spotter_item['departure_airport_time'],0,-2).':'.substr($spotter_item['departure_airport_time'],-2);
+		}
 		if ($spotter_item['real_departure_airport_time'] > 2460) {
 			$real_departure_airport_time = date('H:m',$spotter_item['real_departure_airport_time']);
-		} else $real_departure_airport_time = $spotter_item['real_departure_airport_time'];
+		} else {
+			$real_departure_airport_time = $spotter_item['real_departure_airport_time'];
+		}
 		print '<br /><span class="airport_time">'.$departure_airport_time.' ('.$real_departure_airport_time.')</span>'."\n";
 	} elseif (isset($spotter_item['real_departure_airport_time'])) {
 		if ($spotter_item['real_departure_airport_time'] > 2460) {
 			$real_departure_airport_time = date('H:m',$spotter_item['real_departure_airport_time']);
-		} else $real_departure_airport_time = $spotter_item['real_departure_airport_time'];
+		} else {
+			$real_departure_airport_time = $spotter_item['real_departure_airport_time'];
+		}
 		print '<br /><span class="airport_time">'.$real_departure_airport_time.'</span>'."\n";
 	} elseif (isset($spotter_item['departure_airport_time'])) {
 		if ($spotter_item['departure_airport_time'] > 2460) {
@@ -524,7 +538,9 @@ 
 				$longitude = $spotter_item['longitude'];
 			}
 			$distance = $Spotter->getAirportDistance($spotter_item['departure_airport'],$latitude,$longitude);
-		} else $distance = '';
+		} else {
+			$distance = '';
+		}
 		if ($distance != '') {
 		    if ((!isset($_COOKIE['unitdistance']) && isset($globalUnitDistance) && $globalUnitDistance == 'nm') || (isset($_COOKIE['unitdistance']) && $_COOKIE['unitdistance'] == 'nm')) {
 			    echo '<br/><i>'.round($distance*0.539957).' nm</i>';
@@ -549,7 +565,9 @@ 
 	} else {
 		if (isset($spotter_item['real_arrival_airport']) && $spotter_item['real_arrival_airport'] != $spotter_item['arrival_airport']) {
 			print '<span class="nomobile">Scheduled : <a href="'.$globalURL.'/airport/'.$spotter_item['arrival_airport'].'">'.$spotter_item['arrival_airport_city'].', '.$spotter_item['arrival_airport_country'].' ('.$spotter_item['arrival_airport'].')</a></span>'."\n";
-			if (!isset($Spotter)) $Spotter = new Spotter();
+			if (!isset($Spotter)) {
+				$Spotter = new Spotter();
+			}
 			$arrival_airport_info = $Spotter->getAllAirportInfo($spotter_item['arrival_airport']);
 			print '<br /><span class="nomobile">'._("Real:").' <a href="'.$globalURL.'/airport/'.$spotter_item['real_arrival_airport'].'">'.$arrival_airport_info[0]['city'].','.$arrival_airport_info[0]['country'].' ('.$spotter_item['real_arrival_airport'].')</a></span>'."\n";
 			print '<span class="mobile">'._("Scheduled:").' <a href="'.$globalURL.'/airport/'.$spotter_item['real_arrival_airport'].'">'.$spotter_item['real_arrival_airport'].'</a></span>'."\n";
@@ -565,20 +583,28 @@ 
 	if (isset($spotter_item['arrival_airport_time']) && isset($spotter_item['real_arrival_airport_time'])) {
 		if ($spotter_item['arrival_airport_time'] > 2460) {
 			$arrival_airport_time = date('H:m',$spotter_item['arrival_airport_time']);
-		} else $arrival_airport_time = $spotter_item['arrival_airport_time'];
+		} else {
+			$arrival_airport_time = $spotter_item['arrival_airport_time'];
+		}
 		if ($spotter_item['real_arrival_airport_time'] > 2460) {
 			$real_arrival_airport_time = date('H:m',$spotter_item['real_arrival_airport_time']);
-		} else $real_arrival_airport_time = $spotter_item['real_arrival_airport_time'];
+		} else {
+			$real_arrival_airport_time = $spotter_item['real_arrival_airport_time'];
+		}
 		print '<br /><span class="airport_time">'.$spotter_item['arrival_airport_time'].' ('.$spotter_item['real_arrival_airport_time'].')</span>'."\n";
 	} elseif (isset($spotter_item['real_arrival_airport_time'])) {
 		if ($spotter_item['real_arrival_airport_time'] > 2460) {
 			$real_arrival_airport_time = date('H:m',$spotter_item['real_arrival_airport_time']);
-		} else $real_arrival_airport_time = $spotter_item['real_arrival_airport_time'];
+		} else {
+			$real_arrival_airport_time = $spotter_item['real_arrival_airport_time'];
+		}
 		print '<br /><span class="airport_time">'.$real_arrival_airport_time.'</span>'."\n";
 	} elseif (isset($spotter_item['arrival_airport_time'])) {
 		if ($spotter_item['arrival_airport_time'] > 2460) {
 			$arrival_airport_time = date('H:m',$spotter_item['arrival_airport_time']);
-		} else $arrival_airport_time = $spotter_item['arrival_airport_time'];
+		} else {
+			$arrival_airport_time = $spotter_item['arrival_airport_time'];
+		}
 		print '<br /><span class="airport_time">'.$arrival_airport_time.'</span>'."\n";
 	}
 	if (!isset($spotter_item['real_arrival_airport']) && $spotter_item['arrival_airport'] != 'NA') {
@@ -591,7 +617,9 @@ 
 				$longitude = $spotter_item['longitude'];
 			}
 			$distance = $Spotter->getAirportDistance($spotter_item['arrival_airport'],$latitude,$longitude);
-		} else $distance = '';
+		} else {
+			$distance = '';
+		}
 		if ($distance != '') {
 		    if ((!isset($_COOKIE['unitdistance']) && isset($globalUnitDistance) && $globalUnitDistance == 'nm') || (isset($_COOKIE['unitdistance']) && $_COOKIE['unitdistance'] == 'nm')) {
 			    echo '<br/><i>'.round($distance*0.539957).' nm</i>';

footer.php

Braces +4 added lines, -1 removed lines

@@ -9,7 +9,10 @@
 	<?php
 	    }
 	?>
-	<span>Developed in Barrie by <a href="http://mtru.nz/" target="_blank">Mario Trunz</a> & at <a href="http://www.zugaina.com" target="_blank">Zugaina</a> by Ycarus</span> - <span><a href="<?php if (isset($globalURL)) print $globalURL; ?>/about#source">Source &amp; Credits</a></span> - <span><a href="https://github.com/Ysurac/FlightAirMap/issues" target="_blank">Report any issues</a></span>
+	<span>Developed in Barrie by <a href="http://mtru.nz/" target="_blank">Mario Trunz</a> & at <a href="http://www.zugaina.com" target="_blank">Zugaina</a> by Ycarus</span> - <span><a href="<?php if (isset($globalURL)) {
+	print $globalURL;
+}
+?>/about#source">Source &amp; Credits</a></span> - <span><a href="https://github.com/Ysurac/FlightAirMap/issues" target="_blank">Report any issues</a></span>
 </footer>
 
 <div class="notifications bottom-left"></div>

install/class.update_schema.php

Braces +220 added lines, -82 removed lines

@@ -258,7 +258,9 @@ 
     		// Update table countries
     		if ($Connection->tableExists('airspace')) {
     		    $error .= update_db::update_countries();
-		    if ($error != '') return $error;
+		    if ($error != '') {
+		    	return $error;
+		    }
 		}
 		// Update schema_version to 7
 		$query = "UPDATE `config` SET `value` = '7' WHERE `name` = 'schema_version'";
@@ -314,7 +316,9 @@ 
     		$error = '';
     		// Update table aircraft
 		$error .= create_db::import_file('../db/source_location.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		// Update schema_version to 6
 		$query = "UPDATE `config` SET `value` = '8' WHERE `name` = 'schema_version'";
         	try {
@@ -331,7 +335,9 @@ 
     		$error = '';
     		// Update table aircraft
 		$error .= create_db::import_file('../db/notam.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$query = "DELETE FROM config WHERE name = 'last_update_db';
                         INSERT INTO config (name,value) VALUES ('last_update_db',NOW());
                         DELETE FROM config WHERE name = 'last_update_notam_db';
@@ -365,7 +371,9 @@ 
 		$error = '';
     		// Update table atc
 		$error .= create_db::import_file('../db/atc.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		
 		$query = "UPDATE `config` SET `value` = '10' WHERE `name` = 'schema_version'";
         	try {
@@ -389,13 +397,21 @@ 
 		$error = '';
     		// Add tables
 		$error .= create_db::import_file('../db/aircraft_owner.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/metar.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/taf.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/airport.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		
 		$query = "UPDATE `config` SET `value` = '11' WHERE `name` = 'schema_version'";
         	try {
@@ -469,19 +485,33 @@ 
 		$error = '';
     		// Add tables
 		$error .= create_db::import_file('../db/stats.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/stats_aircraft.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/stats_airline.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/stats_airport.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/stats_owner.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/stats_pilot.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$error .= create_db::import_file('../db/spotter_archive_output.sql');
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		
 		$query = "UPDATE `config` SET `value` = '13' WHERE `name` = 'schema_version'";
         	try {
@@ -521,7 +551,9 @@ 
     		// Add tables
     		if (!$Connection->tableExists('stats_flight')) {
 			$error .= create_db::import_file('../db/stats_flight.sql');
-			if ($error != '') return $error;
+			if ($error != '') {
+				return $error;
+			}
 		}
 		$query = "UPDATE `config` SET `value` = '15' WHERE `name` = 'schema_version'";
         	try {
@@ -545,7 +577,9 @@ 
     		} catch(PDOException $e) {
 		    return "error (update stats) : ".$e->getMessage()."\n";
     		}
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$query = "UPDATE `config` SET `value` = '16' WHERE `name` = 'schema_version'";
         	try {
             	    $sth = $Connection->db->prepare($query);
@@ -566,7 +600,9 @@ 
     		if (!$Connection->tableExists('stats_callsign')) {
 			$error .= create_db::import_file('../db/stats_callsign.sql');
 		}
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$query = "UPDATE `config` SET `value` = '17' WHERE `name` = 'schema_version'";
         	try {
             	    $sth = $Connection->db->prepare($query);
@@ -584,7 +620,9 @@ 
     		if (!$Connection->tableExists('stats_country')) {
 			$error .= create_db::import_file('../db/stats_country.sql');
 		}
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$query = "UPDATE `config` SET `value` = '18' WHERE `name` = 'schema_version'";
         	try {
             	    $sth = $Connection->db->prepare($query);
@@ -607,7 +645,9 @@ 
 			    return "error (update stats) : ".$e->getMessage()."\n";
     			}
     		}
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$query = "UPDATE `config` SET `value` = '19' WHERE `name` = 'schema_version'";
         	try {
             	    $sth = $Connection->db->prepare($query);
@@ -623,7 +663,9 @@ 
 		$error = '';
     		// Update airport table
 		$error .= create_db::import_file('../db/airport.sql');
-		if ($error != '') return 'Import airport.sql : '.$error;
+		if ($error != '') {
+			return 'Import airport.sql : '.$error;
+		}
 		// Remove primary key on Spotter_Archive
 		$query = "alter table spotter_archive drop spotter_archive_id";
         	try {
@@ -699,7 +741,9 @@ 
 				return "error (add source_name column) : ".$e->getMessage()."\n";
     			}
     		}
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		$query = "UPDATE `config` SET `value` = '20' WHERE `name` = 'schema_version'";
         	try {
             	    $sth = $Connection->db->prepare($query);
@@ -717,7 +761,9 @@ 
     		// Update airline table
     		if (!$globalIVAO && !$globalVATSIM && !$globalphpVMS) {
 			$error .= create_db::import_file('../db/airlines.sql');
-			if ($error != '') return 'Import airlines.sql : '.$error;
+			if ($error != '') {
+				return 'Import airlines.sql : '.$error;
+			}
 		}
 		if (!$Connection->checkColumnName('aircraft_modes','type_flight')) {
 			// Add column over_country
@@ -729,7 +775,9 @@ 
 				return "error (add over_country) : ".$e->getMessage()."\n";
     			}
     		}
-		if ($error != '') return $error;
+		if ($error != '') {
+			return $error;
+		}
 		/*
     		if (!$globalIVAO && !$globalVATSIM && !$globalphpVMS) {
 			// Force update ModeS (this will put type_flight data
@@ -759,7 +807,9 @@ 
 			} catch(PDOException $e) {
 				return "error (rename type to stats_type on stats*) : ".$e->getMessage()."\n";
 			}
-			if ($error != '') return $error;
+			if ($error != '') {
+				return $error;
+			}
 		}
 		$query = "UPDATE `config` SET `value` = '22' WHERE `name` = 'schema_version'";
         	try {
@@ -782,7 +832,9 @@ 
 			} else {
 				$error .= create_db::import_file('../db/pgsql/stats_source.sql');
 			}
-			if ($error != '') return $error;
+			if ($error != '') {
+				return $error;
+			}
 		}
 		$query = "UPDATE config SET value = '23' WHERE name = 'schema_version'";
         	try {
@@ -804,12 +856,16 @@ 
 		if ($globalDBdriver == 'mysql') {
 			if (!$Connection->tableExists('tle')) {
 				$error .= create_db::import_file('../db/tle.sql');
-				if ($error != '') return $error;
+				if ($error != '') {
+					return $error;
+				}
 			}
 		} else {
 			if (!$Connection->tableExists('tle')) {
 				$error .= create_db::import_file('../db/pgsql/tle.sql');
-				if ($error != '') return $error;
+				if ($error != '') {
+					return $error;
+				}
 			}
 			$query = "create index flightaware_id_idx ON spotter_archive USING btree(flightaware_id)";
 			try {
@@ -849,7 +905,9 @@ 
 		} else {
 			$error .= create_db::import_file('../db/pgsql/airlines.sql');
 		}
-		if ($error != '') return 'Import airlines.sql : '.$error;
+		if ($error != '') {
+			return 'Import airlines.sql : '.$error;
+		}
 		if (!$Connection->checkColumnName('airlines','forsource')) {
 			// Add forsource to airlines
 			$query = "ALTER TABLE airlines ADD forsource VARCHAR(255) NULL DEFAULT NULL";
@@ -1102,8 +1160,11 @@ 
     	    if ($Connection->tableExists('aircraft')) {
     		if (!$Connection->tableExists('config')) {
     		    $version = '1';
-    		    if ($update) return self::update_from_1();
-    		    else return $version;
+    		    if ($update) {
+    		    	return self::update_from_1();
+    		    } else {
+    		    	return $version;
+    		    }
 		} else {
     		    $Connection = new Connection();
 		    $query = "SELECT value FROM config WHERE name = 'schema_version' LIMIT 1";
@@ -1117,106 +1178,183 @@ 
     		    if ($update) {
     			if ($result['value'] == '2') {
     			    $error = self::update_from_2();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '3') {
     			    $error = self::update_from_3();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '4') {
     			    $error = self::update_from_4();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '5') {
     			    $error = self::update_from_5();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '6') {
     			    $error = self::update_from_6();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '7') {
     			    $error = self::update_from_7();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '8') {
     			    $error = self::update_from_8();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '9') {
     			    $error = self::update_from_9();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '10') {
     			    $error = self::update_from_10();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '11') {
     			    $error = self::update_from_11();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '12') {
     			    $error = self::update_from_12();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '13') {
     			    $error = self::update_from_13();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '14') {
     			    $error = self::update_from_14();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '15') {
     			    $error = self::update_from_15();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '16') {
     			    $error = self::update_from_16();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '17') {
     			    $error = self::update_from_17();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '18') {
     			    $error = self::update_from_18();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '19') {
     			    $error = self::update_from_19();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '20') {
     			    $error = self::update_from_20();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '21') {
     			    $error = self::update_from_21();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '22') {
     			    $error = self::update_from_22();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '23') {
     			    $error = self::update_from_23();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '24') {
     			    $error = self::update_from_24();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
     			} elseif ($result['value'] == '25') {
     			    $error = self::update_from_25();
-    			    if ($error != '') return $error;
-    			    else return self::check_version(true);
-    			} else return '';
+    			    if ($error != '') {
+    			    	return $error;
+    			    } else {
+    			    	return self::check_version(true);
+    			    }
+    			} else {
+    				return '';
+    			}
+    		    } else {
+    		    	return $result['value'];
     		    }
-    		    else return $result['value'];
 		}
 		
-	    } else return $version;
+	    } else {
+	    	return $version;
+	    }
     	}
     	
 }

require/class.NOTAM.php

Braces +208 added lines, -88 removed lines

@@ -1015,8 +1015,11 @@ 
 		$data = file_get_contents($filename);
 		preg_match_all("/%%(.+?)%%/is", $data, $matches);
 		//print_r($matches);
-		if (isset($matches[1])) return $matches[1];
-		else return array();
+		if (isset($matches[1])) {
+			return $matches[1];
+		} else {
+			return array();
+		}
 	}
 	public function getAllNOTAMbyScope($scope) {
 		global $globalDBdriver;
@@ -1043,7 +1046,9 @@ 
 			$minlat = filter_var($coord[1],FILTER_SANITIZE_NUMBER_FLOAT,FILTER_FLAG_ALLOW_FRACTION);
 			$maxlong = filter_var($coord[2],FILTER_SANITIZE_NUMBER_FLOAT,FILTER_FLAG_ALLOW_FRACTION);
 			$maxlat = filter_var($coord[3],FILTER_SANITIZE_NUMBER_FLOAT,FILTER_FLAG_ALLOW_FRACTION);
-		} else return array();
+		} else {
+			return array();
+		}
 		if ($globalDBdriver == 'mysql') {
 			$query  = 'SELECT * FROM notam WHERE center_latitude BETWEEN '.$minlat.' AND '.$maxlat.' AND center_longitude BETWEEN '.$minlong.' AND '.$maxlong.' AND radius > 0 AND date_end > UTC_TIMESTAMP() AND date_begin < UTC_TIMESTAMP()';
 		} else {
@@ -1067,7 +1072,9 @@ 
 			$minlat = filter_var($coord[1],FILTER_SANITIZE_NUMBER_FLOAT,FILTER_FLAG_ALLOW_FRACTION);
 			$maxlong = filter_var($coord[2],FILTER_SANITIZE_NUMBER_FLOAT,FILTER_FLAG_ALLOW_FRACTION);
 			$maxlat = filter_var($coord[3],FILTER_SANITIZE_NUMBER_FLOAT,FILTER_FLAG_ALLOW_FRACTION);
-		} else return array();
+		} else {
+			return array();
+		}
 		if ($globalDBdriver == 'mysql') {
 			$query  = 'SELECT * FROM notam WHERE center_latitude BETWEEN '.$minlat.' AND '.$maxlat.' AND center_longitude BETWEEN '.$minlong.' AND '.$maxlong.' AND radius > 0 AND date_end > UTC_TIMESTAMP() AND date_begin < UTC_TIMESTAMP() AND scope = :scope';
 		} else {
@@ -1094,8 +1101,11 @@ 
 			return "error : ".$e->getMessage();
 		}
 		$all = $sth->fetchAll(PDO::FETCH_ASSOC);
-		if (isset($all[0])) return $all[0];
-		else return array();
+		if (isset($all[0])) {
+			return $all[0];
+		} else {
+			return array();
+		}
 	}
 
 	public function addNOTAM($ref,$title,$type,$fir,$code,$rules,$scope,$lower_limit,$upper_limit,$center_latitude,$center_longitude,$radius,$date_begin,$date_end,$permanent,$text,$full_notam) {
@@ -1173,7 +1183,9 @@ 
 					foreach ($alldata as $initial_data) {
 						$data = $this->parse($initial_data);
 						$notamref = $this->getNOTAMbyRef($data['ref']);
-						if (count($notamref) == 0) $this->addNOTAM($data['ref'],$data['title'],'',$data['fir'],$data['code'],'',$data['scope'],$data['lower_limit'],$data['upper_limit'],$data['latitude'],$data['longitude'],$data['radius'],$data['date_begin'],$data['date_end'],$data['permanent'],$data['text'],$data['full_notam']);
+						if (count($notamref) == 0) {
+							$this->addNOTAM($data['ref'],$data['title'],'',$data['fir'],$data['code'],'',$data['scope'],$data['lower_limit'],$data['upper_limit'],$data['latitude'],$data['longitude'],$data['radius'],$data['date_begin'],$data['date_end'],$data['permanent'],$data['text'],$data['full_notam']);
+						}
 					}
 				}
 			}
@@ -1186,7 +1198,9 @@ 
 			foreach ($alldata as $initial_data) {
 				$data = $this->parse($initial_data);
 				$notamref = $this->getNOTAMbyRef($data['ref']);
-				if (!isset($notamref['notam_id'])) $this->addNOTAM($data['ref'],$data['title'],'',$data['fir'],$data['code'],'',$data['scope'],$data['lower_limit'],$data['upper_limit'],$data['latitude'],$data['longitude'],$data['radius'],$data['date_begin'],$data['date_end'],$data['permanent'],$data['text'],$data['full_notam']);
+				if (!isset($notamref['notam_id'])) {
+					$this->addNOTAM($data['ref'],$data['title'],'',$data['fir'],$data['code'],'',$data['scope'],$data['lower_limit'],$data['upper_limit'],$data['latitude'],$data['longitude'],$data['radius'],$data['date_begin'],$data['date_end'],$data['permanent'],$data['text'],$data['full_notam']);
+				}
 			}
 		}
 	}
@@ -1198,11 +1212,15 @@ 
 		foreach (array_chunk($allairports,20) as $airport) {
 			$airports_icao = array();
 			foreach($airport as $icao) {
-				if (isset($icao['icao'])) $airports_icao[] = $icao['icao'];
+				if (isset($icao['icao'])) {
+					$airports_icao[] = $icao['icao'];
+				}
 			}
 			$airport_icao = implode(',',$airports_icao);
 			$alldata = $this->downloadNOTAM($airport_icao);
-			if ($globalTransaction) $this->db->beginTransaction();
+			if ($globalTransaction) {
+				$this->db->beginTransaction();
+			}
 			if (count($alldata) > 0) {
 				foreach ($alldata as $initial_data) {
 					//print_r($initial_data);
@@ -1211,14 +1229,23 @@ 
 					if (isset($data['ref'])) {
 						$notamref = $this->getNOTAMbyRef($data['ref']);
 						if (count($notamref) == 0) {
-							if (isset($data['ref_replaced'])) $this->deleteNOTAMbyRef($data['ref_replaced']);
-							if (isset($data['ref_cancelled'])) $this->deleteNOTAMbyRef($data['ref_cancelled']);
-							elseif (isset($data['latitude']) && isset($data['scope']) && isset($data['text']) && isset($data['permanent'])) echo $this->addNOTAM($data['ref'],'','',$data['fir'],$data['code'],'',$data['scope'],$data['lower_limit'],$data['upper_limit'],$data['latitude'],$data['longitude'],$data['radius'],$data['date_begin'],$data['date_end'],$data['permanent'],$data['text'],$data['full_notam']);
+							if (isset($data['ref_replaced'])) {
+								$this->deleteNOTAMbyRef($data['ref_replaced']);
+							}
+							if (isset($data['ref_cancelled'])) {
+								$this->deleteNOTAMbyRef($data['ref_cancelled']);
+							} elseif (isset($data['latitude']) && isset($data['scope']) && isset($data['text']) && isset($data['permanent'])) {
+								echo $this->addNOTAM($data['ref'],'','',$data['fir'],$data['code'],'',$data['scope'],$data['lower_limit'],$data['upper_limit'],$data['latitude'],$data['longitude'],$data['radius'],$data['date_begin'],$data['date_end'],$data['permanent'],$data['text'],$data['full_notam']);
+							}
 						}
 					}
 				}
-			} else echo 'Error on download. Nothing matches for '.$airport_icao."\n";
-			if ($globalTransaction) $this->db->commit();
+			} else {
+				echo 'Error on download. Nothing matches for '.$airport_icao."\n";
+			}
+			if ($globalTransaction) {
+				$this->db->commit();
+			}
 			sleep(5);
 		}
 	}
@@ -1231,8 +1258,11 @@ 
 		$data = $Common->getData($url);
 		preg_match_all("/<pre>(.+?)<\/pre>/is", $data, $matches);
 		//print_r($matches);
-		if (isset($matches[1])) return $matches[1];
-		else return array();
+		if (isset($matches[1])) {
+			return $matches[1];
+		} else {
+			return array();
+		}
 	}
 
 	public function parse($data) {
@@ -1255,89 +1285,145 @@ 
 					$rules = str_split($matches[3]);
 					foreach ($rules as $rule) {
 						if ($rule == 'I') {
-							if (isset($result['rules'])) $result['rules'] = $result['rules'].'/IFR';
-							else $result['rules'] = 'IFR';
+							if (isset($result['rules'])) {
+								$result['rules'] = $result['rules'].'/IFR';
+							} else {
+								$result['rules'] = 'IFR';
+							}
 						} elseif ($rule == 'V') {
-							if (isset($result['rules'])) $result['rules'] = $result['rules'].'/VFR';
-							else $result['rules'] = 'VFR';
+							if (isset($result['rules'])) {
+								$result['rules'] = $result['rules'].'/VFR';
+							} else {
+								$result['rules'] = 'VFR';
+							}
 						} elseif ($rule == 'K') {
-							if (isset($result['rules'])) $result['rules'] = $result['rules'].'/Checklist';
-							else $result['rules'] = 'Checklist';
+							if (isset($result['rules'])) {
+								$result['rules'] = $result['rules'].'/Checklist';
+							} else {
+								$result['rules'] = 'Checklist';
+							}
 						}
 					}
 					$attentions = str_split($matches[4]);
 					foreach ($attentions as $attention) {
 						if ($attention == 'N') {
-							if (isset($result['attention'])) $result['attention'] = $result['attention'].' / Immediate attention';
-							else $result['rules'] = 'Immediate attention';
+							if (isset($result['attention'])) {
+								$result['attention'] = $result['attention'].' / Immediate attention';
+							} else {
+								$result['rules'] = 'Immediate attention';
+							}
 						} elseif ($attention == 'B') {
-							if (isset($result['attention'])) $result['attention'] = $result['attention'].' / Operational significance';
-							else $result['rules'] = 'Operational significance';
+							if (isset($result['attention'])) {
+								$result['attention'] = $result['attention'].' / Operational significance';
+							} else {
+								$result['rules'] = 'Operational significance';
+							}
 						} elseif ($attention == 'O') {
-							if (isset($result['attention'])) $result['attention'] = $result['attention'].' / Flight operations';
-							else $result['rules'] = 'Flight operations';
+							if (isset($result['attention'])) {
+								$result['attention'] = $result['attention'].' / Flight operations';
+							} else {
+								$result['rules'] = 'Flight operations';
+							}
 						} elseif ($attention == 'M') {
-							if (isset($result['attention'])) $result['attention'] = $result['attention'].' / Misc';
-							else $result['rules'] = 'Misc';
+							if (isset($result['attention'])) {
+								$result['attention'] = $result['attention'].' / Misc';
+							} else {
+								$result['rules'] = 'Misc';
+							}
 						} elseif ($attention == 'K') {
-							if (isset($result['attention'])) $result['attention'] = $result['attention'].' / Checklist';
-							else $result['rules'] = 'Checklist';
+							if (isset($result['attention'])) {
+								$result['attention'] = $result['attention'].' / Checklist';
+							} else {
+								$result['rules'] = 'Checklist';
+							}
 						}
 					}
-					if ($matches[5] == 'A') $result['scope'] = 'Airport warning';
-					elseif ($matches[5] == 'E') $result['scope'] = 'Enroute warning';
-					elseif ($matches[5] == 'W') $result['scope'] = 'Navigation warning';
-					elseif ($matches[5] == 'K') $result['scope'] = 'Checklist';
-					elseif ($matches[5] == 'AE') $result['scope'] = 'Airport/Enroute warning';
-					elseif ($matches[5] == 'AW') $result['scope'] = 'Airport/Navigation warning';
+					if ($matches[5] == 'A') {
+						$result['scope'] = 'Airport warning';
+					} elseif ($matches[5] == 'E') {
+						$result['scope'] = 'Enroute warning';
+					} elseif ($matches[5] == 'W') {
+						$result['scope'] = 'Navigation warning';
+					} elseif ($matches[5] == 'K') {
+						$result['scope'] = 'Checklist';
+					} elseif ($matches[5] == 'AE') {
+						$result['scope'] = 'Airport/Enroute warning';
+					} elseif ($matches[5] == 'AW') {
+						$result['scope'] = 'Airport/Navigation warning';
+					}
 					$result['lower_limit'] = $matches[6];
 					$result['upper_limit'] = $matches[7];
 					$latitude = $Common->convertDec($matches[8],'latitude');
-					if ($matches[9] == 'S') $latitude = -$latitude;
+					if ($matches[9] == 'S') {
+						$latitude = -$latitude;
+					}
 					$longitude = $Common->convertDec($matches[10],'longitude');
-					if ($matches[11] == 'W') $longitude = -$longitude;
+					if ($matches[11] == 'W') {
+						$longitude = -$longitude;
+					}
 					$result['latitude'] = $latitude;
 					$result['longitude'] = $longitude;
 					$result['radius'] = intval($matches[12]);
-				} else echo 'ERROR : '.$line."\n";
-			}
-			elseif (preg_match('#A\) (.*)#',$line,$matches)) {
+				} else {
+					echo 'ERROR : '.$line."\n";
+				}
+			} elseif (preg_match('#A\) (.*)#',$line,$matches)) {
 				$result['icao'] = $matches[1];
-			}
-			elseif (preg_match('#B\) ([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})#',$line,$matches)) {
-				if ($matches[1] > 50) $year = '19'.$matches[1];
-				else $year = '20'.$matches[1];
+			} elseif (preg_match('#B\) ([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})#',$line,$matches)) {
+				if ($matches[1] > 50) {
+					$year = '19'.$matches[1];
+				} else {
+					$year = '20'.$matches[1];
+				}
 				$result['date_begin'] = $year.'/'.$matches[2].'/'.$matches[3].' '.$matches[4].':'.$matches[5];
-			}
-			elseif (preg_match('#C\) ([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})$#',$line,$matches)) {
-				if ($matches[1] > 50) $year = '19'.$matches[1];
-				else $year = '20'.$matches[1];
+			} elseif (preg_match('#C\) ([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})$#',$line,$matches)) {
+				if ($matches[1] > 50) {
+					$year = '19'.$matches[1];
+				} else {
+					$year = '20'.$matches[1];
+				}
 				$result['date_end'] = $year.'/'.$matches[2].'/'.$matches[3].' '.$matches[4].':'.$matches[5];
 				$result['permanent'] = 0;
-			}
-			elseif (preg_match('#C\) ([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2}) (EST|PERM)$#',$line,$matches)) {
-				if ($matches[1] > 50) $year = '19'.$matches[1];
-				else $year = '20'.$matches[1];
+			} elseif (preg_match('#C\) ([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2}) (EST|PERM)$#',$line,$matches)) {
+				if ($matches[1] > 50) {
+					$year = '19'.$matches[1];
+				} else {
+					$year = '20'.$matches[1];
+				}
 				$result['date_end'] = $year.'/'.$matches[2].'/'.$matches[3].' '.$matches[4].':'.$matches[5];
-				if ($matches[6] == 'EST') $result['estimated'] = 1;
-				else $result['estimated'] = 0;
-				if ($matches[6] == 'PERM') $result['permanent'] = 1;
-				else $result['permanent'] = 0;
-			}
-			elseif (preg_match('#C\) (EST|PERM)$#',$line,$matches)) {
+				if ($matches[6] == 'EST') {
+					$result['estimated'] = 1;
+				} else {
+					$result['estimated'] = 0;
+				}
+				if ($matches[6] == 'PERM') {
+					$result['permanent'] = 1;
+				} else {
+					$result['permanent'] = 0;
+				}
+			} elseif (preg_match('#C\) (EST|PERM)$#',$line,$matches)) {
 				$result['date_end'] = '2030/12/20 12:00';
-				if ($matches[1] == 'EST') $result['estimated'] = 1;
-				else $result['estimated'] = 0;
-				if ($matches[1] == 'PERM') $result['permanent'] = 1;
-				else $result['permanent'] = 0;
-			}
-			elseif (preg_match('#E\) (.*)#',$line,$matches)) {
+				if ($matches[1] == 'EST') {
+					$result['estimated'] = 1;
+				} else {
+					$result['estimated'] = 0;
+				}
+				if ($matches[1] == 'PERM') {
+					$result['permanent'] = 1;
+				} else {
+					$result['permanent'] = 0;
+				}
+			} elseif (preg_match('#E\) (.*)#',$line,$matches)) {
 				$rtext = array();
 				$text = explode(' ',$matches[1]);
 				foreach ($text as $word) {
-					if (isset($this->abbr[$word])) $rtext[] = strtoupper($this->abbr[$word]);
-					elseif (ctype_digit(strval(substr($word,3))) && isset($this->abbr[substr($word,0,3)])) $rtext[] = strtoupper($this->abbr[substr($word,0,3)]).' '.substr($word,3);
-					else $rtext[] = $word;
+					if (isset($this->abbr[$word])) {
+						$rtext[] = strtoupper($this->abbr[$word]);
+					} elseif (ctype_digit(strval(substr($word,3))) && isset($this->abbr[substr($word,0,3)])) {
+						$rtext[] = strtoupper($this->abbr[substr($word,0,3)]).' '.substr($word,3);
+					} else {
+						$rtext[] = $word;
+					}
 				}
 				$result['text'] = implode(' ',$rtext);
 			//} elseif (preg_match('#F\) (.*)#',$line,$matches)) {
@@ -1345,7 +1431,9 @@ 
 			} elseif (preg_match('#(NOTAMN|NOTAMR|NOTAMC)#',$line,$matches)) {
 				$text = explode(' ',$line);
 				$result['ref'] = $text[0];
-				if ($matches[1] == 'NOTAMN') $result['type'] = 'new';
+				if ($matches[1] == 'NOTAMN') {
+					$result['type'] = 'new';
+				}
 				if ($matches[1] == 'NOTAMC') {
 					$result['type'] = 'cancel';
 					$result['ref_cancelled'] = $text[2];
@@ -1367,69 +1455,101 @@ 
 		switch ($code[1]) {
 			case 'A':
 				$result = 'Airspace organization ';
-				if (isset($this->code_airspace[$code_fp])) $result .= $this->code_airspace[$code_fp];
+				if (isset($this->code_airspace[$code_fp])) {
+					$result .= $this->code_airspace[$code_fp];
+				}
 				break;
 			case 'C':
 				$result = 'Communications and radar facilities ';
-				if (isset($this->code_comradar[$code_fp])) $result .= $this->code_comradar[$code_fp];
+				if (isset($this->code_comradar[$code_fp])) {
+					$result .= $this->code_comradar[$code_fp];
+				}
 				break;
 			case 'F':
 				$result = 'Facilities and services ';
-				if (isset($this->code_facilities[$code_fp])) $result .= $this->code_facilities[$code_fp];
+				if (isset($this->code_facilities[$code_fp])) {
+					$result .= $this->code_facilities[$code_fp];
+				}
 				break;
 			case 'I':
 				$result = 'Instrument and Microwave Landing System ';
-				if (isset($this->code_instrumentlanding[$code_fp])) $result .= $this->code_instrumentlanding[$code_fp];
+				if (isset($this->code_instrumentlanding[$code_fp])) {
+					$result .= $this->code_instrumentlanding[$code_fp];
+				}
 				break;
 			case 'L':
 				$result = 'Lighting facilities ';
-				if (isset($this->code_lightingfacilities[$code_fp])) $result .= $this->code_lightingfacilities[$code_fp];
+				if (isset($this->code_lightingfacilities[$code_fp])) {
+					$result .= $this->code_lightingfacilities[$code_fp];
+				}
 				break;
 			case 'M':
 				$result = 'Movement and landing areas ';
-				if (isset($this->code_movementareas[$code_fp])) $result .= $this->code_movementareas[$code_fp];
+				if (isset($this->code_movementareas[$code_fp])) {
+					$result .= $this->code_movementareas[$code_fp];
+				}
 				break;
 			case 'N':
 				$result = 'Terminal and En Route Navigation Facilities ';
-				if (isset($this->code_terminalfacilities[$code_fp])) $result .= $this->code_terminalfacilities[$code_fp];
+				if (isset($this->code_terminalfacilities[$code_fp])) {
+					$result .= $this->code_terminalfacilities[$code_fp];
+				}
 				break;
 			case 'O':
 				$result = 'Other information ';
-				if (isset($this->code_information[$code_fp])) $result .= $this->code_information[$code_fp];
+				if (isset($this->code_information[$code_fp])) {
+					$result .= $this->code_information[$code_fp];
+				}
 				break;
 			case 'P':
 				$result = 'Air Traffic procedures ';
-				if (isset($this->code_airtraffic[$code_fp])) $result .= $this->code_airtraffic[$code_fp];
+				if (isset($this->code_airtraffic[$code_fp])) {
+					$result .= $this->code_airtraffic[$code_fp];
+				}
 				break;
 			case 'R':
 				$result = 'Navigation Warnings: Airspace Restrictions ';
-				if (isset($this->code_navigationw[$code_fp])) $result .= $this->code_navigationw[$code_fp];
+				if (isset($this->code_navigationw[$code_fp])) {
+					$result .= $this->code_navigationw[$code_fp];
+				}
 				break;
 			case 'S':
 				$result = 'Air Traffic and VOLMET Services ';
-				if (isset($this->code_volmet[$code_fp])) $result .= $this->code_volmet[$code_fp];
+				if (isset($this->code_volmet[$code_fp])) {
+					$result .= $this->code_volmet[$code_fp];
+				}
 				break;
 			case 'W':
 				$result = 'Navigation Warnings: Warnings ';
-				if (isset($this->code_warnings[$code_fp])) $result .= $this->code_warnings[$code_fp];
+				if (isset($this->code_warnings[$code_fp])) {
+					$result .= $this->code_warnings[$code_fp];
+				}
 				break;
 		}
 		switch ($code[3]) {
 			case 'A':
 				// Availability
-				if (isset($this->code_sp_availabity[$code_sp])) $result .= ' '.$this->code_sp_availabity[$code_sp];
+				if (isset($this->code_sp_availabity[$code_sp])) {
+					$result .= ' '.$this->code_sp_availabity[$code_sp];
+				}
 				break;
 			case 'C':
 				// Changes
-				if (isset($this->code_sp_changes[$code_sp])) $result .= ' '.$this->code_sp_changes[$code_sp];
+				if (isset($this->code_sp_changes[$code_sp])) {
+					$result .= ' '.$this->code_sp_changes[$code_sp];
+				}
 				break;
 			case 'H':
 				// Hazardous conditions
-				if (isset($this->code_sp_hazardous[$code_sp])) $result .= ' '.$this->code_sp_hazardous[$code_sp];
+				if (isset($this->code_sp_hazardous[$code_sp])) {
+					$result .= ' '.$this->code_sp_hazardous[$code_sp];
+				}
 				break;
 			case 'L':
 				// Limitations
-				if (isset($this->code_sp_limitations[$code_sp])) $result .= ' '.$this->code_sp_limitations[$code_sp];
+				if (isset($this->code_sp_limitations[$code_sp])) {
+					$result .= ' '.$this->code_sp_limitations[$code_sp];
+				}
 				break;
 			case 'X':
 				// Other Information

airspace-data.php

Braces +9 added lines, -3 removed lines

@@ -25,7 +25,9 @@ 
 $airspace = $result[0];
 date_default_timezone_set('UTC');
 print '<div class="top">';
-if (isset($airspace['name'])) $airspace['title'] = $airspace['name']; 
+if (isset($airspace['name'])) {
+	$airspace['title'] = $airspace['name'];
+}
 print '<div class="right"><div class="callsign-details"><div class="callsign">'.$airspace['title'].'</a></div>';
 print '</div>';
 print '<div class="details">';
@@ -44,13 +46,17 @@ 
 	print '</div>';
 }
 
-if (isset($airspace['ceiling'])) $airspace['tops'] = $airspace['ceiling'];
+if (isset($airspace['ceiling'])) {
+	$airspace['tops'] = $airspace['ceiling'];
+}
 print '<div>';
 print '<span>'._("Tops").'</span>';
 print $airspace['tops'];
 print '</div>';
 
-if (isset($airspace['floor'])) $airspace['base'] = $airspace['floor'];
+if (isset($airspace['floor'])) {
+	$airspace['base'] = $airspace['floor'];
+}
 print '<div>';
 print '<span>'._("Base").'</span>';
 print $airspace['base'];

airspace-geojson.php

Braces +16 added lines, -6 removed lines

@@ -67,12 +67,22 @@
 		} else {
 			$geom = geoPHP::load(stream_get_contents($row['wkb']));
 		}
-		if (isset($properties['type'])) $properties['type'] = trim($properties['type']);
-		elseif (isset($properties['class'])) $properties['type'] = trim($properties['class']);
-		if (isset($properties['ogr_fid'])) $properties['id'] = $properties['ogr_fid'];
-		elseif (isset($properties['ogc_fid'])) $properties['id'] = $properties['ogc_fid'];
-		if (isset($properties['ceiling'])) $properties['tops'] = $properties['ceiling'];
-		if (isset($properties['floor'])) $properties['base'] = $properties['floor'];
+		if (isset($properties['type'])) {
+			$properties['type'] = trim($properties['type']);
+		} elseif (isset($properties['class'])) {
+			$properties['type'] = trim($properties['class']);
+		}
+		if (isset($properties['ogr_fid'])) {
+			$properties['id'] = $properties['ogr_fid'];
+		} elseif (isset($properties['ogc_fid'])) {
+			$properties['id'] = $properties['ogc_fid'];
+		}
+		if (isset($properties['ceiling'])) {
+			$properties['tops'] = $properties['ceiling'];
+		}
+		if (isset($properties['floor'])) {
+			$properties['base'] = $properties['floor'];
+		}
 		if (preg_match('/^FL(\s)*(?<alt>\d+)/',strtoupper($properties['tops']),$matches)) {
 			$properties['upper_limit'] = round($matches['alt']*100*0.38048);
 		} elseif (preg_match('/^(?<alt>\d+)(\s)*(FT|AGL|ALT|MSL)/',strtoupper($properties['tops']),$matches)) {

scripts/update_db.php

Braces +20 added lines, -8 removed lines

@@ -9,7 +9,9 @@ 
 if (strtoupper(substr(PHP_OS, 0, 3)) != 'WIN') {
 	exec("ps ux", $output, $result);
 	$j = 0;
-	foreach ($output as $line) if(strpos($line, "update_db.php") && !strpos($line, "sh ")) $j++;
+	foreach ($output as $line) {
+		if(strpos($line, "update_db.php") && !strpos($line, "sh ")) $j++;
+	}
 	if ($j > 1) {
 		echo "Script is already runnning...";
 		die();
@@ -28,8 +30,10 @@ 
 			$update_db->update_notam();
 		}
 		$update_db->insert_last_notam_update();
-	} elseif (isset($globalDebug) && $globalDebug && isset($globalNOTAM) && $globalNOTAM) echo "NOTAM are only updated once a day.\n";
-}
+	} elseif (isset($globalDebug) && $globalDebug && isset($globalNOTAM) && $globalNOTAM) {
+		echo "NOTAM are only updated once a day.\n";
+	}
+	}
 
 if ($update_db->check_last_update() && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM) && (!isset($globalphpVMS) || !$globalphpVMS)) {
 	$update_db->update_all();
@@ -37,7 +41,9 @@ 
 //	$Spotter = new Spotter();
 //	$Spotter->updateFieldsFromOtherTables();
 	$update_db->insert_last_update();
-} elseif (isset($globalDebug) && $globalDebug && (!isset($globalphpVMS) || !$globalphpVMS) && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM)) echo "DB are populated with external data only every 15 days ! Files are not updated more often.\n";
+} elseif (isset($globalDebug) && $globalDebug && (!isset($globalphpVMS) || !$globalphpVMS) && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM)) {
+	echo "DB are populated with external data only every 15 days ! Files are not updated more often.\n";
+}
 
 
 if (isset($globalMETAR) && isset($globalMETARcycle) && $globalMETAR && $globalMETARcycle) {
@@ -47,22 +53,28 @@ 
 	if ($METAR->check_last_update()) {
 		$METAR->addMETARCycle();
 		$METAR->insert_last_update();
-	} else echo "METAR are only updated every 30 minutes.\n";
-}
+	} else {
+		echo "METAR are only updated every 30 minutes.\n";
+	}
+	}
 
 
 if (isset($globalOwner) && $globalOwner && $update_db->check_last_owner_update() && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM) && (!isset($globalphpVMS) || !$globalphpVMS)) {
 	echo "Updating private aircraft's owners...";
 	$update_db->update_owner();
 	$update_db->insert_last_owner_update();
-} elseif (isset($globalDebug) && $globalDebug && isset($globalOwner) && $globalOwner && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM) && (!isset($globalphpVMS) || !$globalphpVMS)) echo "Owner are only updated every 15 days.\n";
+} elseif (isset($globalDebug) && $globalDebug && isset($globalOwner) && $globalOwner && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM) && (!isset($globalphpVMS) || !$globalphpVMS)) {
+	echo "Owner are only updated every 15 days.\n";
+}
 
 if (isset($globalSchedules) && $globalSchedules && $update_db->check_last_schedules_update() && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM) && (!isset($globalphpVMS) || !$globalphpVMS)) {
 	echo "Updating schedules...";
 	$update_db->update_oneworld();
 	$update_db->update_skyteam();
 	$update_db->insert_last_schedules_update();
-} elseif (isset($globalDebug) && $globalDebug && isset($globalOwner) && $globalOwner && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM) && (!isset($globalphpVMS) || !$globalphpVMS)) echo "Schedules are only updated every 15 days.\n";
+} elseif (isset($globalDebug) && $globalDebug && isset($globalOwner) && $globalOwner && (!isset($globalIVAO) || !$globalIVAO) && (!isset($globalVATSIM) || !$globalVATSIM) && (!isset($globalphpVMS) || !$globalphpVMS)) {
+	echo "Schedules are only updated every 15 days.\n";
+}
 
 if (isset($globalArchiveMonths) && $globalArchiveMonths > 0) {
 	echo "Updating statistics and archive old data...";

install/class.create_db.php

Braces +12 added lines, -4 removed lines

@@ -14,7 +14,9 @@ 
 			//foreach ($lines as $line)
 			while (($line = fgets($handle,4096)) !== false)
 			{
-				if (substr($line,0,2) == '--' || $line == '') continue;
+				if (substr($line,0,2) == '--' || $line == '') {
+					continue;
+				}
 				$templine .= $line;
 				if (substr(trim($line), -1,1) == ';')
 				{
@@ -40,7 +42,9 @@ 
 		//foreach (new RecursiveIteratorIterator(new RecursiveDirectoryIterator($directory)) as $filename)
 		while(false !== ($filename = readdir($dh)))
 		{
-		    if (preg_match('/\.sql$/',$filename)) $error .= create_db::import_file($directory.$filename);
+		    if (preg_match('/\.sql$/',$filename)) {
+		    	$error .= create_db::import_file($directory.$filename);
+		    }
 		}
 		return $error;
 	}
@@ -56,13 +60,17 @@ 
 		// Dirty hack
 		if ($host != 'localhost' && $host != '127.0.0.1') {
 		    $grantright = $_SERVER['SERVER_ADDR'];
-		} else $grantright = 'localhost';
+		} else {
+			$grantright = 'localhost';
+		}
 		try {
 			$dbh = new PDO($db_type.':host='.$host,$root,$root_pass);
 			$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 			if ($db_type == 'mysql') {
 				$dbh->exec('CREATE DATABASE IF NOT EXISTS `'.$db.'`;GRANT ALL ON `'.$db."`.* TO '".$user."'@'".$grantright."' IDENTIFIED BY '".$password."';FLUSH PRIVILEGES;");
-				if ($grantright == 'localhost') $dbh->exec('GRANT ALL ON `'.$db."`.* TO '".$user."'@'127.0.0.1' IDENTIFIED BY '".$password."';FLUSH PRIVILEGES;");
+				if ($grantright == 'localhost') {
+					$dbh->exec('GRANT ALL ON `'.$db."`.* TO '".$user."'@'127.0.0.1' IDENTIFIED BY '".$password."';FLUSH PRIVILEGES;");
+				}
 			} else if ($db_type == 'pgsql') {
 				$dbh->exec("CREATE DATABASE ".$db.";");
 				$dbh->exec("CREATE USER ".$user." WITH PASSWORD '".$password."';