Fixed minor bug in Settings_Wifi.html

Minor Bug: If setting a SSID empty, then any followed network will be ignored
wled-dev · Jan 18, 2024 · b1d8baf · b1d8baf
1 parent acd524d
commit b1d8baf
Show file tree

Hide file tree

Showing 2 changed files with 219 additions and 183 deletions.
diff --git a/wled00/data/settings_wifi.htm b/wled00/data/settings_wifi.htm
@@ -152,7 +152,7 @@
 						oE.setAttribute("id", `CS${o - 1}`);
 						oE.setAttribute("name", `CS${o - 1}`);
 					}
-					oE = gId(`CP${iti}`);
+					oE = gId(`CP${o}`);
 					if (oE) {
 						oE.setAttribute("id", `CP${o - 1}`);
 						oE.setAttribute("name", `CP${o - 1}`);
@@ -232,14 +232,41 @@
 			}
 			U();
 		}
+		function PreS() {
+			for (let i = 0; i < wCount; i++) {
+				let oE = gId(`CS${i}`);
+				if (!oE || !oE.value || oE.value.trim().length == 0) {
+					oE.remove();
+					if (oE) oE.remove();
+					oE = gId(`CP${i}`);
+					if (oE) oE.remove();
+					oE = gId(`CR${i}`);
+					if (oE) oE.remove();
+
+					for (let o = i + 1; o < wCount; o++) {
+						oE = gId(`CS${o}`);
+						if (oE) {
+							oE.setAttribute("id", `CS${o - 1}`);
+							oE.setAttribute("name", `CS${o - 1}`);
+						}
+						oE = gId(`CP${o}`);
+						if (oE) {
+							oE.setAttribute("id", `CP${o - 1}`);
+							oE.setAttribute("name", `CP${o - 1}`);
+						}
+					}
+					wCount--;
+				}
+			}
+		}
 		function getURL(path) {
 			return (loc ? locproto + "//" + locip : "") + path;
 		}
 	</script>
 	<style>@import url("style.css");</style>
 </head>
 <body onload="S()">
-	<form id="form_s" name="Sf" method="post">
+	<form id="form_s" name="Sf" method="post" onsubmit="PreS()">
 		<div class="toprow">
 		<div class="helpB"><button type="button" onclick="H()">?</button></div>
 		<button type="button" onclick="B()">Back</button><button type="submit">Save & Connect</button><hr>