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/baseconfig/page.inc.php | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

(limited to 'modules-available/baseconfig/page.inc.php')

diff --git a/modules-available/baseconfig/page.inc.php b/modules-available/baseconfig/page.inc.php
index a1921084..8a9a7534 100644
--- a/modules-available/baseconfig/page.inc.php
+++ b/modules-available/baseconfig/page.inc.php
@@ -120,12 +120,11 @@ class Page_BaseConfig extends Page
 		User::assertPermission('view', $lid);
 		$editForbidden = !User::hasPermission('edit', $lid);
 		// Get stuff that's set in DB already
+		$fields = '';
 		if ($this->targetModule !== false && isset($this->qry_extra['field'])) {
-			$fields = '';
 			$where = " WHERE {$this->qry_extra['field']} = :field_value";
 			$params = array('field_value' => $this->qry_extra['field_value']);
 		} else {
-			$fields = '';
 			$where = '';
 			$params = array();
 		}
@@ -136,7 +135,7 @@ class Page_BaseConfig extends Page
 		// Remember missing variables
 		$missing = $varsFromJson;
 		// Populate structure with existing config from db
-		$this->fillSettings($varsFromJson, $settings, $missing, $this->qry_extra['table'], $fields, $where, $params, false);
+		$this->fillSettings($varsFromJson, $settings, $missing, $this->qry_extra['table'], $fields, $where, $params);
 		// Add entries that weren't in the db (global), setup override checkbox (module specific)
 		foreach ($varsFromJson as $key => $var) {
 			if ($this->targetModule !== false && !isset($settings[$var['catid']]['settings'][$key])) {
@@ -195,7 +194,7 @@ class Page_BaseConfig extends Page
 		) + $this->qry_extra);
 	}
 
-	private function fillSettings($vars, &$settings, &$missing, $table, $fields, $where, $params, $sourceName)
+	private function fillSettings($vars, &$settings, &$missing, $table, $fields, $where, $params): void
 	{
 		$res = Database::simpleQuery("SELECT setting, value, displayvalue $fields FROM $table "
 			. " {$where} ORDER BY setting ASC", $params);
@@ -203,7 +202,7 @@ class Page_BaseConfig extends Page
 			if (!isset($missing[$row['setting']]))
 				continue;
 			if (!isset($vars[$row['setting']]) || !is_array($vars[$row['setting']])) {
-				$unknown[] = $row['setting'];
+				//$unknown[] = $row['setting'];
 				continue;
 			}
 			unset($missing[$row['setting']]);
-- 
cgit v1.2.3-55-g7522