From 0efbba9e20f79477223f3d553e3bdf221bb6cc33 Mon Sep 17 00:00:00 2001
From: Uwe Steinmann <steinm@debian.org>
Date: Sat, 10 Jul 2021 22:28:16 +0200
Subject: [PATCH] fix setting bool and array values

---
 op/op.Settings.php | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/op/op.Settings.php b/op/op.Settings.php
index 8f0c33f2a..cb2ef35e6 100644
--- a/op/op.Settings.php
+++ b/op/op.Settings.php
@@ -67,8 +67,8 @@ if ($action == "saveSettings")
 	}
 	function setBoolValue($name) {
 		global $_POST, $settings;
-		if(isset($_POST[$name]) && !in_array($name, $settings->_hiddenConfFields)) {
-			if ($_POST[$name]=="on")
+		if(!in_array($name, $settings->_hiddenConfFields)) {
+			if (isset($_POST[$name]) && $_POST[$name]=="on")
 				$settings->{"_".$name} = true;
 			else
 				$settings->{"_".$name} = false;
@@ -81,8 +81,8 @@ if ($action == "saveSettings")
 	}
 	function setArrayValue($name) {
 		global $_POST, $settings;
-		if(isset($_POST[$name]) && !in_array($name, $settings->_hiddenConfFields)) {
-			if($_POST[$name])
+		if(!in_array($name, $settings->_hiddenConfFields)) {
+			if(isset($_POST[$name]) && $_POST[$name])
 				$settings->{"_".$name} = $_POST[$name];
 			else
 				$settings->{"_".$name} = array();