correctly convert allowed_mysqlserver json-string to array

Signed-off-by: Michael Kaufmann <d00p@froxlor.org>
froxlor · Mar 3, 2024 · c0fdc62 · c0fdc62
1 parent b14eaf4
commit c0fdc62
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/Froxlor/Api/Commands/Customers.php b/lib/Froxlor/Api/Commands/Customers.php
@@ -1343,7 +1343,8 @@ public function update()
 				]);
 
 				// enable/disable global mysql-user (loginname)
-				foreach ($result['allowed_mysqlserver'] as $dbserver) {
+				$current_allowed_mysqlserver =  isset($result['allowed_mysqlserver']) && !empty($result['allowed_mysqlserver']) ? json_decode($result['allowed_mysqlserver'], true) : [];
+				foreach ($current_allowed_mysqlserver as $dbserver) {
 					// require privileged access for target db-server
 					Database::needRoot(true, $dbserver, false);
 					// get DbManager
@@ -1674,7 +1675,8 @@ public function delete()
 			$id = $result['customerid'];
 
 			// remove global mysql-user (loginname)
-			foreach ($result['allowed_mysqlserver'] as $dbserver) {
+			$current_allowed_mysqlserver =  isset($result['allowed_mysqlserver']) && !empty($result['allowed_mysqlserver']) ? json_decode($result['allowed_mysqlserver'], true) : [];
+			foreach ($current_allowed_mysqlserver as $dbserver) {
 				// require privileged access for target db-server
 				Database::needRoot(true, $dbserver, false);
 				// get DbManager