From adf9853ce456bf6844ae91babd6359f7558df50b Mon Sep 17 00:00:00 2001 From: Jeremy Ruston Date: Tue, 20 Jan 2026 15:54:10 +0000 Subject: [PATCH] More joy of linting --- core/modules/filters.js | 2 +- core/modules/widgets/setmultiplevariables.js | 62 ++++++++++---------- 2 files changed, 32 insertions(+), 32 deletions(-) diff --git a/core/modules/filters.js b/core/modules/filters.js index f5e2af875..0a2e808a3 100644 --- a/core/modules/filters.js +++ b/core/modules/filters.js @@ -256,7 +256,7 @@ options: optional hashmap of options */ exports.compileFilter = function(filterString,options) { var defaultFilterRunPrefix = (options || {}).defaultFilterRunPrefix || "or"; - var cacheKey = filterString + '|' + defaultFilterRunPrefix; + var cacheKey = filterString + "|" + defaultFilterRunPrefix; if(!this.filterCache) { this.filterCache = Object.create(null); this.filterCacheCount = 0; diff --git a/core/modules/widgets/setmultiplevariables.js b/core/modules/widgets/setmultiplevariables.js index 5669dab80..3cb1c3c1c 100644 --- a/core/modules/widgets/setmultiplevariables.js +++ b/core/modules/widgets/setmultiplevariables.js @@ -12,7 +12,7 @@ Widget to set multiple variables at once from a list of names and a list of valu var Widget = require("$:/core/modules/widgets/widget.js").widget; var SetMultipleVariablesWidget = function(parseTreeNode,options) { - this.initialise(parseTreeNode,options); + this.initialise(parseTreeNode,options); }; /* @@ -24,52 +24,52 @@ SetMultipleVariablesWidget.prototype = new Widget(); Render this widget into the DOM */ SetMultipleVariablesWidget.prototype.render = function(parent,nextSibling) { - this.parentDomNode = parent; - this.computeAttributes(); - this.execute(); - this.renderChildren(parent,nextSibling); + this.parentDomNode = parent; + this.computeAttributes(); + this.execute(); + this.renderChildren(parent,nextSibling); }; /* Compute the internal state of the widget */ SetMultipleVariablesWidget.prototype.execute = function() { - // Setup our variables - this.setVariables(); - // Construct the child widgets - this.makeChildWidgets(); + // Setup our variables + this.setVariables(); + // Construct the child widgets + this.makeChildWidgets(); }; SetMultipleVariablesWidget.prototype.setVariables = function() { - // Set the variables - var self = this, - filterNames = this.getAttribute("$names",""), - filterValues = this.getAttribute("$values",""); - this.variableNames = []; - this.variableValues = []; - if(filterNames && filterValues) { - this.variableNames = this.wiki.filterTiddlers(filterNames,this,{defaultFilterRunPrefix: "all"}); - this.variableValues = this.wiki.filterTiddlers(filterValues,this,{defaultFilterRunPrefix: "all"}); - $tw.utils.each(this.variableNames,function(varname,index) { - self.setVariable(varname,self.variableValues[index]); - }); - } + // Set the variables + var self = this, + filterNames = this.getAttribute("$names",""), + filterValues = this.getAttribute("$values",""); + this.variableNames = []; + this.variableValues = []; + if(filterNames && filterValues) { + this.variableNames = this.wiki.filterTiddlers(filterNames,this,{defaultFilterRunPrefix: "all"}); + this.variableValues = this.wiki.filterTiddlers(filterValues,this,{defaultFilterRunPrefix: "all"}); + $tw.utils.each(this.variableNames,function(varname,index) { + self.setVariable(varname,self.variableValues[index]); + }); + } }; /* Refresh the widget by ensuring our attributes are up to date */ SetMultipleVariablesWidget.prototype.refresh = function(changedTiddlers) { - var filterNames = this.getAttribute("$names",""), - filterValues = this.getAttribute("$values",""), - variableNames = this.wiki.filterTiddlers(filterNames,this,{defaultFilterRunPrefix: "all"}), - variableValues = this.wiki.filterTiddlers(filterValues,this,{defaultFilterRunPrefix: "all"}); - if(!$tw.utils.isArrayEqual(this.variableNames,variableNames) || !$tw.utils.isArrayEqual(this.variableValues,variableValues)) { - this.refreshSelf(); - return true; - } - return this.refreshChildren(changedTiddlers); + var filterNames = this.getAttribute("$names",""), + filterValues = this.getAttribute("$values",""), + variableNames = this.wiki.filterTiddlers(filterNames,this,{defaultFilterRunPrefix: "all"}), + variableValues = this.wiki.filterTiddlers(filterValues,this,{defaultFilterRunPrefix: "all"}); + if(!$tw.utils.isArrayEqual(this.variableNames,variableNames) || !$tw.utils.isArrayEqual(this.variableValues,variableValues)) { + this.refreshSelf(); + return true; + } + return this.refreshChildren(changedTiddlers); }; exports["setmultiplevariables"] = SetMultipleVariablesWidget;