show memos with expiration date first

src/storage.rs

@@ -53,16 +53,38 @@ pub fn open_sqlite_storage(path: &str) -> Result<SQLiteStorage, Error> {
 
 impl MemoStorage for SQLiteStorage {
 	fn all(&self) -> Result<Vec<Memo>, Error> {
-		let mut statement = self.conn.prepare("SELECT * FROM memo ORDER BY due, id")?;
-		let mut rows = statement.query([])?;
 		let mut results = Vec::new();
 
-		while let Some(row) = rows.next()? {
+		/*
-			results.push(Memo {
+		 * SQLite considers NULL as smallest value, so we will always get events with no due date
-				id: row.get(0)?,
+		 * first. To circumvent this, we first query all memos with a due date, and then all
-				body: row.get(1)?,
+		 * others. This is kinda jank but will do for now.
-				due: row.get(2)?,
+		 */
-			});
+
+		{
+			let mut statement = self.conn.prepare("SELECT * FROM memo WHERE due IS NOT NULL ORDER BY due, id")?;
+			let mut rows = statement.query([])?;
+
+			while let Some(row) = rows.next()? {
+				results.push(Memo {
+					id: row.get(0)?,
+					body: row.get(1)?,
+					due: row.get(2)?,
+				});
+			}
+		}
+
+		{
+			let mut statement = self.conn.prepare("SELECT * FROM memo WHERE due IS NULL ORDER BY due, id")?;
+			let mut rows = statement.query([])?;
+
+			while let Some(row) = rows.next()? {
+				results.push(Memo {
+					id: row.get(0)?,
+					body: row.get(1)?,
+					due: row.get(2)?,
+				});
+			}
 		}
 
 		return Ok(results);