From d36a572cd36a3070c966b6f3ff8135a2eda53237 Mon Sep 17 00:00:00 2001
From: Kevin Lyda <kevin@lyda.ie>
Date: Sun, 18 May 2025 12:25:58 +0100
Subject: [PATCH] Permanent messages are permanent...

Sigh.  Boolean errors strike again.
---
 NOTES.md                  | 6 +++---
 storage/batch.sql.go      | 4 ++--
 storage/queries/batch.sql | 4 ++--
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/NOTES.md b/NOTES.md
index a368c8b..8249392 100644
--- a/NOTES.md
+++ b/NOTES.md
@@ -20,9 +20,6 @@ Switch between MAIL and BULLETIN modes? MAIL commands are documented
 
 ## Things to do
 
-  * Remove all file related things.  Which means no need for most
-    (all?) /EDIT flags
-  * Stop the seeded messages from being deleted by the expire batch command.
   * Run [godoc](http://localhost:6060/) and then review where the help text is lacking.
   * Implement each command.
     * Next: folder commands - MODIFY
@@ -69,6 +66,9 @@ Done:
     * ~~Remove BBOARD references.~~
     * ~~format with `par w72j1`~~
   * ~~Handle MARK for SELECT and DIRECTORY.~~
+  * ~~Remove all file related things.  Which means no need for most
+    (all?) /EDIT flags~~
+  * ~~Stop the seeded messages from being deleted by the expire batch command.~~
 
 ## Module links
 
diff --git a/storage/batch.sql.go b/storage/batch.sql.go
index 793f70d..a8c8113 100644
--- a/storage/batch.sql.go
+++ b/storage/batch.sql.go
@@ -11,7 +11,7 @@ import (
 
 const deleteAllExpiredMessages = `-- name: DeleteAllExpiredMessages :execrows
 DELETE FROM messages
-WHERE permanent != 0 AND expiration < CURRENT_TIMESTAMP
+WHERE permanent != 1 AND expiration < CURRENT_TIMESTAMP
 `
 
 func (q *Queries) DeleteAllExpiredMessages(ctx context.Context) (int64, error) {
@@ -24,7 +24,7 @@ func (q *Queries) DeleteAllExpiredMessages(ctx context.Context) (int64, error) {
 
 const deleteAllShutdownMessages = `-- name: DeleteAllShutdownMessages :execrows
 DELETE FROM messages
-WHERE permanent != 0 AND shutdown > 0
+WHERE permanent != 1 AND shutdown > 0
 `
 
 func (q *Queries) DeleteAllShutdownMessages(ctx context.Context) (int64, error) {
diff --git a/storage/queries/batch.sql b/storage/queries/batch.sql
index e78577f..5eb8127 100644
--- a/storage/queries/batch.sql
+++ b/storage/queries/batch.sql
@@ -1,7 +1,7 @@
 -- name: DeleteAllExpiredMessages :execrows
 DELETE FROM messages
-WHERE permanent != 0 AND expiration < CURRENT_TIMESTAMP;
+WHERE permanent != 1 AND expiration < CURRENT_TIMESTAMP;
 
 -- name: DeleteAllShutdownMessages :execrows
 DELETE FROM messages
-WHERE permanent != 0 AND shutdown > 0;
+WHERE permanent != 1 AND shutdown > 0;
-- 
GitLab