From 06bff0b9b84d47c43f9bc8aff06a29d85ebb7ed0 Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Tue, 14 Nov 2023 14:47:55 +0100
Subject: Add function param/return types, fix a lot more phpstorm complaints

---
 apis/cron.inc.php | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

(limited to 'apis')

diff --git a/apis/cron.inc.php b/apis/cron.inc.php
index 0b1e92a9..03b6201f 100644
--- a/apis/cron.inc.php
+++ b/apis/cron.inc.php
@@ -14,7 +14,7 @@ define('CRON_KEY_STATUS', 'cron.key.status');
 define('CRON_KEY_BLOCKED', 'cron.key.blocked');
 
 // Crash report mode - used by system crontab entry
-if (($report = Request::get('crashreport', false, 'string'))) {
+if (($report = Request::get('crashreport', false, 'string')) !== false) {
 	$list = Property::getList(CRON_KEY_STATUS);
 	if (empty($list)) {
 		error_log('Cron crash report triggered but no cronjob marked active.');
@@ -62,10 +62,7 @@ function getJobStatus($id)
 }
 
 // Hooks by other modules
-/**
- * @param Hook $hook
- */
-function handleModule($hook)
+function handleModule(Hook $hook): void
 {
 	global $cron_log_text;
 	$cron_log_text = '';
-- 
cgit v1.2.3-55-g7522