Found the usage of undeclared variables JS-0125
Bug risk
8 months ago
—
8 months old
'$' is not defined To fix this, add jquery in
environment array of .deepsource.toml. Read more in our documentation https://deepsource.io/docs/analyzer/javascript68}
69
70function scrollActiveToc() {
71 var active = $('nav#toc a.active').get(0);72 if (active) {
73 active.scrollIntoView({ block: 'center' });
74 }'$' is not defined To fix this, add jquery in
environment array of .deepsource.toml. Read more in our documentation https://deepsource.io/docs/analyzer/javascript64}
65
66function queryShow() {
67 $('#search-hits').show(200);68}
69
70function scrollActiveToc() {'$' is not defined To fix this, add jquery in
environment array of .deepsource.toml. Read more in our documentation https://deepsource.io/docs/analyzer/javascript60}
61
62function queryClose() {
63 window.setTimeout(function () { $('#search-hits').hide(200); }, 500);64}
65
66function queryShow() {'$' is not defined To fix this, add jquery in
environment array of .deepsource.toml. Read more in our documentation https://deepsource.io/docs/analyzer/javascript54}
55
56function queryChange() {
57 if ($('#search-query').val() != '') {58 queryShow();
59 }
60}'$' is not defined To fix this, add jquery in
environment array of .deepsource.toml. Read more in our documentation https://deepsource.io/docs/analyzer/javascript44 addHeadingAnchors();
45 var anchor = href.split('#')[1];
46 if (anchor) {
47 var anchorlink = $(`main a[href="#${anchor}"]`).get(0)48 if (anchorlink) {
49 anchorlink.scrollIntoView({ behavior: 'smooth', block: 'start' });
50 }
Description
Variables that aren't defined, but accessed may throw reference errors at runtime.
Potential ReferenceErrors may result from misspellings of variable and parameter names, or accidental implicit globals (for example, forgetting the var keyword in a for loop initializer).
Any reference to an undeclared variable causes a warning, unless the variable is explicitly mentioned in a /*global ...*/ comment, or specified in the globals key in the configuration file.
A common use case for these is if you intentionally use globals that are defined elsewhere (e.g. in a script sourced from HTML).
Bad Practice
const foo = someFunction();
const bar = a + 1; // 'a' is undeclared
Recommended
/*global someFunction, a*/
/*eslint no-undef: "error"*/
const foo = someFunction();
const bar = a + 1;