Prompt for database save if user modifies HOTP counter (#494)

Author: replydev

Cargo.lock

@@ -27,14 +27,14 @@ debug = false
 strip = "symbols"
 
 [dependencies]
-serde_json = "1.0.127"
-serde = { version = "1.0.209", features = ["derive"] }
+serde_json = "1.0.128"
+serde = { version = "1.0.210", features = ["derive"] }
 dirs = "5.0.1"
 rpassword = "7.3.1"
 data-encoding = "2.6.0"
 copypasta-ext = "0.4.4"
 zeroize = { version = "1.8.1", features = ["zeroize_derive"] }
-clap = { version = "4.5.16", features = ["derive"] }
+clap = { version = "4.5.17", features = ["derive"] }
 hmac = "0.12.1"
 sha1 = "0.10.6"
 sha2 = "0.10.8"

Cargo.toml

@@ -137,6 +137,7 @@ fn handle_counter_switch(app: &mut App, increment: bool) {
                 } else {
                     Some(counter.saturating_sub(1))
                 };
+                app.database.mark_modified();
                 app.tick(true);
             }
         }

src/interface/handlers/main_window.rs

@@ -1,11 +1,9 @@
 use super::otp_element::OTPDatabase;
-
-pub struct Migration<'a> {
+struct Migration<'a> {
     to_version: u16, // Database version which we are migrating on
     migration_function: &'a dyn Fn(&mut OTPDatabase) -> color_eyre::Result<()>, // Function to execute the migration
 }
-
-pub const MIGRATIONS_LIST: [Migration; 1] = [Migration {
+const MIGRATIONS_LIST: [Migration; 1] = [Migration {
     to_version: 2,
     migration_function: &migrate_to_2,
 }];

src/otp/migrations/mod.rs

@@ -39,7 +39,7 @@ fn get_default_db_path() -> PathBuf {
     data_dir()
         .map(PathBuf::from)
         .map(|p| p.join(XDG_PATH))
-        .map(|xdg| {
+        .inspect(|xdg| {
             if !xdg.exists() {
                 if let Some(home) = &home_path {
                     if home.exists() {
@@ -49,7 +49,6 @@ fn get_default_db_path() -> PathBuf {
                     }
                 }
             }
-            xdg
         })
         .or(home_path)
         .unwrap_or(portable_path)