Don’t create an event if repetition is not valid

1 files changed, 15 insertions, 11 deletions

diff --git a/src/gui/form/mod.rs b/src/gui/form/mod.rs
index 4bf6f90..57ccac7 100644
--- a/src/gui/form/mod.rs
+++ b/src/gui/form/mod.rs
@@ -81,19 +81,23 @@ pub async fn show(app: &App, event: Event, is_new: bool) {
     let conn = app.conn.clone();
     let tx = app.tx.clone();
     button.connect_clicked(glib::clone!(@weak dialog, @strong event => move |_| {
-        let repetition = repetition::validate(&repetition_model);
-        match event::validate(event.id, date.buffer().text(), name.buffer().text(), start.buffer().text(), end.buffer().text(), repetition) {
-            Some(new) => {
-                match if is_new { db::insert(&conn, &new) } else { db::update(&conn, &new) } {
-                    Ok(_) => {
-                        let msg = if is_new { Msg::AddEvent { new } } else { Msg::UpdateEvent { old: event.clone(), new } };
-                        update::send(tx.clone(), msg);
-                        dialog.close()
-                    },
-                    Err(err) => eprintln!("Error when upserting event: {err}")
+        match repetition::validate(&repetition_model) {
+            Ok(repetition) => {
+                match event::validate(event.id, date.buffer().text(), name.buffer().text(), start.buffer().text(), end.buffer().text(), repetition) {
+                    Some(new) => {
+                        match if is_new { db::insert(&conn, &new) } else { db::update(&conn, &new) } {
+                            Ok(_) => {
+                                let msg = if is_new { Msg::AddEvent { new } } else { Msg::UpdateEvent { old: event.clone(), new } };
+                                update::send(tx.clone(), msg);
+                                dialog.close()
+                            },
+                            Err(err) => eprintln!("Error when upserting event: {err}")
+                        }
+                    }
+                    None => eprintln!("Event is not valid: {event:?}")
                 }
             },
-            None => eprintln!("Event is not valid: {event:?}")
+            Err(message) => eprintln!("{message}")
         }
     }));