From 74bff0912b061b3970ed07d1ba7746f7490c4762 Mon Sep 17 00:00:00 2001 From: Peter Woolery Date: Mon, 13 Apr 2026 14:05:26 -0700 Subject: [PATCH] =?UTF-8?q?fix:=20config=5Fsave=5Fwifi=20=E2=80=94=20alway?= =?UTF-8?q?s=20write=20both=20credentials?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Replace short-circuit boolean evaluation of putString return values with separate size_t variables so both writes always execute regardless of whether the first succeeds. Co-Authored-By: Claude Sonnet 4.6 --- firmware/src/config.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/firmware/src/config.cpp b/firmware/src/config.cpp index 1ae4a9e..8e349d4 100644 --- a/firmware/src/config.cpp +++ b/firmware/src/config.cpp @@ -25,10 +25,10 @@ bool config_load(DeviceConfig& cfg) { bool config_save_wifi(const String& ssid, const String& pass) { Preferences prefs; prefs.begin(NS, false); - bool ok = prefs.putString("wifi_ssid", ssid) && - prefs.putString("wifi_pass", pass); + size_t r1 = prefs.putString("wifi_ssid", ssid); + size_t r2 = prefs.putString("wifi_pass", pass); prefs.end(); - return ok; + return (r1 > 0) && (r2 > 0); } bool config_has_wifi() {