Changeset 12090

Timestamp:

01/18/08 14:43:37 (12 months ago)

Author:

doughays

Message:

Fixes #5620 !strict. Added support to dojo.attr for boolean attributes on Firefox.

Files:

• dojo/trunk/_base/html.js (modified) (2 diffs)

dojo/trunk/_base/html.js

@@ -1012 +1012 @@
                 name = _fixAttrName(name);
                 if(arguments.length == 3){
-                        if(typeof value == "function"){ // e.g. onsubmit
+                        if(typeof value == "function" || (dojo.isFF && (typeof value == "boolean"))){ // e.g. onsubmit, disabled
                                 node[name] = value;
                         }else{
@@ -1022 +1022 @@
                         // via getAttribute()?
                         var prop = _attrProps[name.toLowerCase()];
-                        return prop ? node[prop] :
-                                dojo.hasAttr(node, name) ? node.getAttribute(name) : (node[name]/*function*/||null);
+                        if(prop){
+                                return node[prop];
+                        }else if(dojo.hasAttr(node, name) && !(dojo.isFF && (typeof node[name] == "boolean"))){
+                                return node.getAttribute(name);
+                        }else{
+                                var value = node[name];
+                                return (value === undefined ? null : value);
+                        }
                 }
         }