JSON parsing of data-mkws-config attribute now protected by try/catch

[mkws-moved-to-github.git] / src / mkws-widgets.js

diff --git a/src/mkws-widgets.js b/src/mkws-widgets.js
index 2366614..c3a309d 100644 (file)
--- a/src/mkws-widgets.js
+++ b/src/mkws-widgets.js
@@ -1,9 +1,16 @@
 // Factory function for widget objects.
 function widget($, team, type, node) {
+    // Static register of attributes that do not contribute to config
+    var ignoreAttrs = {
+       id:1, class:1, style:1, name:1, action:1, type:1, size:1,
+       value:1, width:1, valign:1
+    };
+
     var that = {
        team: team,
        type: type,
-       node: node
+       node: node,
+       config: Object.create(team.config())
     };
 
     function log(s) {
@@ -15,6 +22,31 @@ function widget($, team, type, node) {
        return '[Widget ' + team.name() + ':' + type + ']';
     };
 
+    for (var i = 0; i < node.attributes.length; i++) {
+       var a = node.attributes[i];
+       if (a.name === 'data-mkws-config') {
+           // Treat as a JSON fragment configuring just this widget
+           log(node + ": parsing config fragment '" + a.value + "'");
+           var data;
+           try {
+               data = $.parseJSON(a.value);
+               for (var key in data) {
+                   log(node + ": adding config element " + key + "='" + data[key] + "'");
+                   that.config[key] = data[key];
+               }
+           } catch (err) {
+               alert("Can't parse " + node + " data-mkws-config as JSON: " + a.value);
+           }
+       } else if (a.name.match (/^data-mkws-/)) {
+           var name = a.name.replace(/^data-mkws-/, '')
+           that.config[name] = a.value;
+           log(node + ": set data-mkws attribute " + name + "='" + a.value + "'");
+       } else if (!ignoreAttrs[a.name]) {
+           that.config[a.name] = a.value;
+           log(node + ": set regular attribute " + a.name + "='" + a.value + "'");
+       }
+    }
+
     var fn = mkws.promotionFunction(type);
     if (fn) {
        fn.call(that);
@@ -87,7 +119,7 @@ mkws.registerWidgetType('Termlists', function() {
        }
 
        // no facets: this should never happen
-       var facets = mkws.config.facets;
+       var facets = that.config.facets;
        if (!facets || facets.length == 0) {
            alert("onTerm called even though we have no facets: " + $.toJSON(data));
            $(that.node).hide();