From c2d95ffaeaa289752b4c7b6664b6ca112a02e350 Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Thu, 2 Nov 2023 19:08:58 +0100
Subject: Roundup of issues reported by PHPStorm

Mostly redundant checks, logic errors, dead code, etc.
---
 modules-available/rebootcontrol/page.inc.php           | 2 --
 modules-available/rebootcontrol/pages/jumphost.inc.php | 1 -
 2 files changed, 3 deletions(-)

(limited to 'modules-available/rebootcontrol')

diff --git a/modules-available/rebootcontrol/page.inc.php b/modules-available/rebootcontrol/page.inc.php
index 872f27d4..c189e1fc 100644
--- a/modules-available/rebootcontrol/page.inc.php
+++ b/modules-available/rebootcontrol/page.inc.php
@@ -101,7 +101,6 @@ class Page_RebootControl extends Page
 		if (Taskmanager::isTask($task)) {
 			Util::redirect("?do=rebootcontrol&show=task&what=task&taskid=" . $task["id"]);
 		}
-		return;
 	}
 
 	/**
@@ -127,7 +126,6 @@ class Page_RebootControl extends Page
 
 		if ($this->haveSubpage) {
 			SubPage::doRender();
-			return;
 		}
 	}
 
diff --git a/modules-available/rebootcontrol/pages/jumphost.inc.php b/modules-available/rebootcontrol/pages/jumphost.inc.php
index 90508e91..d9aae234 100644
--- a/modules-available/rebootcontrol/pages/jumphost.inc.php
+++ b/modules-available/rebootcontrol/pages/jumphost.inc.php
@@ -32,7 +32,6 @@ class SubPage
 		if ($id !== false) {
 			User::assertPermission('jumphost.edit');
 			self::deleteJumphost($id);
-			return;
 		}
 	}
 
-- 
cgit v1.2.3-55-g7522