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

2 files changed, 1 insertions, 12 deletions

diff --git a/modules-available/rebootcontrol/pages/subnet.inc.php b/modules-available/rebootcontrol/pages/subnet.inc.php
index c1631cbd..3e593505 100644
--- a/modules-available/rebootcontrol/pages/subnet.inc.php
+++ b/modules-available/rebootcontrol/pages/subnet.inc.php
@@ -24,7 +24,7 @@ class SubPage
 		User::assertPermission('subnet.edit');
 		$cidr = Request::post('cidr', Request::REQUIRED, 'string');
 		$range = IpUtil::parseCidr($cidr);
-		if ($range === false) {
+		if ($range === null) {
 			Message::addError('invalid-cidr', $cidr);
 			return;
 		}
diff --git a/modules-available/rebootcontrol/pages/task.inc.php b/modules-available/rebootcontrol/pages/task.inc.php
index dc50bfff..7e1f6fbf 100644
--- a/modules-available/rebootcontrol/pages/task.inc.php
+++ b/modules-available/rebootcontrol/pages/task.inc.php
@@ -145,14 +145,3 @@ class SubPage
 	}
 
 }
-
-
-// Remove when we require >= 7.3.0
-if (!function_exists('array_key_first')) {
-	function array_key_first(array $arr) {
-		foreach($arr as $key => $unused) {
-			return $key;
-		}
-		return NULL;
-	}
-}