From 30a847a563a4f8f39f11d98bca78a36f25721410 Mon Sep 17 00:00:00 2001
From: Pavel Zbitskiy <pavel.zbitskiy@gmail.com>
Date: Fri, 2 Aug 2024 21:48:35 -0400
Subject: [PATCH] Add option to configure NTP server

---
 main/config.h                |  2 +-
 main/htmls/html_pages.h      | 33 +++++++++++++++++++--------------
 main/language_translations.h |  1 +
 main/languages/ca-ES.h       |  3 ++-
 main/languages/da-DA.h       |  3 ++-
 main/languages/de-DE.h       |  1 +
 main/languages/en-GB.h       |  3 ++-
 main/languages/es-ES.h       |  3 ++-
 main/languages/fr-FR.h       |  7 ++++---
 main/languages/it-IT.h       |  7 ++++---
 main/languages/ja-JP.h       |  7 ++++---
 main/languages/vi-VN.h       |  3 ++-
 main/languages/zh-CN.h       |  7 ++++---
 main/main.cpp                | 17 ++++++++++++-----
 14 files changed, 60 insertions(+), 37 deletions(-)

diff --git a/main/config.h b/main/config.h
index 6bb84e5..a4d08c0 100644
--- a/main/config.h
+++ b/main/config.h
@@ -194,7 +194,7 @@ String ap_pwd;
 String ota_pwd;
 
 // time and time zone
-const char *ntpServer = "pool.ntp.org";
+String ntpServer = "pool.ntp.org";
 const long gmtOffset_sec = 3600;
 const int daylightOffset_sec = 3600;
 
diff --git a/main/htmls/html_pages.h b/main/htmls/html_pages.h
index af250e8..8563775 100644
--- a/main/htmls/html_pages.h
+++ b/main/htmls/html_pages.h
@@ -13,8 +13,8 @@
   License along with this library; if not, write to the Free Software
   Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 */
- 
-const char html_page_reboot[] PROGMEM = 
+
+const char html_page_reboot[] PROGMEM =
         "<p>_TXT_M_REBOOT_ <span id='count'>20s</span>...</p>"
         "<script>"
             "setTimeout(function() {"
@@ -22,12 +22,12 @@ const char html_page_reboot[] PROGMEM =
             "}, 20000);"
         "</script>";
 
-const char html_page_reset[] PROGMEM = 
+const char html_page_reset[] PROGMEM =
         "<p>_TXT_M_RESET_ <span id='count'>20s</span>. _TXT_M_RESET_1_ _UNIT_NAME_</p>"
 ;
 
 
-const char html_page_save_reboot[] PROGMEM = 
+const char html_page_save_reboot[] PROGMEM =
         "<p>_TXT_M_SAVE_ <span id='count'>20s</span>...</p>"
         "<script>"
             "setTimeout(function() {"
@@ -36,7 +36,7 @@ const char html_page_save_reboot[] PROGMEM =
         "</script>"
 ;
 
-const char html_page_mqtt[] PROGMEM = 
+const char html_page_mqtt[] PROGMEM =
         "<div id='l1' name='l1'>"
             "<fieldset>"
                "<legend><b>&nbsp; _TXT_MQTT_TITLE_ &nbsp;</b></legend>"
@@ -90,7 +90,7 @@ const char html_page_mqtt[] PROGMEM =
         "</div>"
 ;
 
-const char html_page_others[] PROGMEM = 
+const char html_page_others[] PROGMEM =
         "<div id='l1' name='l1'>"
             "<fieldset>"
                 "<legend><b>&nbsp; _TXT_OTHERS_TITLE_ &nbsp;</b></legend>"
@@ -131,6 +131,11 @@ const char html_page_others[] PROGMEM =
                         "<input id='tz' name='tz' placeholder=' ' value='_TIME_ZONE_' "
 						"autocomplete='off' autocorrect='off' autocapitalize='off' spellcheck='false'>"
                     "</p>"
+                    "<p><b>_TXT_OTHER_NTP_SERVER_</b>"
+                        "<br/>"
+                        "<input id='ntp' name='ntp' placeholder='pool.ntp.org' value='_NTP_SERVER_' "
+						"autocomplete='off' autocorrect='off' autocapitalize='off' spellcheck='false'>"
+                    "</p>"
                     "<br/>"
                     "<button name='save' type='submit' class='button bgrn'>_TXT_SAVE_</button>"
                 "</form>"
@@ -143,7 +148,7 @@ const char html_page_others[] PROGMEM =
         "</div>"
 ;
 
-const char html_page_status[] PROGMEM = 
+const char html_page_status[] PROGMEM =
         "<div id='l1' name='l1'>"
             "<fieldset>"
             "<legend><b>&nbsp; _TXT_STATUS_TITLE_ &nbsp;</b></legend>"
@@ -168,7 +173,7 @@ const char html_page_status[] PROGMEM =
             "</p>"
         "</div>";
 
-const char html_page_wifi[] PROGMEM = 
+const char html_page_wifi[] PROGMEM =
         "<div id='l1' name='l1'>"
             "<fieldset>"
                 "<legend><b>&nbsp; _TXT_WIFI_TITLE_ &nbsp;</b></legend>"
@@ -211,7 +216,7 @@ const char html_page_wifi[] PROGMEM =
         "</div>"
 ;
 
-const char html_page_control[] PROGMEM = 
+const char html_page_control[] PROGMEM =
 
         "<div style='text-align:center;'>"
             "<h2>_TXT_CTRL_CTEMP_ <span id='room_temperature'>_ROOMTEMP_</span>&#176;</h2>"
@@ -307,7 +312,7 @@ const char html_page_control[] PROGMEM =
             "</p>"
         "</div>";
 
-const char html_page_unit[] PROGMEM = 
+const char html_page_unit[] PROGMEM =
         "<div id='l1' name='l1'>"
             "<fieldset>"
                 "<legend><b>&nbsp; _TXT_UNIT_TITLE_ &nbsp;</b></legend>"
@@ -355,7 +360,7 @@ const char html_page_unit[] PROGMEM =
                         "<input id='lpwc' name='lpwc' type='password' placeholder=' ' value='_LOGIN_PASSWORD_' "
 						"autocomplete='off' autocorrect='off' autocapitalize='off' spellcheck='false'>"
                     "</p>"
-                    "<span id='message_not_match' style='color:red'> </span> <br>" 
+                    "<span id='message_not_match' style='color:red'> </span> <br>"
                     "<p>_TXT_UNIT_LOGIN_USERNAME_ <b>admin</b></p>"
                     "<br/>"
                     "<button name='save' type='submit' class='button bgrn'>_TXT_SAVE_</button>"
@@ -370,7 +375,7 @@ const char html_page_unit[] PROGMEM =
         "</div>"
 ;
 
-const char html_page_login[] PROGMEM = 
+const char html_page_login[] PROGMEM =
     "<script>"
         "var loginSucess = _LOGIN_SUCCESS_;"
         "document.onreadystatechange = function() {"
@@ -406,7 +411,7 @@ const char html_page_login[] PROGMEM =
         "</div>"
 ;
 
-const char html_page_upgrade[] PROGMEM = 
+const char html_page_upgrade[] PROGMEM =
     "<script>"
         "function eb(s) {"
             "return document.getElementById(s);"
@@ -438,7 +443,7 @@ const char html_page_upgrade[] PROGMEM =
         "<div id='f2' style='display:none;text-align:center;'><b>_TXT_UPGRADE_START_ ...</b></div>"
 ;
 
-const char html_page_upload[] PROGMEM = 
+const char html_page_upload[] PROGMEM =
         "<div style='text-align:center;'>"
             "<h2>_TXT_UPLOAD_FW_PAGE_</h2>"
         "</div>"
diff --git a/main/language_translations.h b/main/language_translations.h
index 4ba1561..cbd1b5c 100644
--- a/main/language_translations.h
+++ b/main/language_translations.h
@@ -127,6 +127,7 @@ MAKE_WORD_TRANSLATION(txt_others_tx_pin, en::txt_others_tx_pin, vi::txt_others_t
 MAKE_WORD_TRANSLATION(txt_others_rx_pin, en::txt_others_rx_pin, vi::txt_others_rx_pin, da::txt_others_rx_pin, de::txt_others_rx_pin, es::txt_others_rx_pin, fr::txt_others_rx_pin, it::txt_others_rx_pin, ja::txt_others_rx_pin, zh::txt_others_rx_pin, ca::txt_others_rx_pin)                                             // TODO translate
 MAKE_WORD_TRANSLATION(txt_others_tz, en::txt_others_tz, vi::txt_others_tz, da::txt_others_tz, de::txt_others_tz, es::txt_others_tz, fr::txt_others_tz, it::txt_others_tz, ja::txt_others_tz, zh::txt_others_tz, ca::txt_others_tz)                                             // TODO translate
 MAKE_WORD_TRANSLATION(txt_others_tz_list, en::txt_others_tz_list, vi::txt_others_tz_list, da::txt_others_tz_list, de::txt_others_tz_list, es::txt_others_tz_list, fr::txt_others_tz_list, it::txt_others_tz_list, ja::txt_others_tz_list, zh::txt_others_tz_list, ca::txt_others_tz_list)                                             // TODO translate
+MAKE_WORD_TRANSLATION(txt_others_ntp_server, en::txt_others_ntp_server, vi::txt_others_ntp_server, da::txt_others_ntp_server, de::txt_others_ntp_server, es::txt_others_ntp_server, fr::txt_others_ntp_server, it::txt_others_ntp_server, ja::txt_others_ntp_server, zh::txt_others_ntp_server, ca::txt_others_ntp_server)            // TODO translate
 
 // Page Status
 MAKE_WORD_TRANSLATION(txt_status_title, en::txt_status_title, vi::txt_status_title, da::txt_status_title, de::txt_status_title, es::txt_status_title, fr::txt_status_title, it::txt_status_title, ja::txt_status_title, zh::txt_status_title, ca::txt_status_title)                                                                   // TODO translate
diff --git a/main/languages/ca-ES.h b/main/languages/ca-ES.h
index 1013b82..7e5ce66 100644
--- a/main/languages/ca-ES.h
+++ b/main/languages/ca-ES.h
@@ -97,6 +97,7 @@ namespace ca
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
   const char txt_others_tz[] PROGMEM = "Fus horari";
   const char txt_others_tz_list[] PROGMEM = "Vegeu-ne la llista";
+  const char txt_others_ntp_server[] PROGMEM = "Server NTP";
 
   // Page Status
   const char txt_status_title[] PROGMEM = "Estat";
@@ -177,7 +178,7 @@ namespace ca
   const char txt_upload_success[] PROGMEM = "Correcta";
   const char txt_upload_refresh[] PROGMEM = "Refrescant en";
 
-  // Page Init  
+  // Page Init
   const char txt_init_title[] PROGMEM = "Configuració inicial";
   const char txt_init_reboot_mes[] PROGMEM = "S'està reiniciant i connectant a la xarxa WiFi! Hauries de veure'l llistat al teu punt d'accés.";
   const char txt_init_reboot_mes_1[] PROGMEM = "Utilitzeu aquesta adreça per accedir al vostre dispositiu després de reiniciar-lo. S'està reiniciant";
diff --git a/main/languages/da-DA.h b/main/languages/da-DA.h
index 09ddc7c..d328b73 100644
--- a/main/languages/da-DA.h
+++ b/main/languages/da-DA.h
@@ -96,7 +96,8 @@ namespace da
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
   const char txt_others_tz[] PROGMEM = "Timezone";
   const char txt_others_tz_list[] PROGMEM = "See list";
-  
+  const char txt_others_ntp_server[] PROGMEM = "NTP Server";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "Status";
   const char txt_status_hvac[] PROGMEM = "HVAC Status";
diff --git a/main/languages/de-DE.h b/main/languages/de-DE.h
index ba43dd9..2ca511a 100644
--- a/main/languages/de-DE.h
+++ b/main/languages/de-DE.h
@@ -96,6 +96,7 @@ namespace de
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
   const char txt_others_tz[] PROGMEM = "Timezone";
   const char txt_others_tz_list[] PROGMEM = "See list";
+  const char txt_others_ntp_server[] PROGMEM = "NTP Server";
 
   // Page Status
   const char txt_status_title[] PROGMEM = "Status";
diff --git a/main/languages/en-GB.h b/main/languages/en-GB.h
index 513021a..db185e6 100644
--- a/main/languages/en-GB.h
+++ b/main/languages/en-GB.h
@@ -97,7 +97,8 @@ namespace en
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
   const char txt_others_tz[] PROGMEM = "Timezone";
   const char txt_others_tz_list[] PROGMEM = "See list";
-  
+  const char txt_others_ntp_server[] PROGMEM = "NTP Server";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "Status";
   const char txt_status_hvac[] PROGMEM = "HVAC Status";
diff --git a/main/languages/es-ES.h b/main/languages/es-ES.h
index 46d6e4a..4c0bed6 100644
--- a/main/languages/es-ES.h
+++ b/main/languages/es-ES.h
@@ -97,7 +97,8 @@ namespace es
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
   const char txt_others_tz[] PROGMEM = "Timezone";
   const char txt_others_tz_list[] PROGMEM = "See list";
-  
+  const char txt_others_ntp_server[] PROGMEM = "Servidor NTP";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "Estado";
   const char txt_status_hvac[] PROGMEM = "Estado HVAC";
diff --git a/main/languages/fr-FR.h b/main/languages/fr-FR.h
index 9d517a8..d0568e8 100644
--- a/main/languages/fr-FR.h
+++ b/main/languages/fr-FR.h
@@ -95,9 +95,10 @@ namespace fr
   const char txt_others_debug_log[] PROGMEM = "MQTT topic debug logs";
   const char txt_others_tx_pin[] PROGMEM = "TX pin (ESP32, 0 use UART1)";
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
-  const char txt_others_tz[] PROGMEM = "Timezone";
-  const char txt_others_tz_list[] PROGMEM = "See list";
-  
+  const char txt_others_tz[] PROGMEM = "Fuseau horaire";
+  const char txt_others_tz_list[] PROGMEM = "Voir la liste";
+  const char txt_others_ntp_server[] PROGMEM = "Serveur NTP";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "Etats";
   const char txt_status_hvac[] PROGMEM = "Etat HVAC";
diff --git a/main/languages/it-IT.h b/main/languages/it-IT.h
index 8d3c243..8455547 100644
--- a/main/languages/it-IT.h
+++ b/main/languages/it-IT.h
@@ -95,9 +95,10 @@ namespace it
   const char txt_others_debug_log[] PROGMEM = "MQTT topic debug logs";
   const char txt_others_tx_pin[] PROGMEM = "TX pin (ESP32, 0 use UART1)";
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
-  const char txt_others_tz[] PROGMEM = "Timezone";
-  const char txt_others_tz_list[] PROGMEM = "See list";
-  
+  const char txt_others_tz[] PROGMEM = "Fuso orario";
+  const char txt_others_tz_list[] PROGMEM = "Vedi l'elenco";
+  const char txt_others_ntp_server[] PROGMEM = "Servidor NTP";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "Stato";
   const char txt_status_hvac[] PROGMEM = "Stato HVAC";
diff --git a/main/languages/ja-JP.h b/main/languages/ja-JP.h
index 708311e..412dc4b 100644
--- a/main/languages/ja-JP.h
+++ b/main/languages/ja-JP.h
@@ -95,9 +95,10 @@ namespace ja
   const char txt_others_debug_log[] PROGMEM = "MQTT topic debug logs";
   const char txt_others_tx_pin[] PROGMEM = "TX pin (ESP32, 0 use UART1)";
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
-  const char txt_others_tz[] PROGMEM = "Timezone";
-  const char txt_others_tz_list[] PROGMEM = "See list";
-  
+  const char txt_others_tz[] PROGMEM = "タイムゾーン";
+  const char txt_others_tz_list[] PROGMEM = "リストを見る";
+  const char txt_others_ntp_server[] PROGMEM = "NTP サーバー";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "ステータス";
   const char txt_status_hvac[] PROGMEM = "エアコン本体";
diff --git a/main/languages/vi-VN.h b/main/languages/vi-VN.h
index 9205238..08a7511 100644
--- a/main/languages/vi-VN.h
+++ b/main/languages/vi-VN.h
@@ -97,7 +97,8 @@ namespace vi
   const char txt_others_rx_pin[] PROGMEM = "Chân RX (ESP32, 0 dùng UART1)";
   const char txt_others_tz[] PROGMEM = "Múi giờ";
   const char txt_others_tz_list[] PROGMEM = "Xem danh sách";
-  
+  const char txt_others_ntp_server[] PROGMEM = "Máy chủ NTP";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "Trạng thái";
   const char txt_status_hvac[] PROGMEM = "Trạng thái ĐH";
diff --git a/main/languages/zh-CN.h b/main/languages/zh-CN.h
index 3544101..90b80de 100644
--- a/main/languages/zh-CN.h
+++ b/main/languages/zh-CN.h
@@ -95,9 +95,10 @@ namespace zh
   const char txt_others_debug_log[] PROGMEM = "MQTT topic debug logs";
   const char txt_others_tx_pin[] PROGMEM = "TX pin (ESP32, 0 use UART1)";
   const char txt_others_rx_pin[] PROGMEM = "RX pin (ESP32, 0 use UART1)";
-  const char txt_others_tz[] PROGMEM = "Timezone";
-  const char txt_others_tz_list[] PROGMEM = "See list";
-  
+  const char txt_others_tz[] PROGMEM = "时区";
+  const char txt_others_tz_list[] PROGMEM = "查看列表";
+  const char txt_others_ntp_server[] PROGMEM = "NTP 服务器";
+
   // Page Status
   const char txt_status_title[] PROGMEM = "状态";
   const char txt_status_hvac[] PROGMEM = "空调状态";
diff --git a/main/main.cpp b/main/main.cpp
index 184e13c..a70bd3d 100644
--- a/main/main.cpp
+++ b/main/main.cpp
@@ -543,7 +543,7 @@ bool loadOthers()
     return false;
   }
   // Allocate document capacity.
-  const size_t capacity = JSON_OBJECT_SIZE(7) + 350;
+  const size_t capacity = JSON_OBJECT_SIZE(9) + 366;
   DynamicJsonDocument doc(capacity);
   deserializeJson(doc, configFile);
   // unit
@@ -581,6 +581,10 @@ bool loadOthers()
   {
     timezone = doc["tz"].as<String>();
   }
+  if (doc.containsKey("ntp"))
+  {
+    ntpServer = doc["ntp"].as<String>();
+  }
   return true;
 }
 
@@ -686,10 +690,10 @@ void saveWifi(String apSsid, const String& apPwd, String hostName, const String&
   configFile.close();
 }
 
-void saveOthers(const String& haa, const String& haat, const String& debugPckts, const String& debugLogs, const String& txPin, const String& rxPin, const String& tz)
+void saveOthers(const String& haa, const String& haat, const String& debugPckts, const String& debugLogs, const String& txPin, const String& rxPin, const String& tz, const String &ntp)
 {
   // Allocate document capacity.
-  const size_t capacity = JSON_OBJECT_SIZE(7) + 350;
+  const size_t capacity = JSON_OBJECT_SIZE(9) + 366;
   DynamicJsonDocument doc(capacity);
   doc["haa"] = haa;
   doc["haat"] = haat;
@@ -698,6 +702,7 @@ void saveOthers(const String& haa, const String& haat, const String& debugPckts,
   doc["txPin"] = txPin;
   doc["rxPin"] = rxPin;
   doc["tz"] = tz;
+  doc["ntp"] = ntp;
   File configFile = SPIFFS.open(others_conf, "w");
   if (!configFile)
   {
@@ -712,7 +717,7 @@ void saveCurrentOthers()
   String haa = others_haa ? "ON" : "OFF";
   String debugPckts = _debugModePckts ? "ON" : "OFF";
   String debugLogs = _debugModeLogs ? "ON" : "OFF";
-  saveOthers(haa, others_haa_topic, debugPckts, debugLogs, String(HP_TX), String(HP_RX), timezone);
+  saveOthers(haa, others_haa_topic, debugPckts, debugLogs, String(HP_TX), String(HP_RX), timezone, ntpServer);
 }
 
 // Initialize captive portal page
@@ -1138,7 +1143,7 @@ void handleOthers(AsyncWebServerRequest *request)
   checkLogin(request);
   if (request->hasArg("save"))
   {
-    saveOthers(request->arg("HAA"), request->arg("haat"), request->arg("DebugPckts"), request->arg("DebugLogs"), request->arg("tx_pin"), request->arg("rx_pin"), request->arg("tz"));
+    saveOthers(request->arg("HAA"), request->arg("haat"), request->arg("DebugPckts"), request->arg("DebugLogs"), request->arg("tx_pin"), request->arg("rx_pin"), request->arg("tz"), request->arg("ntp"));
     String saveRebootPage = FPSTR(html_page_save_reboot);
     // localize
     saveRebootPage.replace("_TXT_M_SAVE_", translatedWord(FL_(txt_m_save)));
@@ -1156,6 +1161,7 @@ void handleOthers(AsyncWebServerRequest *request)
     othersPage.replace("_TXT_OTHERS_DEBUG_PCKTS_", translatedWord(FL_(txt_others_debug_packets)));
     othersPage.replace("_TXT_OTHERS_DEBUG_LOGS_", translatedWord(FL_(txt_others_debug_log)));
     othersPage.replace("_TXT_OTHERS_TIME_ZONE_", translatedWord(FL_(txt_others_tz)));
+    othersPage.replace("_TXT_OTHER_NTP_SERVER_", translatedWord(FL_(txt_others_ntp_server)));
     othersPage.replace("_SEE_TZ_LIST", translatedWord(FL_(txt_others_tz_list)));
     othersPage.replace("_TXT_TX_PIN_", translatedWord(FL_(txt_others_tx_pin)));
     othersPage.replace("_TXT_RX_PIN_", translatedWord(FL_(txt_others_rx_pin)));
@@ -1168,6 +1174,7 @@ void handleOthers(AsyncWebServerRequest *request)
     othersPage.replace("_TX_PIN_", String(HP_TX));
     othersPage.replace("_RX_PIN_", String(HP_RX));
     othersPage.replace("_TIME_ZONE_", timezone);
+    othersPage.replace("_NTP_SERVER_", String(ntpServer));
     if (others_haa)
     {
       othersPage.replace("_HAA_ON_", "selected");