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

1 files changed, 4 insertions, 2 deletions

diff --git a/modules-available/vmstore/inc/vmstorebenchmark.inc.php b/modules-available/vmstore/inc/vmstorebenchmark.inc.php
index b13f4e37..b819ef8a 100644
--- a/modules-available/vmstore/inc/vmstorebenchmark.inc.php
+++ b/modules-available/vmstore/inc/vmstorebenchmark.inc.php
@@ -24,13 +24,12 @@ class VmStoreBenchmark
 	}
 
 	/**
-	 * @param array $machineUuids
 	 * @param string $image relative path/name of image
 	 * @param string $serverOrMode IP address of DNBD3 server, OR 'auto' for all servers known to client, or 'nfs' for NFS
 	 * @param int $start timestamp when the clients should start
 	 * @return ?string taskId, or null on error
 	 */
-	public static function start(string $id, array $machineUuids, string $image, string $serverOrMode, int &$start)
+	public static function start(string $id, array $machineUuids, string $image, string $serverOrMode, int &$start): ?string
 	{
 		Module::isAvailable('rebootcontrol');
 		$clients = Database::queryAll('SELECT machineuuid, clientip FROM machine WHERE machineuuid IN (:uuids)',
@@ -68,6 +67,9 @@ COMMAND;
 		return $task['id'] ?? null;
 	}
 
+	/**
+	 * @return array{cpu: array, net: array}
+	 */
 	public static function parseBenchLine(string $line): array
 	{
 		$out = ['cpu' => [], 'net' => []];