From e7913736e53cababcbcff2d42537644527d13eb9 Mon Sep 17 00:00:00 2001 From: Philipp Hofer Date: Mon, 7 Apr 2025 23:04:41 +0200 Subject: [PATCH] clippy + fmt --- src/lib.rs | 4 ++-- src/route.rs | 50 ++++++++++++++++++++++--------------------- src/station/mod.rs | 14 ++++++------ src/station/routes.rs | 8 +++---- 4 files changed, 39 insertions(+), 37 deletions(-) diff --git a/src/lib.rs b/src/lib.rs index 3c5a69e..bfb6c5c 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -1,5 +1,5 @@ -use axum::{body::Body, response::Response, routing::get, Router}; -use maud::{html, Markup}; +use axum::{Router, body::Body, response::Response, routing::get}; +use maud::{Markup, html}; use partials::page; use sqlx::SqlitePool; use std::sync::Arc; diff --git a/src/route.rs b/src/route.rs index 08305cb..45f912c 100644 --- a/src/route.rs +++ b/src/route.rs @@ -1,11 +1,11 @@ use crate::{err, page, station::Station, succ}; use axum::{ + Form, Router, extract::State, response::{IntoResponse, Redirect}, routing::{get, post}, - Form, Router, }; -use maud::{html, Markup, PreEscaped}; +use maud::{Markup, PreEscaped, html}; use serde::{Deserialize, Serialize}; use sqlx::{FromRow, SqlitePool}; use std::sync::Arc; @@ -133,13 +133,13 @@ DROP TABLE temp_pos;", async fn stations(&self, db: &SqlitePool) -> Vec { sqlx::query_as::<_, Station>( - r#" - SELECT s.id, s.name, s.notes, s.amount_people, s.last_login, s.pw, s.lat, s.lng - FROM station s - JOIN route_station rs ON s.id = rs.station_id - WHERE rs.route_id = ? - ORDER BY rs.pos - "#, + " + SELECT s.id, s.name, s.notes, s.amount_people, s.last_login, s.pw, s.lat, s.lng + FROM station s + JOIN route_station rs ON s.id = rs.station_id + WHERE rs.route_id = ? + ORDER BY rs.pos + ", ) .bind(self.id) .fetch_all(db) @@ -149,16 +149,16 @@ DROP TABLE temp_pos;", async fn stations_not_in_route(&self, db: &SqlitePool) -> Vec { sqlx::query_as::<_, Station>( - r#" - SELECT id, name, notes, amount_people, last_login, pw, lat, lng - FROM station - WHERE id NOT IN ( - SELECT station_id - FROM route_station - WHERE route_id = ? - ) - ORDER BY name - "#, + " + SELECT id, name, notes, amount_people, last_login, pw, lat, lng + FROM station + WHERE id NOT IN ( + SELECT station_id + FROM route_station + WHERE route_id = ? + ) + ORDER BY name + ", ) .bind(self.id) .fetch_all(db) @@ -226,7 +226,7 @@ async fn create( Form(form): Form, ) -> impl IntoResponse { match Route::create(&db, &form.name).await { - Ok(_) => succ!(session,"Route '{}' erfolgreich erstellt!", form.name), + Ok(()) => succ!(session, "Route '{}' erfolgreich erstellt!", form.name), Err(e) => err!( session, "Route '{}' konnte _NICHT_ erstellt werden, da es bereits eine Route mit diesem Namen gibt ({e})!", @@ -252,7 +252,7 @@ async fn delete( }; match route.delete(&db).await { - Ok(_) => succ!(session, "Route '{}' erfolgreich gelöscht!", route.name), + Ok(()) => succ!(session, "Route '{}' erfolgreich gelöscht!", route.name), Err(e) => err!( session, "Route '{}' kann nicht gelöscht werden, da sie bereits verwendet wird. ({e})", @@ -403,7 +403,7 @@ async fn add_station( }; match route.add_station(&db, &station).await { - Ok(_) => succ!( + Ok(()) => succ!( session, "Station {} wurde der Route {} hinzugefügt", station.name, @@ -436,7 +436,8 @@ async fn delete_station( let Some(station) = Station::find_by_id(&db, station_id).await else { err!( session, - "Konnte Station mit der ID {station_id} nicht von der Route {} entfernen, da die Station nicht existiert.", route.name + "Konnte Station mit der ID {station_id} nicht von der Route {} entfernen, da die Station nicht existiert.", + route.name ); return Redirect::to(&format!("/route/{route_id}")); @@ -477,7 +478,8 @@ async fn move_station_higher( let Some(station) = Station::find_by_id(&db, station_id).await else { err!( session, - "Konnte Station mit der ID {station_id} nicht in der Route {} verschieben, da die Station nicht existiert.", route.name + "Konnte Station mit der ID {station_id} nicht in der Route {} verschieben, da die Station nicht existiert.", + route.name ); return Redirect::to(&format!("/route/{route_id}")); diff --git a/src/station/mod.rs b/src/station/mod.rs index 31a1847..c70b7e9 100644 --- a/src/station/mod.rs +++ b/src/station/mod.rs @@ -114,13 +114,13 @@ impl Station { async fn routes(&self, db: &SqlitePool) -> Vec { sqlx::query_as::<_, Route>( - r#" - SELECT r.id, r.name - FROM route r - JOIN route_station rs ON r.id = rs.route_id - WHERE rs.station_id = ? - ORDER BY rs.pos - "#, + " + SELECT r.id, r.name + FROM route r + JOIN route_station rs ON r.id = rs.route_id + WHERE rs.station_id = ? + ORDER BY rs.pos + ", ) .bind(self.id) .fetch_all(db) diff --git a/src/station/routes.rs b/src/station/routes.rs index 01488b2..818278f 100644 --- a/src/station/routes.rs +++ b/src/station/routes.rs @@ -1,11 +1,11 @@ use crate::{err, partials::page, station::Station, succ}; use axum::{ + Form, Router, extract::State, response::{IntoResponse, Redirect}, routing::{get, post}, - Form, Router, }; -use maud::{html, Markup}; +use maud::{Markup, html}; use serde::Deserialize; use sqlx::SqlitePool; use std::sync::Arc; @@ -22,7 +22,7 @@ async fn create( Form(form): Form, ) -> impl IntoResponse { match Station::create(&db, &form.name).await { - Ok(_) => succ!(session,"Station '{}' erfolgreich erstellt!", form.name), + Ok(()) => succ!(session, "Station '{}' erfolgreich erstellt!", form.name), Err(e) => err!( session, "Station '{}' konnte _NICHT_ erstellt werden, da es bereits eine Station mit diesem Namen gibt ({e})!", @@ -48,7 +48,7 @@ async fn delete( }; match station.delete(&db).await { - Ok(_) => succ!(session, "Station '{}' erfolgreich gelöscht!", station.name), + Ok(()) => succ!(session, "Station '{}' erfolgreich gelöscht!", station.name), Err(e) => err!( session, "Station '{}' kann nicht gelöscht werden, da sie bereits verwendet wird. ({e})",