From 4f1d84131247b4b0e3e1d00e43d21c035e845ce6 Mon Sep 17 00:00:00 2001 From: Rebecca Meritz Date: Sat, 11 Sep 2021 18:59:23 -0700 Subject: [PATCH 1/2] Allow users to change case of own username Usernames are case-insensitive so now one have the name `foo` and someone else have the name `FOO`, but user `foo` should be able to change their own name to `FOO` if they want to for display purposes. --- src/routes/users/change_username.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/routes/users/change_username.rs b/src/routes/users/change_username.rs index 95a08ca5..92e28087 100644 --- a/src/routes/users/change_username.rs +++ b/src/routes/users/change_username.rs @@ -45,7 +45,8 @@ pub async fn req( let mut set = doc! {}; if let Some(username) = &data.username { - if User::is_username_taken(&username).await? { + if username.to_lowercase() == user.username.to_lowercase() { + } else if User::is_username_taken(&username).await? { return Err(Error::UsernameTaken); } From a6d49d388f40709c5bcc0a288235b6c3398fb5fc Mon Sep 17 00:00:00 2001 From: Paul Makles Date: Fri, 17 Sep 2021 13:26:32 +0100 Subject: [PATCH 2/2] Update src/routes/users/change_username.rs Co-authored-by: Splatterxl --- src/routes/users/change_username.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/routes/users/change_username.rs b/src/routes/users/change_username.rs index 92e28087..5f906bca 100644 --- a/src/routes/users/change_username.rs +++ b/src/routes/users/change_username.rs @@ -45,8 +45,7 @@ pub async fn req( let mut set = doc! {}; if let Some(username) = &data.username { - if username.to_lowercase() == user.username.to_lowercase() { - } else if User::is_username_taken(&username).await? { + if (username.to_lowercase() != user.username.to_lowercase()) && User::is_username_taken(&username).await? { return Err(Error::UsernameTaken); }