From 920de44e404bfcb595ccdea678ee3c4645f088cd Mon Sep 17 00:00:00 2001 From: Paulo Fernandes Date: Thu, 6 Jul 2017 19:52:23 -0300 Subject: [PATCH] fixes #459 --- src/RadioGroup.js | 5 +---- src/__tests__/RadioGroup-test.js | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/RadioGroup.js b/src/RadioGroup.js index 0c72361..6a0992f 100644 --- a/src/RadioGroup.js +++ b/src/RadioGroup.js @@ -6,13 +6,10 @@ const RadioGroup = (props) => { const { name, value, children, container, childContainer, onChange, ...otherProps } = props; - const hasOnChange = typeof onChange === 'function'; - const checked = hasOnChange ? 'checked' : 'defaultChecked'; - return React.createElement(container, otherProps, React.Children.map(children, child => { const clonedChild = React.cloneElement(child, { - [checked]: child.props.value === value, + checked: child.props.value === value, name, onChange, ...otherProps diff --git a/src/__tests__/RadioGroup-test.js b/src/__tests__/RadioGroup-test.js index 205ca3d..06c5c6a 100644 --- a/src/__tests__/RadioGroup-test.js +++ b/src/__tests__/RadioGroup-test.js @@ -62,7 +62,7 @@ describe('RadioGroup', () => { for (let i = 0; i < output.props.children.length; i++) { const radio = output.props.children[i]; - expect(radio.props.defaultChecked).toBe(value === radio.props.value); + expect(radio.props.checked).toBe(value === radio.props.value); } });