From d6a727a8d13764f72a88dbd9340d87e49086fc62 Mon Sep 17 00:00:00 2001
From: "Zachary J. Fields" <zachary_fields@yahoo.com>
Date: Mon, 7 Oct 2024 11:54:30 -0500
Subject: [PATCH] chore: Remove vestigial database template from NCH

---
 src/NotecardConnectionHandler.cpp | 37 -------------------------------
 src/NotecardConnectionHandler.h   |  3 ++-
 2 files changed, 2 insertions(+), 38 deletions(-)

diff --git a/src/NotecardConnectionHandler.cpp b/src/NotecardConnectionHandler.cpp
index 5205c4a..b254c79 100644
--- a/src/NotecardConnectionHandler.cpp
+++ b/src/NotecardConnectionHandler.cpp
@@ -31,13 +31,11 @@
 #define NOTEFILE_BASE_NAME "arduino_iot_cloud"
 
 // Notecard LoRa requires us to choose an arbitrary port between 1-99
-#define NOTEFILE_DATABASE_LORA_PORT 1
 #define NOTEFILE_INBOUND_LORA_PORT 2
 #define NOTEFILE_OUTBOUND_LORA_PORT 3
 
 // Note that we use "s" versions of the Notefile extensions to ensure that
 // traffic always happens on a secure transport
-#define NOTEFILE_SECURE_DATABASE NOTEFILE_BASE_NAME ".dbs"
 #define NOTEFILE_SECURE_INBOUND NOTEFILE_BASE_NAME ".qis"
 #define NOTEFILE_SECURE_OUTBOUND NOTEFILE_BASE_NAME ".qos"
 
@@ -423,41 +421,6 @@ NetworkConnectionState NotecardConnectionHandler::update_handleInit()
   }
 #endif
 
-  // Set database template to support LoRa/Satellite Notecard
-  if (NetworkConnectionState::INIT == result) {
-    if (J *req = _notecard.newRequest("note.template")) {
-      JAddStringToObject(req, "file", NOTEFILE_SECURE_DATABASE);
-      JAddStringToObject(req, "format", "compact");               // Support LoRa/Satellite Notecards
-      JAddIntToObject(req, "port", NOTEFILE_DATABASE_LORA_PORT);  // Support LoRa/Satellite Notecards
-      if (J *body = JAddObjectToObject(req, "body")) {
-        JAddStringToObject(body, "text", TSTRINGV);
-        JAddNumberToObject(body, "value", TFLOAT64);
-        JAddBoolToObject(body, "flag", TBOOL);
-        if (J *rsp = _notecard.requestAndResponse(req)) {
-          // Check the response for errors
-          if (NoteResponseError(rsp)) {
-            const char *err = JGetString(rsp, "err");
-            Debug.print(DBG_ERROR, F("%s"), err);
-            result = NetworkConnectionState::ERROR;
-          } else {
-            result = NetworkConnectionState::INIT;
-          }
-          JDelete(rsp);
-        } else {
-          Debug.print(DBG_ERROR, F("Failed to receive response from Notecard."));
-          result = NetworkConnectionState::ERROR; // Assume the worst
-        }
-      } else {
-        Debug.print(DBG_ERROR, "Failed to allocate request: note.template:body");
-        JFree(req);
-        result = NetworkConnectionState::ERROR; // Assume the worst
-      }
-    } else {
-      Debug.print(DBG_ERROR, "Failed to allocate request: note.template");
-      result = NetworkConnectionState::ERROR; // Assume the worst
-    }
-  }
-
   // Set inbound template to support LoRa/Satellite Notecard
   if (NetworkConnectionState::INIT == result) {
     if (J *req = _notecard.newRequest("note.template")) {
diff --git a/src/NotecardConnectionHandler.h b/src/NotecardConnectionHandler.h
index c758030..2b3a873 100644
--- a/src/NotecardConnectionHandler.h
+++ b/src/NotecardConnectionHandler.h
@@ -29,7 +29,7 @@
 
 #define NOTECARD_CONNECTION_HANDLER_VERSION_MAJOR 1
 #define NOTECARD_CONNECTION_HANDLER_VERSION_MINOR 0
-#define NOTECARD_CONNECTION_HANDLER_VERSION_PATCH 0
+#define NOTECARD_CONNECTION_HANDLER_VERSION_PATCH 1
 
 #define NOTECARD_CONNECTION_HANDLER_VERSION NOTE_C_STRINGIZE(NOTECARD_CONNECTION_HANDLER_VERSION_MAJOR) "." NOTE_C_STRINGIZE(NOTECARD_CONNECTION_HANDLER_VERSION_MINOR) "." NOTE_C_STRINGIZE(NOTECARD_CONNECTION_HANDLER_VERSION_PATCH)
 
@@ -261,6 +261,7 @@ class NotecardConnectionHandler final : public ConnectionHandler
      * @param interval_min[in] The inbound polling interval (in minutes)
      *
      * @note Set the interval to 0 to disable inbound polling.
+     * @note Must be set prior to initializing the connection to the Notecard.
      */
     inline void setNotehubPollingInterval (int32_t interval_min) {
       _inbound_polling_interval_min = (interval_min ? interval_min : -1);