diff --git a/app/bridges/bridge_edit.php b/app/bridges/bridge_edit.php index 663813de9c..173f258505 100644 --- a/app/bridges/bridge_edit.php +++ b/app/bridges/bridge_edit.php @@ -40,7 +40,7 @@ $text = $language->get(); //action add or update - if (is_uuid($_REQUEST["id"])) { + if (!empty($_REQUEST["id"]) && is_uuid($_REQUEST["id"])) { $action = "update"; $bridge_uuid = $_REQUEST["id"]; $id = $_REQUEST["id"]; @@ -50,11 +50,11 @@ } //get http post variables and set them to php variables - if (count($_POST) > 0) { - $bridge_uuid = $_POST["bridge_uuid"]; + if (!empty($_POST)) { + $bridge_uuid = $_POST["bridge_uuid"] ?? null; $bridge_name = $_POST["bridge_name"]; $bridge_destination = $_POST["bridge_destination"]; - $bridge_enabled = $_POST["bridge_enabled"] ?: 'false'; + $bridge_enabled = $_POST["bridge_enabled"] ?? 'false'; $bridge_description = $_POST["bridge_description"]; } @@ -146,7 +146,7 @@ } //pre-populate the form - if (is_array($_GET) && $_POST["persistformvar"] != "true") { + if (!empty($_GET) && is_array($_GET) && (empty($_POST["persistformvar"]) || $_POST["persistformvar"] != "true")) { $bridge_uuid = $_GET["id"]; $sql = "select * from v_bridges "; $sql .= "where bridge_uuid = :bridge_uuid "; @@ -199,7 +199,7 @@ echo " ".$text['label-bridge_name']."\n"; echo "\n"; echo "