Fix checked bug

Author: zackify

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-validify",
-  "version": "2.0.8",
+  "version": "2.0.9",
   "description": "Form validation made easy",
   "main": "dist/form.js",
   "directories": {

src/form.js

@@ -55,7 +55,7 @@ export default class Form extends React.Component {
     let values = { ...this.props.values, ...this.state.values };
 
     values[target.name] =
-      target.checked || target.checked === false
+      target.type === 'checkbox' || target.type === 'radio'
         ? target.checked
         : target.value;
 

src/input.js

@@ -3,16 +3,16 @@ import React from 'react';
 import getError from './get-error';
 
 const filters = {
-  checkbox: value => value || value === false,
-  radio: value => value || value === false,
-  default: value => value,
+  checkbox: value => value || false,
+  radio: value => value || false,
+  text: value => value,
 };
 
 const getValue = (type, values) => {
-  let filter = filters[type] || filters.default;
+  let filter = filters[type || 'text'];
 
-  let value = values.filter(filter)[0];
-  if (filter(value)) return value;
+  let value = filter(values.filter(filter)[0]);
+  if (value || value === false) return value;
 
   return '';
 };

tests/checkbox-radio.js

@@ -15,13 +15,17 @@ test('Input works with checkbox', () => {
 
   wrapper
     .find(Input)
-    .simulate('change', { target: { name: 'Awesome', checked: true } });
+    .simulate('change', {
+      target: { name: 'Awesome', checked: true, type: 'checkbox' },
+    });
 
   expect(wrapper.find(Input).props().value).toEqual(true);
 
   wrapper
     .find(Input)
-    .simulate('change', { target: { name: 'Awesome', checked: false } });
+    .simulate('change', {
+      target: { name: 'Awesome', checked: false, type: 'checkbox' },
+    });
 
   expect(wrapper.find(Input).props().value).toEqual(false);
 });

tests/inputs.js

@@ -15,7 +15,9 @@ test('Input is passed in value correctly', () => {
 
   wrapper
     .find(Input)
-    .simulate('change', { target: { name: 'Awesome', value: 'yes' } });
+    .simulate('change', {
+      target: { name: 'Awesome', value: 'yes', checked: false },
+    });
 
   expect(wrapper.find(Input).props().value).toEqual('yes');
 });