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

1 files changed, 22 insertions, 25 deletions

diff --git a/modules-available/minilinux/inc/minilinux.inc.php b/modules-available/minilinux/inc/minilinux.inc.php
index daae7048..9c1b9011 100644
--- a/modules-available/minilinux/inc/minilinux.inc.php
+++ b/modules-available/minilinux/inc/minilinux.inc.php
@@ -19,7 +19,7 @@ class MiniLinux
 	 * Query all known sources for metadata
 	 * @return int number of sources query was just initialized for
 	 */
-	public static function updateList()
+	public static function updateList(): int
 	{
 		$stamp = time();
 		$last = Property::get(self::PROPERTY_KEY_FETCHTIME);
@@ -49,10 +49,11 @@ class MiniLinux
 
 	/**
 	 * Called when downloading metadata from a specific update source is finished
-	 * @param mixed $task task structure
+	 *
+	 * @param array $task task structure
 	 * @param string $sourceid see minilinux_source table
 	 */
-	public static function listDownloadCallback($task, $sourceid)
+	public static function listDownloadCallback(array $task, string $sourceid): void
 	{
 		if (!Taskmanager::isFinished($task))
 			return;
@@ -177,7 +178,7 @@ class MiniLinux
 		]);
 	}
 
-	private static function isValidIdPart($str)
+	private static function isValidIdPart(string $str): bool
 	{
 		return preg_match('/^[a-z0-9_\-]+$/', $str) > 0;
 	}
@@ -186,10 +187,10 @@ class MiniLinux
 	 * Download of specific version
 	 */
 
-	public static function validateDownloadTask($versionid, $taskid)
+	public static function validateDownloadTask(string $versionid, ?string $taskid): ?string
 	{
 		if ($taskid === null)
-			return false;
+			return null;
 		$task = Taskmanager::status($taskid);
 		if (Taskmanager::isTask($task) && !Taskmanager::isFailed($task)
 				&& (is_dir(CONFIG_HTTP_DIR . '/' . $versionid) || !Taskmanager::isFinished($task)))
@@ -197,15 +198,13 @@ class MiniLinux
 		Database::exec('UPDATE minilinux_version SET taskid = NULL
 				WHERE versionid = :versionid AND taskid = :taskid',
 			['versionid' => $versionid, 'taskid' => $taskid]);
-		return false;
+		return null;
 	}
 
 	/**
 	 * Download the files for the given version id
-	 * @param $versionid
-	 * @return bool
 	 */
-	public static function downloadVersion($versionid)
+	public static function downloadVersion(string $versionid): ?string
 	{
 		$ver = Database::queryFirst('SELECT s.url, s.pubkey, v.versionid, v.taskid, v.data FROM minilinux_version v
 			INNER JOIN minilinux_branch b USING (branchid)
@@ -213,17 +212,17 @@ class MiniLinux
 			WHERE versionid = :versionid',
 			['versionid' => $versionid]);
 		if ($ver === false)
-			return false;
+			return null;
 		$taskid = self::validateDownloadTask($versionid, $ver['taskid']);
-		if ($taskid !== false)
+		if ($taskid !== null)
 			return $taskid;
 		$data = json_decode($ver['data'], true);
 		if (!is_array($data)) {
 			EventLog::warning("Cannot download Linux '$versionid': Corrupted meta data.", $ver['data']);
-			return false;
+			return null;
 		}
 		if (empty($data['files']))
-			return false;
+			return null;
 		$list = [];
 		$legacyDir = preg_replace(',^[^/]*/,', '', $versionid);
 		foreach ($data['files'] as $file) {
@@ -242,6 +241,7 @@ class MiniLinux
 		Database::exec('LOCK TABLES minilinux_version WRITE');
 		$aff = Database::exec('UPDATE minilinux_version SET taskid = :taskid WHERE versionid = :versionid AND taskid IS NULL',
 			['taskid' => $uuid, 'versionid' => $versionid]);
+		$task = false;
 		if ($aff > 0) {
 			$task = Taskmanager::submit('DownloadFiles', [
 				'id' => $uuid,
@@ -252,10 +252,8 @@ class MiniLinux
 			if (Taskmanager::isFailed($task)) {
 				$task = false;
 			} else {
-				$task = $task['id'];
+				$task = (string)$task['id'];
 			}
-		} else {
-			$task = false;
 		}
 		Database::exec('UNLOCK TABLES');
 		if ($task !== false) {
@@ -269,7 +267,7 @@ class MiniLinux
 		return $task;
 	}
 
-	public static function fileToId($versionid, $fileName)
+	public static function fileToId(string $versionid, string $fileName): string
 	{
 		return 'x' . substr(md5($fileName . $versionid), 0, 8);
 	}
@@ -282,7 +280,7 @@ class MiniLinux
 	 * Generate messages regarding setup und update availability.
 	 * @return bool true if severe problems were found, false otherwise
 	 */
-	public static function generateUpdateNotice()
+	public static function generateUpdateNotice(): bool
 	{
 		// Messages in here are with module name, as required by the
 		// main-warning hook.
@@ -325,7 +323,7 @@ class MiniLinux
 	 * actually installed locally.
 	 * @return bool true if installed locally, false otherwise
 	 */
-	public static function updateCurrentBootSetting()
+	public static function updateCurrentBootSetting(): bool
 	{
 		$default = Property::get(self::PROPERTY_DEFAULT_BOOT);
 		if ($default === false)
@@ -371,7 +369,7 @@ class MiniLinux
 		}
 	}
 
-	public static function queryAllVersionsByBranch()
+	public static function queryAllVersionsByBranch(): array
 	{
 		$list = [];
 		$res = Database::simpleQuery('SELECT branchid, versionid, title, Length(description) AS desclen,
@@ -401,11 +399,12 @@ class MiniLinux
 	 * Check whether an optionally required stage4 is available.
 	 * Return true if there is no stage4, otherwise check filesystem,
 	 * or try to request from local dnbd3-server.
+	 *
 	 * @param array $data decoded data column from minilinux_version
 	 * @param string[] $errors in array of error messages if not available
 	 * @return bool true if stage4 is available or none required
 	 */
-	public static function checkStage4($data, &$errors = false)
+	public static function checkStage4(array $data, &$errors = false): bool
 	{
 		$errors = [];
 		$image = false;
@@ -506,8 +505,6 @@ class MiniLinux
 			if (!isset($data['id']))
 				continue;
 			$id = self::resolveEntryId($data['id']);
-			if ($id === false)
-				continue;
 			$new = [
 				'entryids' => [$row['entryid']],
 				'menus' => explode(',', $row['menus'] ?? ''),
@@ -546,7 +543,7 @@ class MiniLinux
 	 * Take a configured versionid from a bootentry (serversetup module) and translate
 	 * it, in case it's "default" or just a branch name.
 	 */
-	private static function resolveEntryId($id)
+	private static function resolveEntryId(string $id): string
 	{
 		if ($id === 'default') { // Special case
 			$id = Property::get(MiniLinux::PROPERTY_DEFAULT_BOOT_EFFECTIVE);