Fix error when toggling Auto Refresh (#416)

jlu5 · oetiker · web-flow · commit b6cc48636e35 · 2024-12-02T10:45:36.000+01:00
* Fix error when toggling Auto Refresh

The clearTimeout call was using the wrong parameter, leading to the error "Uncaught ReferenceError: reload is not defined"

* Update CHANGES

---------

Co-authored-by: Tobias Oetiker &lt;tobi@oetiker.ch&gt;
diff --git a/CHANGES b/CHANGES
@@ -1,3 +1,4 @@
+ - fix error when toggling autorefresh @jlu5
  - Support parsing IPv6 output in OpenSSHJunOSPing Plugin @zimage
  - Enhancement to SSHProbe allows specification of IPv4 vs IPv6 transport, #385
  - InfluxDB 2.x v1 (compatibility) api doesn't set the "WWW-Authenticate" header (violating rfc7235).
diff --git a/htdocs/js/smokeping.js b/htdocs/js/smokeping.js
@@ -150,7 +150,7 @@ Event.observe(
                 }, window.options.step * 1000);
                 $('refresh-button').style.textDecoration = "line-through";
             } else {
-                clearTimeout(reload);
+                clearTimeout(refresh);
                 localStorage.setItem("noRefresh", true);
                $('refresh-button').style.textDecoration = "none";
             }

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+ - fix error when toggling autorefresh @jlu5`
`1`	`2`	`- Support parsing IPv6 output in OpenSSHJunOSPing Plugin @zimage`
`2`	`3`	`- Enhancement to SSHProbe allows specification of IPv4 vs IPv6 transport, #385`
`3`	`4`	`- InfluxDB 2.x v1 (compatibility) api doesn't set the "WWW-Authenticate" header (violating rfc7235).`
Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,7 @@ Event.observe(`
`150`	`150`	`}, window.options.step * 1000);`
`151`	`151`	`$('refresh-button').style.textDecoration = "line-through";`
`152`	`152`	`} else {`
`153`		`- clearTimeout(reload);`
	`153`	`+ clearTimeout(refresh);`
`154`	`154`	`localStorage.setItem("noRefresh", true);`
`155`	`155`	`$('refresh-button').style.textDecoration = "none";`
`156`	`156`	`}`