expandValue demoted from a member function to a private function.
authorMike Taylor <mike@indexdata.com>
Tue, 7 Oct 2014 12:01:05 +0000 (13:01 +0100)
committerMike Taylor <mike@indexdata.com>
Tue, 7 Oct 2014 12:01:05 +0000 (13:01 +0100)
(It's never called outside of the widget object.)

src/mkws-widget.js

index 9a3a7f7..d57620b 100644 (file)
@@ -66,8 +66,7 @@ function widget($, team, type, node) {
     return s.join('');
   };
 
-  // ### why is this a member function? It's never called from outside this file.
-  that.expandValue = function(val) {
+  function expandValue(val) {
     if (val.match(/^!param!/)) {
       var param = val.replace(/^!param!/, '');
       val = mkws.getParameterByName(param);
@@ -153,7 +152,7 @@ function widget($, team, type, node) {
 
   for (var i = 0; i < node.attributes.length; i++) {
     var a = node.attributes[i];
-    var val = that.expandValue(a.value);
+    var val = expandValue(a.value);
     if (a.name === 'data-mkws-config') {
       // Treat as a JSON fragment configuring just this widget
       log(node + ": parsing config fragment '" + val + "'");