Proper handling of special attributes in jqlite

3 files changed, 23 insertions, 12 deletions

diff --git a/src/Angular.js b/src/Angular.js
index de6f2e5b..7249fb69 100644
--- a/src/Angular.js
+++ b/src/Angular.js
@@ -413,6 +413,19 @@ function isElement(node) {
 }
 
 /**
+ * @param str 'key1,key2,...'
+ * @returns {object} in the form of {key1:true, key2:true, ...}
+ */
+function makeMap(str){
+  var obj = {}, items = str.split(","), i;
+  for ( i = 0; i < items.length; i++ )
+    obj[ items[i] ] = true;
+  return obj;
+}
+
+
+
+/**
  * HTML class which is the only class which can be used in ng:bind to inline HTML for security reasons.
  * @constructor
  * @param html raw (unsafe) html
diff --git a/src/jqLite.js b/src/jqLite.js
index c86c265a..68cda632 100644
--- a/src/jqLite.js
+++ b/src/jqLite.js
@@ -84,6 +84,7 @@ function getStyle(element) {
   return current;
 }
 
+//TODO: delete me! dead code?
 if (msie) {
   extend(JQLite.prototype, {
     text: function(value) {
@@ -226,6 +227,8 @@ var JQLitePrototype = JQLite.prototype = {
 // these functions return self on setter and
 // value on get.
 //////////////////////////////////////////
+var SPECIAL_ATTR = makeMap("multiple,selected,checked,disabled,readonly");
+
 forEach({
   data: JQLiteData,
 
@@ -252,7 +255,13 @@ forEach({
   },
 
   attr: function(element, name, value){
-    if (isDefined(value)) {
+    if (SPECIAL_ATTR[name]) {
+      if (isDefined(value)) {
+        element[name] = !!value;
+      } else {
+        return element[name];
+      }
+    } else if (isDefined(value)) {
       element.setAttribute(name, value);
     } else if (element.getAttribute) {
       // the extra argument "2" is to get the right thing for a.href in IE, see jQuery code
diff --git a/src/sanitizer.js b/src/sanitizer.js
index 7bd26455..2cf126dc 100644
--- a/src/sanitizer.js
+++ b/src/sanitizer.js
@@ -187,17 +187,6 @@ function htmlParser( html, handler ) {
 }
 
 /**
- * @param str 'key1,key2,...'
- * @returns {object} in the form of {key1:true, key2:true, ...}
- */
-function makeMap(str){
-  var obj = {}, items = str.split(","), i;
-  for ( i = 0; i < items.length; i++ )
-    obj[ items[i] ] = true;
-  return obj;
-}
-
-/**
  * decodes all entities into regular string
  * @param value
  * @returns {string} A string with decoded entities.