src/model/planned_event.rs

@@ -240,29 +240,11 @@ INNER JOIN trip_details ON planned_event.trip_details_id = trip_details.id",
         .unwrap(); //Okay, as planned_event can only be created with proper DB backing
     }
 
-    pub async fn delete(&self, db: &SqlitePool) -> Result<(), String> {
-        if Registration::all_rower(db, self.trip_details_id)
-            .await
-            .len()
-            > 0
-        {
-            return Err(
-                "Event kann nicht gelöscht werden, weil mind. 1 Ruderer angemeldet ist.".into(),
-            );
-        }
-        if Registration::all_cox(db, self.trip_details_id).await.len() > 0 {
-            return Err(
-                "Event kann nicht gelöscht werden, weil mind. 1 Steuerperson angemeldet ist."
-                    .into(),
-            );
-        }
-
+    pub async fn delete(&self, db: &SqlitePool) {
         sqlx::query!("DELETE FROM planned_event WHERE id = ?", self.id)
             .execute(db)
             .await
             .unwrap(); //Okay, as PlannedEvent can only be created with proper DB backing
-
-        Ok(())
     }
 
     pub async fn get_ics_feed(db: &SqlitePool) -> String {

src/tera/admin/planned_event.rs

@@ -78,13 +78,12 @@ async fn update(
 
 #[get("/planned-event/<id>/delete")]
 async fn delete(db: &State<SqlitePool>, id: i64, _admin: PlannedEventUser) -> Flash<Redirect> {
-    let Some(event) = PlannedEvent::find_by_id(db, id).await else {
-        return Flash::error(Redirect::to("/planned"), "PlannedEvent does not exist");
-    };
-
-    match event.delete(db).await {
-        Ok(()) => Flash::success(Redirect::to("/planned"), "Event gelöscht"),
-        Err(e) => Flash::error(Redirect::to("/planned"), e),
+    match PlannedEvent::find_by_id(db, id).await {
+        Some(planned_event) => {
+            planned_event.delete(db).await;
+            Flash::success(Redirect::to("/planned"), "Event gelöscht")
+        }
+        None => Flash::error(Redirect::to("/planned"), "PlannedEvent does not exist"),
     }
 }