Skip to content
This repository was archived by the owner on Jan 14, 2021. It is now read-only.

Commit aaacfc7

Browse files
committed
fix event wrapper and type number
1 parent 45eecf0 commit aaacfc7

File tree

2 files changed

+17
-5
lines changed

2 files changed

+17
-5
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "react-hook-form-input",
3-
"version": "1.1.1",
3+
"version": "1.1.2-beta.1",
44
"description": "Wrapper component for controlled inputs",
55
"main": "dist/index.js",
66
"module": "dist/index.es.js",

src/index.tsx

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,15 @@ const RHFInput = React.memo(
5151
return data;
5252
};
5353

54-
const eventWrapper = (event: EventFunction) => {
54+
const eventWrapper = (event: EventFunction, eventName: string) => {
5555
return (...arg: any) => {
56-
commonTask(event(arg));
56+
const data = commonTask(event(arg));
57+
setValue(
58+
name,
59+
data,
60+
(isOnChange && eventName === 'onChange') ||
61+
(isOnBlur && eventName === 'onBlur'),
62+
);
5763
};
5864
};
5965

@@ -78,6 +84,7 @@ const RHFInput = React.memo(
7884
Object.defineProperty(
7985
{
8086
name,
87+
type,
8188
},
8289
'value',
8390
{
@@ -102,11 +109,16 @@ const RHFInput = React.memo(
102109

103110
return React.cloneElement(as, {
104111
...(onChangeEvent
105-
? { [onChangeName || 'onChange']: eventWrapper(onChangeEvent) }
112+
? {
113+
[onChangeName || 'onChange']: eventWrapper(
114+
onChangeEvent,
115+
'onChange',
116+
),
117+
}
106118
: { onChange: handleChange }),
107119
...(isOnBlur
108120
? onBlurEvent
109-
? { [onBlurName || 'onBlur']: eventWrapper(onBlurEvent) }
121+
? { [onBlurName || 'onBlur']: eventWrapper(onBlurEvent, 'onBlur') }
110122
: { onBlur: handleBlur }
111123
: {}),
112124
value: value || inputValue,

0 commit comments

Comments
 (0)