www

commit cb3542b2966b11fd7d8b4845145e176f51fcf133
parent d8c06eb29366e1063f29995519d5c6810d753253
Author: Davide P. Cervone <dpvc@union.edu>
Date:   Mon, 29 Apr 2013 16:54:53 -0400

Use a service routine in MathML input to filter attributes (rather than override AddAttributes).  Also, make fontsize default allow all (authors can restrict if they want to).

Diffstat:
M
unpacked/extensions/Safe.js
 | 
55
++++++++++++++++++-------------------------------------
M
unpacked/jax/input/MathML/jax.js
 | 
14
+++++++++-----

2 files changed, 27 insertions(+), 42 deletions(-)
diff --git a/unpacked/extensions/Safe.js b/unpacked/extensions/Safe.js
@@ -35,7 +35,7 @@
       classes: "safe",   // safe start with MJX-
       cssIDs: "safe",    // safe start with MJX-
       styles: "safe",    // safe are in safeStyles below
-      fontsize: "safe",  // safe are between sizeMin and sizeMax em's
+      fontsize: "all",   // safe are between sizeMin and sizeMax em's
       require: "safe",   // safe are in safeRequire below
     },
     sizeMin: .7,        // \scriptsize
@@ -164,6 +164,9 @@
         {size = Math.min(Math.max(size,CONFIG.sizeMin),CONFIG.sizeMax)}
       return size;
     },
+    filterFontSize: function (size) {
+      return (ALLOW.fontsize === "all" ? size: null);
+    },
     
     //
     //  Filter TeX extension names
@@ -287,47 +290,25 @@
   });
   
   HUB.Register.StartupHook("MathML Jax Ready",function () {
-    var MATHML = MathJax.InputJax.MathML,
-        ATTRIBUTES = MATHML.Parse.prototype.AddAttributes;
+    var PARSE = MathJax.InputJax.MathML.Parse;
+    
+    var METHOD = {
+      href:     "filterURL",
+      "class":  "filterClass", 
+      id:       "filterID",
+      fontsize: "filterFontSize",
+      style:    "filterStyles"
+    };
 
     //
     //  Filter MathML attributes
     //
-    MATHML.Parse.prototype.AddAttributes = function (mml,node) {
-      ATTRIBUTES.apply(MATHML,arguments);
-      for (var i = 0, m = mml.attrNames.length; i < m; i++) {
-        var name = mml.attrNames[i], value = null;
-        switch (name) {
-          case "href":
-            value = SAFE.filterURL(mml.href);
-            break;
-
-          case "class":
-            value = SAFE.filterClass(mml["class"]);
-            break;
-
-          case "id":
-            value = SAFE.filterID(mml.id);
-            break;
-
-          case "fontsize":
-            value = (ALLOW.fontsize === "all" ? mml.fontsize: null);
-            break;
-
-          case "style":
-            value = SAFE.filterStyles(mml.style);
-            break;
-
-          default:
-            name = null;
-            break;
-        }
-        if (name) {
-          if (value) {mml[name] = value} 
-            else {delete mml[name]; mml.attrNames.splice(i,1); m--}
-        }
+    PARSE.Augment({
+      filterAttribute: function (name,value) {
+        if (METHOD[name]) {value = SAFE[METHOD[name]](value)}
+        return value;
       }
-    };
+    });
 
   });
   
diff --git a/unpacked/jax/input/MathML/jax.js b/unpacked/jax/input/MathML/jax.js
@@ -121,13 +121,17 @@
         if (name.match(/:/)) continue;
         if (name.match(/^_moz-math-((column|row)(align|line)|font-style)$/)) continue;
         var value = node.attributes[i].value;
-        if (value.toLowerCase() === "true") {value = true}
-          else if (value.toLowerCase() === "false") {value = false}
-        if (mml.defaults[name] != null || MML.copyAttributes[name])
-          {mml[name] = value} else {mml.attr[name] = value}
-        mml.attrNames.push(name);
+        value = this.filterAttribute(name,value);
+        if (value != null) {
+          if (value.toLowerCase() === "true") {value = true}
+            else if (value.toLowerCase() === "false") {value = false}
+          if (mml.defaults[name] != null || MML.copyAttributes[name])
+            {mml[name] = value} else {mml.attr[name] = value}
+          mml.attrNames.push(name);
+        }
       }
     },
+    filterAttribute: function (name,value) {return value}, // safe mode overrides this
     
     //
     //  Create the children for the mml node