Merge pull request #50 from yuliya-ivaniukovich/bool-props-fix

Fix boolean attributes parsing

Author: remarkablemark

lib/attributes-to-props.js

@@ -9,6 +9,9 @@ var styleToObject = require('style-to-object');
 var config = propertyConfig.config;
 var isCustomAttribute = propertyConfig.HTMLDOMPropertyConfig.isCustomAttribute;
 
+var DOMProperty = require('react-dom-core/lib/DOMProperty');
+DOMProperty.injection.injectDOMPropertyConfig(propertyConfig.HTMLDOMPropertyConfig);
+
 /**
  * Make attributes compatible with React props.
  *
@@ -34,7 +37,11 @@ function attributesToProps(attributes) {
         // make HTML DOM attribute/property consistent with React attribute/property
         reactProperty = config.html[propertyName.toLowerCase()];
         if (reactProperty) {
-            props[reactProperty] = propertyValue;
+            if (DOMProperty.properties.hasOwnProperty(reactProperty) && DOMProperty.properties[reactProperty].hasBooleanValue) {
+                props[reactProperty] = true;
+            } else {
+                props[reactProperty] = propertyValue;
+            }
             continue;
         }
 

test/attributes-to-props.js

@@ -102,6 +102,26 @@ describe('attributes to props helper', function() {
             );
         });
 
+        it('converts bool properties', function() {
+            assert.deepEqual(
+                attributesToProps({
+                    readonly: ''
+                }),
+                {
+                    readOnly: true
+                }
+            );
+
+            assert.deepEqual(
+                attributesToProps({
+                    disabled: 'disabled'
+                }),
+                {
+                    disabled: true
+                }
+            );
+        });
+
     });
 
     /**