Require steamid for players and make player name optional in the database schema

Author: jenrik

migrations/002_update_player.sql

@@ -0,0 +1,3 @@
+ALTER TABLE players ALTER COLUMN steamid SET NOT NULL;
+
+ALTER TABLE players ALTER COLUMN name DROP NOT NULL;

src/database/models.rs

@@ -28,14 +28,14 @@ pub struct Player {
         deserialize_with = "deserialize_uuid"
     )]
     pub id: Uuid,
-    pub name: String,
+    pub name: Option<String>,
     #[serde(
         serialize_with = "serialize_uuid",
         deserialize_with = "deserialize_uuid"
     )]
     pub team_id: Uuid,
     pub tag: Option<String>,
-    pub steamid: Option<String>,
+    pub steamid: String,
 }
 
 #[derive(Deserialize, Serialize, Debug, Clone, sqlx::FromRow)]

src/web/get5.rs

@@ -12,14 +12,10 @@ use crate::get5::basic::{
 use crate::web::State;
 
 fn format_player(player: &Player) -> Option<Get5Player> {
-    if let Some(steamid) = player.steamid.clone() {
-        Some(Get5Player {
-            steamID: steamid,
-            name: Some(player.name.clone()), // TODO make player name optional in database
-        })
-    } else {
-        None
-    }
+    Some(Get5Player {
+        steamID: player.steamid.clone(),
+        name: player.name.clone(),
+    })
 }
 
 #[derive(Deserialize, Debug)]