Ignored all default value props (#135)

Author: erikras

README.md

@@ -89,6 +89,14 @@ MyForm = reduxForm({
 export default MyForm
 ```
 
+## Quirks
+
+Because of the strict "controlled component" nature of `redux-form`,
+some of the Material UI functionality related to defaulting of values has been disabled
+e.g. `defaultValue`, `defaultDate`, `defaultTime`, `defaultToggled`, `defaultChecked`, etc.
+If you need a field to be initialized to a certain state, you should use the `initialValues`
+API of `redux-form`.
+
 ## Instance API
 
 #### `getRenderedComponent()`

src/Checkbox.js

@@ -5,6 +5,7 @@ export default createComponent(Checkbox, ({
   input: { onChange, value, ...inputProps },
   meta,
   onCheck,
+  defaultChecked,
   ...props
 }) => ({
   ...inputProps,

src/DatePicker.js

@@ -4,6 +4,7 @@ import mapError from './mapError'
 
 export default createComponent(DatePicker, ({
   input: { onBlur, ...inputProps },
+  defaultDate,
   onChange,
   ...props
 }) => ({

src/Slider.js

@@ -5,6 +5,7 @@ import mapError from './mapError'
 export default createComponent(Slider, ({
   input: { onDragStart, onChange, ...inputProps },
   onChange: onChangeFromField,
+  defaultValue,
   ...props
 }) => ({
   // eslint-disable-line no-unused-vars

src/TimePicker.js

@@ -4,6 +4,7 @@ import mapError from './mapError'
 
 export default createComponent(TimePicker, ({
   input: { onBlur, ...inputProps },
+  defaultTime,
   onChange,
   ...props
 }) => ({

src/Toggle.js

@@ -3,6 +3,7 @@ import createComponent from './createComponent'
 
 export default createComponent(Toggle, ({
   input: { onChange, value, ...inputProps },
+  defaultToggled,
   meta,
   ...props
 }) => ({

src/__tests__/Checkbox.spec.js

@@ -54,6 +54,24 @@ describe('Checkbox', () => {
     )
   })
 
+  it('should ignore defaultChecked', () => {
+    expect(
+      new ReduxFormMaterialUICheckbox({
+        input: {
+          name: 'myCheckbox',
+          onChange: noop
+        },
+        defaultChecked: true
+      }).render()
+    ).toEqualJSX(
+      <Checkbox
+        name="myCheckbox"
+        onCheck={noop}
+        ref="component"
+      />
+    )
+  })
+
   it('provides getRenderedComponent', () => {
     const dom = TestUtils.renderIntoDocument(
       <MuiThemeProvider muiTheme={getMuiTheme()}>

src/__tests__/DatePicker.spec.js

@@ -98,6 +98,29 @@ describe('DatePicker', () => {
     )
   })
 
+  it('should ignore defaultDate', () => {
+    const defaultDate = new Date('2016-01-01')
+    expect(
+      new ReduxFormMaterialUIDatePicker({
+        input: {
+          name: 'myDatePicker'
+        },
+        meta: {
+          warning: 'FooWarning',
+          touched: true
+        },
+        defaultDate
+      }).render()
+    ).toEqualJSX(
+      <DatePicker
+        name="myDatePicker"
+        errorText="FooWarning"
+        onChange={noop}
+        ref="component"
+      />
+    )
+  })
+
   it('maps onChange properly', () => {
     const onChange = createSpy()
     const first = new Date('2016-01-01')

src/__tests__/Slider.spec.js

@@ -31,6 +31,21 @@ describe('Slider', () => {
     )
   })
 
+  it('should ignore defaultValue', () => {
+    expect(
+      new ReduxFormMaterialUISlider({
+        input: {
+          name: 'mySlider',
+          value: 0.5,
+          onDragStart: noop
+        },
+        defaultValue: 0.8
+      }).render()
+    ).toEqualJSX(
+      <Slider name="mySlider" value={0.5} onChange={noop} ref="component" />
+    )
+  })
+
   it('renders a Slider with no error when not touched', () => {
     expect(
       new ReduxFormMaterialUISlider({

src/__tests__/TimePicker.spec.js

@@ -98,6 +98,29 @@ describe('TimePicker', () => {
     )
   })
 
+  it('should ignore defaultTime', () => {
+    const defaultTime = new Date('2016-01-01')
+    expect(
+      new ReduxFormMaterialUITimePicker({
+        input: {
+          name: 'myTimePicker'
+        },
+        meta: {
+          warning: 'FooWarning',
+          touched: true
+        },
+        defaultTime
+      }).render()
+    ).toEqualJSX(
+      <TimePicker
+        name="myTimePicker"
+        errorText="FooWarning"
+        onChange={noop}
+        ref="component"
+      />
+    )
+  })
+
   it('maps onChange properly', () => {
     const onChange = createSpy()
     const first = new Date('August 22, 2016 10:15:00')