Fix: [Client][SettingsStore] サーバーから設定データを同期する際に値が変更されている場合のみストアを更新する

実際にはサーバー側で値が変更されていない場合でも、全 Pinia ストアに紐づくすべてのコンポーネントの再描画が発生してしまっていて (?)、結果パフォーマンスに大きく影響していた
tsukumijima · Nov 12, 2023 · 205df5b · 205df5b
1 parent 0b10402
commit 205df5b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/client/src/main.ts b/client/src/main.ts
@@ -105,7 +105,7 @@ settings_store.$subscribe(async () => {
     console.log('Client Settings Changed:', settings_store.settings);
     setLocalStorageSettings(settings_store.settings);
 
-    // 設定データをサーバーに同期する (ログイン時かつ同期が有効な場合のみ)
+    // このクライアントの設定をサーバーに同期する (ログイン時かつ同期が有効な場合のみ実行される)
     await settings_store.syncClientSettingsToServer();
 
 }, {detached: true});
@@ -115,7 +115,7 @@ settings_store.$subscribe(async () => {
 window.setInterval(async () => {
     if (Utils.getAccessToken() !== null && settings_store.settings.sync_settings === true) {
 
-        // 設定データをサーバーにアップロード
+        // サーバーに保存されている設定データをこのクライアントに同期する
         is_uploading_settings = true;
         await settings_store.syncClientSettingsFromServer();
         is_uploading_settings = false;

diff --git a/client/src/stores/SettingsStore.ts b/client/src/stores/SettingsStore.ts
@@ -401,8 +401,12 @@ const useSettingsStore = defineStore('settings', {
             }
 
             // クライアントの設定データをサーバーからの設定データで上書き
+            // 両者の値に変更がある場合のみ上書きする
+            // さもなければ、実際にはサーバー側で値が変更されていない場合でも定義されているストアに紐づく全てのコンポーネントの再描画が発生してしまう (?)
             for (const [settings_server_key, settings_server_value] of Object.entries(settings_server)) {
-                this.settings[settings_server_key] = settings_server_value;
+                if (JSON.stringify(this.settings[settings_server_key]) !== JSON.stringify(settings_server_value)) {
+                    this.settings[settings_server_key] = settings_server_value;
+                }
             }
         },