Merge pull request #16 from fermyon/bacongobbler/good-ol-string-repla…

…cement use some good ol' string replacement
fermyon · Nov 8, 2024 · 0242b0f · 0242b0f
2 parents 3e6cacb + d9dce4b
commit 0242b0f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/api/src/handlers.ts b/api/src/handlers.ts
@@ -15,10 +15,15 @@ const getSubsetByDay = (dayString: string, res: ResponseBuilder) => {
         res.send()
         return;
     }
-    let conn = Sqlite.openDefault();
-    let result = conn.execute(sqlSelectSubsetByDay, [d]);
-    let items = result.rows.map(row => { return asNoiseLogItem(row) });
-    sendJson(res, items)
+    try {
+        let conn = Sqlite.openDefault();
+        let result = conn.execute(sqlSelectSubsetByDay.replace('$1', '\'' + d + '\''), []);
+        let items = result.rows.map(row => { return asNoiseLogItem(row) });
+        sendJson(res, items)
+    } catch (e: any) {
+        console.log(e);
+        console.log(e.payload);
+    }
 
 }
 const getSubsetByRelativeTime = (relativeTimeExpression: string, res: ResponseBuilder) => {
@@ -65,4 +70,4 @@ export {
     getAll,
     getSubsetByRelativeTime,
     getSubsetByDay
-}
+}
diff --git a/mqtt-message-persister/migration.up.sql b/mqtt-message-persister/migration.up.sql
@@ -2,5 +2,5 @@ CREATE TABLE IF NOT EXISTS noise_log (
     id INTEGER PRIMARY KEY AUTOINCREMENT,
     source TEXT NOT NULL,
     volume INTEGER NOT NULL,
-    timestamp TEXT NOT NULL
+    timestamp TIMESTAMP NOT NULL
 );