From 86e25bc017df714910798a811fb108bb7aa840af Mon Sep 17 00:00:00 2001
From: Manos Pitsidianakis <manos@pitsidianak.is>
Date: Wed, 28 Aug 2024 10:25:02 +0300
Subject: [PATCH] sqlite: fix database reset sequence

Signed-off-by: Manos Pitsidianakis <manos@pitsidianak.is>
---
 melib/src/utils/sqlite3.rs | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/melib/src/utils/sqlite3.rs b/melib/src/utils/sqlite3.rs
index ad391754..54962547 100644
--- a/melib/src/utils/sqlite3.rs
+++ b/melib/src/utils/sqlite3.rs
@@ -179,7 +179,7 @@ impl DatabaseDescription {
                     }
                     self.reset_db()?;
                     second_try = true;
-                    return Ok(conn);
+                    return Ok(None);
                 }
 
                 if version == 0 {
@@ -190,12 +190,12 @@ impl DatabaseDescription {
                         .map_err(|err| Error::new(err.to_string()))?;
                 }
 
-                Ok(conn)
+                Ok(Some(conn))
             };
             inner_fn().unwrap();
             match inner_fn() {
-                Ok(_) if second_try => continue,
-                Ok(conn) => return Ok(conn),
+                Ok(None) => continue,
+                Ok(Some(conn)) => return Ok(conn),
                 Err(err) => {
                     return Err(Error::new(format!(
                         "{}: Could not open or create database",
@@ -219,6 +219,11 @@ impl DatabaseDescription {
                 .set_kind(ErrorKind::from(err.kind()))
                 .set_source(Some(Arc::new(err)))
         })?;
+        log::info!(
+            "{} {} database reset successful",
+            self.name,
+            db_path.display()
+        );
         Ok(())
     }
 }