From 6b5ef7be2f532278a388bf3da96a68bba1359a35 Mon Sep 17 00:00:00 2001 From: David Bonnet Date: Mon, 3 Jun 2019 11:38:54 +0200 Subject: [PATCH] fix(values): transformable adds onChange if set The `transformable` decorator injected `onChange` even if `onChange` was not provided from the parent. --- src/values.js | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/src/values.js b/src/values.js index 3b010956..f806f680 100644 --- a/src/values.js +++ b/src/values.js @@ -80,13 +80,14 @@ export const transformable = (Component) => : { ...props, value: transformValue ? state.transformedValue : props.value, - onChange: transformOnChange - ? lazyProperty( - this, - 'transformedOnChange', - transformedOnChange, - ) - : props.onChange, + onChange: + transformOnChange && props.onChange + ? lazyProperty( + this, + 'transformedOnChange', + transformedOnChange, + ) + : props.onChange, }, ) }