jimmy/nas-tools
1
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix: resolve variable redeclaration in Terminal.svelte
Deploy Dashboard (Dev) / deploy-dev (push) Successful in 3m50s
Details

Browse Source 
Change 'const d' to 'let d' to allow reassignment in nested scopes
This commit is contained in:
Gan, Jimmy
2026-03-13 22:45:55 +08:00
parent 7b17a53cab
commit d69b744ae7
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
dashboard/frontend/src/routes/Terminal.svelte
+4 -4
View File
@@ -267,7 +267,7 @@
async function connect(forceRefresh = false) {
// Always refresh token before first connection to ensure fresh cookies
const d = tabData.get(tabId);
let d = tabData.get(tabId);
const isFirstConnect = !d?.hasConnectedOnce;
if (forceRefresh || isFirstConnect) {
@@ -343,7 +343,7 @@
};
ws.onclose = async (e) => {
const reason = e.reason || `Connection closed (code ${e.code})`;
const d = tabData.get(tabId);
d = tabData.get(tabId);
if (d) d.ws = null;
// Handle auth failures (code 1008)
@@ -386,13 +386,13 @@
}
};
ws.onerror = () => {
const d = tabData.get(tabId);
d = tabData.get(tabId);
if (d) d.ws = null;
if (!closedManually) scheduleReconnect("Connection failed");
else updateTab(tabId, { connected: false, error: "Connection failed" });
};
const d = tabData.get(tabId);
d = tabData.get(tabId);
if (d) {
d.ws = ws;
d.heartbeat = heartbeat;