Ticket #2844: dojo_0.4.3_formbind_bug.patch

src/io/ScriptSrcIO.js

@@ -119 +119 @@
                 var query = "";
                 
                 if(kwArgs["formNode"]){
-                        var ta = kwArgs.formNode.getAttribute("action");
+                        var ta = dojo.io._getAttribute(kwArgs.formNode, "action");
                         if((ta)&&(!kwArgs["url"])){ url = ta; }
-                        var tp = kwArgs.formNode.getAttribute("method");
+                        var tp = dojo.io._getAttribute(kwArgs.formNode, "method");
                         if((tp)&&(!kwArgs["method"])){ kwArgs.method = tp; }
                         query += dojo.io.encodeForm(kwArgs.formNode, kwArgs.encoding, kwArgs["formFilter"]);
                 }

src/io/BrowserIO.js

@@ -48 +48 @@
         dojo.io.bind(args);
 }
 
+dojo.io._getAttribute = function(node, key){
+        //      summary:        Function to get around IE problems with getAttribute.
+        var attrNode = node.getAttributeNode(key);
+        if(attrNode){
+                return attrNode.value;
+        }
+        return null;
+}
+
+
 dojo.io.formFilter = function(/*DOMNode*/node) {
         //summary: Returns true if the node is an input element that is enabled, has
         //a name, and whose type is one of the following values: ["file", "submit", "image", "reset", "button"]
@@ -476 +486 @@
                 var url = kwArgs.url;
                 var query = "";
                 if(kwArgs["formNode"]){
-                        var ta = kwArgs.formNode.getAttribute("action");
+                        var ta = dojo.io._getAttribute(kwArgs.formNode, "action");
                         if((ta)&&(!kwArgs["url"])){ url = ta; }
-                        var tp = kwArgs.formNode.getAttribute("method");
+                        var tp = dojo.io._getAttribute(kwArgs.formNode, "method");
                         if((tp)&&(!kwArgs["method"])){ kwArgs.method = tp; }
                         query += dojo.io.encodeForm(kwArgs.formNode, kwArgs.encoding, kwArgs["formFilter"]);
                 }

src/io/IframeIO.js

@@ -107 +107 @@
                                         }
                                 }
                                 if(cr["url"]){
-                                        cr._originalAction = fn.getAttribute("action");
+                                        cr._originalAction = dojo.io._getAttribute(fn, "action");
                                         fn.setAttribute("action", cr.url);
                                 }
-                                if(!fn.getAttribute("method")){
+                                if(!dojo.io._getAttribute(fn, "method")){
                                         fn.setAttribute("method", (cr["method"]) ? cr["method"] : "post");
                                 }
                                 cr._originalTarget = fn.getAttribute("target");