github/Bose-SoundTouch
1
0
Fork 0
mirror of https://github.com/gesellix/Bose-SoundTouch.git synced 2026-08-19 09:06:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(web): guard pushState in showSummary to break popstate loop

Browse Source 
skip history.pushState when the hash already matches, so popstate -> selectMigrationDevice -> showSummary no longer pushes a duplicate entry that traps browser-Back in an oscillation between identical `#tab-migration?<id>` entries.
This commit is contained in:
Tobias Gesellchen
2026-05-18 22:21:28 +02:00
parent fbbc0de55c
commit 48e8ff8352
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/service/handlers/web/js/script.js
+4 -1
View File
@@ -1743,7 +1743,10 @@ async function showSummary(deviceId) {
document.getElementById("migration-summary").style.display = "none";
return;
}
history.pushState(null, '', '#tab-migration?' + deviceId);
const newHash = '#tab-migration?' + deviceId;
if (window.location.hash !== newHash) {
history.pushState(null, '', newHash);
}
const targetUrl = document.getElementById("target-domain").value;
// Per-field URL overrides (Plan card). The summary endpoint uses