Use is_busy for job update empty tables

src/app.rs

@@ -14,7 +14,7 @@ pub struct TwoDBApp {
 
     pub is_busy_old: bool, // This field is for Spinner
 
-    pub is_busy: Arc<Mutex<bool>>,
+    pub is_busy: Arc<Mutex<bool>>, // for synchronize thread
     pub field: Arc<Mutex<i128>>,
 }
 
@@ -41,7 +41,14 @@ impl TwoDBApp {
         // Note that you must enable the `persistence` feature for this to work.
         if let Some(storage) = cc.storage {
             let mut app: TwoDBApp = eframe::get_value(storage, eframe::APP_KEY).unwrap_or_default();
-            app.is_busy_old = false;
+
+            {
+                println!("App loaded");
+                /// Reset is_busy to false
+                let is_busy = app.is_busy.clone();
+                *is_busy.lock().unwrap() = false;
+            }
+
             return app;
         }
 
@@ -78,7 +85,7 @@ impl eframe::App for TwoDBApp {
                         self.render_get_empty_tables_button(ui);
                     });
 
-                    if self.is_busy_old.eq(&true) {
+                    if self.is_busy.lock().unwrap().clone() {
                         ui.add(egui::Spinner::new());
                     }
 
@@ -126,8 +133,8 @@ impl eframe::App for TwoDBApp {
     }
 
     /// Called by the framework to save state before shutdown.
+    /// Notes by Keios: this fn can be call despite the app is not closing
     fn save(&mut self, storage: &mut dyn eframe::Storage) {
-        self.is_busy_old = false;
         eframe::set_value(storage, eframe::APP_KEY, self);
     }
 }

src/widgets/update_empty_tables_button.rs

@@ -16,12 +16,12 @@ impl TwoDBApp {
     }
 
     fn get_empty_tables_event(&mut self) {
-        let field = self.field.clone();
+        let is_busy = self.is_busy.clone();
+        *is_busy.lock().unwrap() = true;
         thread::spawn(move || {
-            *field.lock().unwrap() += 1;
-            let mut thread_toasts = Toasts::new()
-                .anchor(Align2::RIGHT_BOTTOM, (-10.0, -10.0)) // 10 units from the bottom right corner
-                .direction(egui::Direction::BottomUp);
+            // let mut thread_toasts = Toasts::new()
+            //     .anchor(Align2::RIGHT_BOTTOM, (-10.0, -10.0)) // 10 units from the bottom right corner
+            //     .direction(egui::Direction::BottomUp);
 
             let database_name_source = var("POSTGRES_DB_SOURCE").unwrap_or(String::from(""));
             update_empty_tables(&database_name_source);
@@ -30,14 +30,16 @@ impl TwoDBApp {
             update_empty_tables(&database_name_target);
 
             let text = format!("Done Get **Empty** Tables for {} and {}", database_name_source, database_name_target);
-            thread_toasts.add(Toast {
-                text: text.into(),
-                kind: ToastKind::Success,
-                options: ToastOptions::default()
-                    .duration_in_seconds(5.0)
-                    .show_progress(true),
-                ..Default::default()
-            });
+            // thread_toasts.add(Toast {
+            //     text: text.clone().into(),
+            //     kind: ToastKind::Success,
+            //     options: ToastOptions::default()
+            //         .duration_in_seconds(5.0)
+            //         .show_progress(true),
+            //     ..Default::default()
+            // });
+            println!("{}", text);
+            *is_busy.lock().unwrap() = false;
         });
     }
 }

src/working_database.rs

@@ -1,7 +1,7 @@
 use std::env::var;
 use std::time::SystemTime;
 use postgres::error::SqlState;
-use rusqlite::{Connection, params};
+use rusqlite::{Connection};
 use crate::core::get_tables;
 use crate::database::connect;
 use crate::table::{build_base_simple_table, create_tables_table, insert_new_table, Table};