From 69c4d965e30aef9abd938ea29ddd238c72e059e3 Mon Sep 17 00:00:00 2001
From: anatawa12 <anatawa12@icloud.com>
Date: Sun, 4 Aug 2024 00:13:02 +0900
Subject: [PATCH] Revert empty name (#219)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* Revert "fix(backend): 名前を空白文字列だけにできる問題を修正 (#14119)"

This reverts commit 9ef6c4716ca25d8adeb5344c82a433370a086f88.

* docs(changelog): ユーザー名の空白処理をリバーと
---
 CHANGELOG.md                                        |  2 ++
 .../backend/src/server/api/endpoints/i/update.ts    |  9 +--------
 packages/backend/test/e2e/endpoints.ts              | 13 ++-----------
 3 files changed, 5 insertions(+), 19 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index bc6d46d59bb4..ecac67b1e1c3 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -9,6 +9,8 @@
 	- ドライブにアップロードする画像の初期設定(設定/ドライブ)の`画像を縮小する`が初めから無効になります。
 
 ### Server
+- Feat: ユーザー名の前後に空白文字列がある場合も省略しないように
+- Feat: プロフィール編集時に名前を空白文字列のみにできるように
 
 ## 2024.7.0 (merged to 2024.7.0-kinel.1)
 
diff --git a/packages/backend/src/server/api/endpoints/i/update.ts b/packages/backend/src/server/api/endpoints/i/update.ts
index a1e2fa5e4cdf..91e5e15e19fd 100644
--- a/packages/backend/src/server/api/endpoints/i/update.ts
+++ b/packages/backend/src/server/api/endpoints/i/update.ts
@@ -258,14 +258,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
 			const profile = await this.userProfilesRepository.findOneByOrFail({ userId: user.id });
 			let policies: RolePolicies | null = null;
 
-			if (ps.name !== undefined) {
-				if (ps.name === null) {
-					updates.name = null;
-				} else {
-					const trimmedName = ps.name.trim();
-					updates.name = trimmedName === '' ? null : trimmedName;
-				}
-			}
+			if (ps.name !== undefined) updates.name = ps.name;
 			if (ps.description !== undefined) profileUpdates.description = ps.description;
 			if (ps.lang !== undefined) profileUpdates.lang = ps.lang;
 			if (ps.location !== undefined) profileUpdates.location = ps.location;
diff --git a/packages/backend/test/e2e/endpoints.ts b/packages/backend/test/e2e/endpoints.ts
index 5aaec7f6f9d0..32384b83fb9b 100644
--- a/packages/backend/test/e2e/endpoints.ts
+++ b/packages/backend/test/e2e/endpoints.ts
@@ -117,21 +117,12 @@ describe('Endpoints', () => {
 			assert.strictEqual(res.body.birthday, myBirthday);
 		});
 
-		test('名前を空白のみにした場合nullになる', async () => {
+		test('名前を空白にできる', async () => {
 			const res = await api('i/update', {
 				name: ' ',
 			}, alice);
 			assert.strictEqual(res.status, 200);
-			assert.strictEqual(res.body.name, null);
-		});
-
-		test('名前の前後に空白（ホワイトスペース）を入れてもトリムされる', async () => {
-			const res = await api('i/update', {
-				// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Lexical_grammar#white_space
-				name: ' あ い う \u0009\u000b\u000c\u0020\u00a0\u1680\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200a\u202f\u205f\u3000\ufeff',
-			}, alice);
-			assert.strictEqual(res.status, 200);
-			assert.strictEqual(res.body.name, 'あ い う');
+			assert.strictEqual(res.body.name, ' ');
 		});
 
 		test('誕生日の設定を削除できる', async () => {