Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Commit 21281de

Browse files
author
Shammamah Hossain
committed
Remove unnecessary wrapper function.
1 parent 24b3b9d commit 21281de

1 file changed

Lines changed: 3 additions & 8 deletions

File tree

src/components/Input.react.js

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,11 @@ export default class Input extends PureComponent {
2828
this.onKeyPress = this.onKeyPress.bind(this);
2929
this.setInputValue = this.setInputValue.bind(this);
3030
this.setPropValue = this.setPropValue.bind(this);
31-
this.getValueAsNumber = this.getValueAsNumber.bind(this);
3231
}
3332

3433
componentWillReceiveProps(nextProps) {
3534
const {value} = this.input.current;
36-
const valueAsNumber = this.getValueAsNumber(value);
35+
const valueAsNumber = convert(value);
3736
this.setInputValue(
3837
isNil(valueAsNumber) ? value : valueAsNumber,
3938
nextProps.value
@@ -45,7 +44,7 @@ export default class Input extends PureComponent {
4544

4645
componentDidMount() {
4746
const {value} = this.input.current;
48-
const valueAsNumber = this.getValueAsNumber(value);
47+
const valueAsNumber = convert(value);
4948
this.setInputValue(
5049
isNil(valueAsNumber) ? value : valueAsNumber,
5150
this.props.value
@@ -92,10 +91,6 @@ export default class Input extends PureComponent {
9291
);
9392
}
9493

95-
getValueAsNumber(value) {
96-
return convert(value);
97-
}
98-
9994
setInputValue(base, value) {
10095
const __value = value;
10196
base = this.input.current.checkValidity() ? convert(base) : NaN;
@@ -117,7 +112,7 @@ export default class Input extends PureComponent {
117112

118113
onEvent() {
119114
const {value} = this.input.current;
120-
const valueAsNumber = this.getValueAsNumber(value);
115+
const valueAsNumber = convert(value);
121116
if (this.props.type === 'number') {
122117
this.setPropValue(
123118
this.props.value,

0 commit comments

Comments
 (0)