Add function param/return types, fix a lot more phpstorm complaints

1 files changed, 9 insertions, 8 deletions

diff --git a/modules-available/locations/pages/details.inc.php b/modules-available/locations/pages/details.inc.php
index e75aaf7d..279eee44 100644
--- a/modules-available/locations/pages/details.inc.php
+++ b/modules-available/locations/pages/details.inc.php
@@ -3,24 +3,25 @@
 class SubPage
 {
 
-	public static function doPreprocess($action)
+	public static function doPreprocess($action): bool
 	{
 		if ($action === 'updatelocation') {
 			self::updateLocation();
 			return true;
-		} else if ($action === 'updateOpeningtimes') {
+		}
+		if ($action === 'updateOpeningtimes') {
 			self::updateOpeningTimes();
 			return true;
 		}
 		return false;
 	}
 
-	public static function doRender($action)
+	public static function doRender($action): bool
 	{
 		return false;
 	}
 
-	public static function doAjax($action)
+	public static function doAjax($action): bool
 	{
 		if ($action === 'showlocation') {
 			self::ajaxShowLocation();
@@ -172,7 +173,7 @@ class SubPage
 		Util::redirect('?do=Locations');
 	}
 
-	private static function updateLocationData($location)
+	private static function updateLocationData(array $location): bool
 	{
 		$locationId = (int)$location['locationid'];
 		$newParent = Request::post('parentlocationid', false, 'integer');
@@ -216,7 +217,7 @@ class SubPage
 		return $newParent != $location['parentlocationid'];
 	}
 
-	private static function updateLocationSubnets()
+	private static function updateLocationSubnets(): bool
 	{
 		$locationId = Request::post('locationid', false, 'integer');
 		if (!User::hasPermission('location.edit.subnets', $locationId))
@@ -257,7 +258,7 @@ class SubPage
 				continue;
 			}
 			$range = LocationUtil::rangeToLongVerbose($start, $end);
-			if ($range === false)
+			if ($range === null)
 				continue;
 			list($startLong, $endLong) = $range;
 			if ($stmt->execute(array('id' => $subnetid, 'start' => $startLong, 'end' => $endLong))) {
@@ -275,7 +276,7 @@ class SubPage
 		return $change;
 	}
 
-	private static function addNewLocationSubnets($location)
+	private static function addNewLocationSubnets(array $location): bool
 	{
 		$locationId = (int)$location['locationid'];
 		if (!User::hasPermission('location.edit.subnets', $locationId))