allow to create users

src/model/user/basic.rs

@@ -20,7 +20,7 @@ impl User {
         let note = note.trim();
 
         ActivityBuilder::new(&format!("({updated_by}) {note}"))
-            .relevant_for_user(&user)
+            .relevant_for_user(user)
             .save(db)
             .await;
 
@@ -48,7 +48,9 @@ impl User {
 
         let msg = match &self.mail {
             Some(old_mail) => {
-                format!("{updated_by} hat die Mail-Adresse von {self} von {old_mail} auf {new_mail} geändert.")
+                format!(
+                    "{updated_by} hat die Mail-Adresse von {self} von {old_mail} auf {new_mail} geändert."
+                )
             }
             None => {
                 format!("{updated_by} eine neue Mail-Adresse für {self} hinzugefügt: {new_mail}")
@@ -87,9 +89,15 @@ impl User {
         query.execute(db).await.unwrap(); //Okay, because we can only create a User of a valid id
 
         let msg = match &self.phone {
-            Some(old_phone) if new_phone.is_empty() => format!("{updated_by} hat die Telefonnummer von {self} entfernt (alte Nummer: {old_phone})"),
-            Some(old_phone) => format!("{updated_by} hat die Telefonnummer von {self} von {old_phone} auf {new_phone} geändert."),
-            None => format!("{updated_by} hat eine neue Telefonnummer für {self} hinzugefügt: {new_phone}")
+            Some(old_phone) if new_phone.is_empty() => format!(
+                "{updated_by} hat die Telefonnummer von {self} entfernt (alte Nummer: {old_phone})"
+            ),
+            Some(old_phone) => format!(
+                "{updated_by} hat die Telefonnummer von {self} von {old_phone} auf {new_phone} geändert."
+            ),
+            None => format!(
+                "{updated_by} hat eine neue Telefonnummer für {self} hinzugefügt: {new_phone}"
+            ),
         };
 
         ActivityBuilder::new(&msg)
@@ -107,7 +115,7 @@ impl User {
         let new_address = new_address.trim();
 
         let query = if new_address.is_empty() {
-            if !self.address.is_none() {
+            if self.address.is_none() {
                 return; // nothing to do
             }
             sqlx::query!("UPDATE user SET address = NULL where id = ?", self.id)
@@ -126,9 +134,13 @@ impl User {
         query.execute(db).await.unwrap(); //Okay, because we can only create a User of a valid id
 
         let msg = match &self.address {
-            Some(old_address) if new_address.is_empty() => format!("{updated_by} hat die Adresse von {self} entfernt (alte Adresse: {old_address})"),
-            Some(old_address) => format!("{updated_by} hat die Adresse von {self} von {old_address} auf {new_address} geändert."),
-            None => format!("{updated_by} hat eine Adresse für {self} hinzugefügt: {new_address}")
+            Some(old_address) if new_address.is_empty() => format!(
+                "{updated_by} hat die Adresse von {self} entfernt (alte Adresse: {old_address})"
+            ),
+            Some(old_address) => format!(
+                "{updated_by} hat die Adresse von {self} von {old_address} auf {new_address} geändert."
+            ),
+            None => format!("{updated_by} hat eine Adresse für {self} hinzugefügt: {new_address}"),
         };
 
         ActivityBuilder::new(&msg)
@@ -157,9 +169,15 @@ impl User {
         query.execute(db).await.unwrap(); //Okay, because we can only create a User of a valid id
 
         let msg = match &self.nickname {
-            Some(old_nickname) if new_nickname.is_empty() => format!("{updated_by} hat den Sitznamen von {self} entfernt (alter Spitzname: {old_nickname})"),
-            Some(old_nickname) => format!("{updated_by} hat den Spitznamen von {self} von {old_nickname} auf {new_nickname} geändert."),
-            None => format!("{updated_by} hat einen neuen Spitznamen für {self} hinzugefügt: {new_nickname}")
+            Some(old_nickname) if new_nickname.is_empty() => format!(
+                "{updated_by} hat den Sitznamen von {self} entfernt (alter Spitzname: {old_nickname})"
+            ),
+            Some(old_nickname) => format!(
+                "{updated_by} hat den Spitznamen von {self} von {old_nickname} auf {new_nickname} geändert."
+            ),
+            None => format!(
+                "{updated_by} hat einen neuen Spitznamen für {self} hinzugefügt: {new_nickname}"
+            ),
         };
         ActivityBuilder::new(&msg)
             .relevant_for_user(self)
@@ -185,8 +203,12 @@ impl User {
         .unwrap(); //Okay, because we can only create a User of a valid id
 
         let msg = match &self.member_since_date {
-            Some(old_member_since_date) => format!("{updated_by} hat das Beitrittsdatum von {self} von {old_member_since_date} auf {new_member_since_date} geändert."),
-            None => format!("{updated_by} hat ein neues Beitrittsdatum für {self} hinzugefügt: {new_member_since_date}")
+            Some(old_member_since_date) => format!(
+                "{updated_by} hat das Beitrittsdatum von {self} von {old_member_since_date} auf {new_member_since_date} geändert."
+            ),
+            None => format!(
+                "{updated_by} hat ein neues Beitrittsdatum für {self} hinzugefügt: {new_member_since_date}"
+            ),
         };
 
         ActivityBuilder::new(&msg)
@@ -210,9 +232,13 @@ impl User {
         .await
         .unwrap(); //Okay, because we can only create a User of a valid id
 
-        let msg = match &self.birthdate{
-            Some(old_birthdate) => format!("{updated_by} hat das Geburtsdatum von {self} von {old_birthdate} auf {new_birthdate} geändert."),
-            None => format!("{updated_by} hat ein Geburtsdatum für {self} hinzugefügt: {new_birthdate}")
+        let msg = match &self.birthdate {
+            Some(old_birthdate) => format!(
+                "{updated_by} hat das Geburtsdatum von {self} von {old_birthdate} auf {new_birthdate} geändert."
+            ),
+            None => {
+                format!("{updated_by} hat ein Geburtsdatum für {self} hinzugefügt: {new_birthdate}")
+            }
         };
 
         ActivityBuilder::new(&msg)
@@ -272,7 +298,7 @@ impl User {
         let bootsfuehrer = Role::find_by_name(db, "Bootsführer").await.unwrap();
 
         match (old_skill, skill) {
-            (old, new) if old == None && new == Some(cox.clone()) => {
+            (None, new) if new == Some(cox.clone()) => {
                 self.add_role(db, updated_by, &cox).await?;
                 Notification::create_for_role(
                     db,
@@ -309,7 +335,7 @@ impl User {
                     .save(db)
                     .await;
             }
-            (old, new) if new == None => {
+            (old, None) => {
                 if let Some(old) = old {
                     self.remove_role(db, updated_by, &old).await?;
                     let vorstand = Role::find_by_name(db, "Vorstand").await.unwrap();
@@ -360,7 +386,7 @@ impl User {
         ActivityBuilder::new(&format!(
             "{updated_by} hat die Ermäßigung von {self} von {old} auf {new} geändert"
         ))
-        .relevant_for_user(&self)
+        .relevant_for_user(self)
         .save(db)
         .await;
 
@@ -374,7 +400,9 @@ impl User {
         role: &Role,
     ) -> Result<(), String> {
         if !self.has_role(db, &role.name).await {
-            return Err(format!("Kann Rolle {role} von User {self} nicht entfernen, da der User die Rolle gar nicht hat"));
+            return Err(format!(
+                "Kann Rolle {role} von User {self} nicht entfernen, da der User die Rolle gar nicht hat"
+            ));
         }
 
         sqlx::query!(
@@ -389,7 +417,7 @@ impl User {
         ActivityBuilder::new(&format!(
             "{updated_by} hat die Rolle {role} von {self} entfernt."
         ))
-        .relevant_for_user(&self)
+        .relevant_for_user(self)
         .save(db)
         .await;
 
@@ -415,7 +443,7 @@ impl User {
         ActivityBuilder::new(&format!(
             "{updated_by} hat den Bezahlstatus von {self} auf 'nicht bezahlt' gesetzt."
         ))
-        .relevant_for_user(&self)
+        .relevant_for_user(self)
         .save(db)
         .await;
     }
@@ -438,7 +466,7 @@ impl User {
         ActivityBuilder::new(&format!(
             "{updated_by} hat den Bezahlstatus von {self} auf 'bezahlt' gesetzt."
         ))
-        .relevant_for_user(&self)
+        .relevant_for_user(self)
         .save(db)
         .await;
     }
@@ -450,7 +478,9 @@ impl User {
         role: &Role,
     ) -> Result<(), String> {
         if self.has_role(db, &role.name).await {
-            return Err(format!("Kann Rolle {role} von User {self} nicht hinzufügen, da der User die Rolle schon hat"));
+            return Err(format!(
+                "Kann Rolle {role} von User {self} nicht hinzufügen, da der User die Rolle schon hat"
+            ));
         }
 
         sqlx::query!(
@@ -473,7 +503,7 @@ impl User {
             ActivityBuilder::new(&format!(
                 "{updated_by} hat die Rolle '{role}' dem Benutzer {self} hinzugefügt."
             ))
-            .relevant_for_user(&self)
+            .relevant_for_user(self)
             .save(db)
             .await;
         }
@@ -491,7 +521,7 @@ impl User {
             return Err(format!("User {self} hat bereits eine Beitrittserklärung."));
         }
         if membership_pdf.len() == 0 {
-            return Err(format!("Keine Beitrittserklärung mitgeschickt."));
+            return Err("Keine Beitrittserklärung mitgeschickt.".to_string());
         }
 
         let mut stream = membership_pdf.open().await.unwrap();
@@ -509,7 +539,7 @@ impl User {
         ActivityBuilder::new(&format!(
             "{updated_by} hat die Mitgliedserklärung (PDF) für user {self} hinzugefügt."
         ))
-        .relevant_for_user(&self)
+        .relevant_for_user(self)
         .save(db)
         .await;