From bb8a012b1ca6350b134df11280abea42929cb528 Mon Sep 17 00:00:00 2001
From: Chocobo1 <Chocobo1@users.noreply.github.com>
Date: Sun, 18 Feb 2024 14:16:00 +0800
Subject: [PATCH] Revise cache internals

Now cache initialization and `get()` is less costly to use and it shifts the weight to `set()`.

PR #20430.
---
 src/webui/www/private/scripts/cache.js | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/src/webui/www/private/scripts/cache.js b/src/webui/www/private/scripts/cache.js
index aab1433be..ad89a26c9 100644
--- a/src/webui/www/private/scripts/cache.js
+++ b/src/webui/www/private/scripts/cache.js
@@ -40,6 +40,18 @@ window.qBittorrent.Cache = (() => {
         };
     };
 
+    const deepFreeze = (obj) => {
+        // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/freeze#examples
+
+        const keys = Reflect.ownKeys(obj);
+        for (const key of keys) {
+            const value = obj[key];
+            if ((value && (typeof value === 'object')) || (typeof value === 'function'))
+                deepFreeze(value);
+        }
+        Object.freeze(obj);
+    };
+
     class BuildInfoCache {
         #m_store = {};
 
@@ -51,13 +63,15 @@ window.qBittorrent.Cache = (() => {
                 onSuccess: (responseJSON) => {
                     if (!responseJSON)
                         return;
+
+                    deepFreeze(responseJSON);
                     this.#m_store = responseJSON;
                 }
             }).send();
         }
 
         get() {
-            return structuredClone(this.#m_store);
+            return this.#m_store;
         }
     }
 
@@ -80,7 +94,9 @@ window.qBittorrent.Cache = (() => {
                 onSuccess: (responseJSON, responseText) => {
                     if (!responseJSON)
                         return;
-                    this.#m_store = structuredClone(responseJSON);
+
+                    deepFreeze(responseJSON);
+                    this.#m_store = responseJSON;
 
                     if (typeof obj.onSuccess === 'function')
                         obj.onSuccess(responseJSON, responseText);
@@ -89,7 +105,7 @@ window.qBittorrent.Cache = (() => {
         }
 
         get() {
-            return structuredClone(this.#m_store);
+            return this.#m_store;
         }
 
         // obj: {
@@ -114,6 +130,7 @@ window.qBittorrent.Cache = (() => {
                         obj.onFailure(xhr);
                 },
                 onSuccess: (responseText, responseXML) => {
+                    this.#m_store = structuredClone(this.#m_store);
                     for (const key in obj.data) {
                         if (!Object.hasOwn(obj.data, key))
                             continue;
@@ -121,6 +138,7 @@ window.qBittorrent.Cache = (() => {
                         const value = obj.data[key];
                         this.#m_store[key] = value;
                     }
+                    deepFreeze(this.#m_store);
 
                     if (typeof obj.onSuccess === 'function')
                         obj.onSuccess(responseText, responseXML);