Merge pull request 'don't respond with 500 if no rower is selected' (#451 ) from require-user-for-logentry into main

Reviewed-on: #451
don't respond with 500 if no rower is selected
2024-04-28 19:25:33 +02:00 · 2024-04-28 19:16:38 +02:00
1 changed files with 7 additions and 1 deletions
--- a/src/tera/log.rs
+++ b/src/tera/log.rs
@ -250,7 +250,13 @@ async fn create_kiosk(
    } else if let Some(shipmaster) = data.shipmaster {
        User::find_by_id(db, shipmaster as i32).await.unwrap()
    } else {
-        User::find_by_id(db, data.rowers[0] as i32).await.unwrap()
+        let Some(rower) = data.rowers.get(0) else {
+            return Flash::error(
+                Redirect::to("/log"),
+                "Ausfahrt ohne Benutzer kann nicht angelegt werden.",
+            );
+        };
+        User::find_by_id(db, *rower as i32).await.unwrap()
    };
    Log::create(
        db,
Author	SHA1	Message	Date
philipp	d01895df90	Merge pull request 'don't respond with 500 if no rower is selected' (#451 ) from require-user-for-logentry into main All checks were successful CI/CD Pipeline / test (push) Successful in 7m46s Details CI/CD Pipeline / deploy-staging (push) Has been skipped Details CI/CD Pipeline / deploy-main (push) Successful in 6m18s Details Reviewed-on: #451	2024-04-28 19:25:33 +02:00
philipp	c27a2ad15e	don't respond with 500 if no rower is selected All checks were successful CI/CD Pipeline / test (push) Successful in 8m43s Details CI/CD Pipeline / deploy-staging (push) Has been skipped Details CI/CD Pipeline / deploy-main (push) Has been skipped Details	2024-04-28 19:16:38 +02:00