chore: cleanup onChange usages

Related to #1147
bpmn-io · Apr 17, 2024 · a35df2b · a35df2b
1 parent fbff495
commit a35df2b
Show file tree

Hide file tree

Showing 11 changed files with 6 additions and 15 deletions.
diff --git a/packages/form-js-playground/test/custom/viewer.js b/packages/form-js-playground/test/custom/viewer.js
@@ -21,7 +21,6 @@ function RangeRenderer(props) {
 
   const onChange = ({ target }) => {
     props.onChange({
-      field,
       value: Number(target.value),
     });
   };

diff --git a/packages/form-js-viewer/src/features/repeatRender/RepeatRenderManager.js b/packages/form-js-viewer/src/features/repeatRender/RepeatRenderManager.js
@@ -128,9 +128,7 @@ export class RepeatRenderManager {
       shouldScroll.current = true;
 
       props.onChange({
-        field: repeaterField,
         value: updatedValues,
-        indexes,
       });
 
       setSharedRepeatState((state) => ({ ...state, isCollapsed: false }));

diff --git a/packages/form-js-viewer/src/render/components/form-fields/Checkbox.js b/packages/form-js-viewer/src/render/components/form-fields/Checkbox.js
@@ -17,7 +17,6 @@ export function Checkbox(props) {
 
   const onChange = ({ target }) => {
     props.onChange({
-      field,
       value: target.checked,
     });
   };

diff --git a/packages/form-js-viewer/src/render/components/form-fields/Checklist.js b/packages/form-js-viewer/src/render/components/form-fields/Checklist.js
@@ -31,7 +31,6 @@ export function Checklist(props) {
       : [...values, toggledValue];
 
     props.onChange({
-      field,
       value: newValues,
     });
   };

diff --git a/packages/form-js-viewer/src/render/components/form-fields/Radio.js b/packages/form-js-viewer/src/render/components/form-fields/Radio.js
@@ -27,7 +27,6 @@ export function Radio(props) {
 
   const onChange = (v) => {
     props.onChange({
-      field,
       value: v,
     });
   };

diff --git a/packages/form-js-viewer/src/render/components/form-fields/Taglist.js b/packages/form-js-viewer/src/render/components/form-fields/Taglist.js
@@ -76,12 +76,12 @@ export function Taglist(props) {
       return;
     }
 
-    props.onChange({ value: [...values, value], field });
+    props.onChange({ value: [...values, value] });
   };
 
   const deselectValue = (value) => {
     const newValues = values.filter((v) => !isEqual(v, value));
-    props.onChange({ value: newValues, field });
+    props.onChange({ value: newValues });
   };
 
   const onInputChange = ({ target }) => {

diff --git a/packages/form-js-viewer/src/render/components/form-fields/Textarea.js b/packages/form-js-viewer/src/render/components/form-fields/Textarea.js
@@ -21,7 +21,6 @@ export function Textarea(props) {
 
   const [onChange, flushOnChange] = useFlushDebounce(({ target }) => {
     props.onChange({
-      field,
       value: target.value,
     });
   });

diff --git a/packages/form-js-viewer/src/render/components/form-fields/Textfield.js b/packages/form-js-viewer/src/render/components/form-fields/Textfield.js
@@ -21,7 +21,6 @@ export function Textfield(props) {
 
   const [onInputChange, flushOnChange] = useFlushDebounce(({ target }) => {
     props.onChange({
-      field,
       value: target.value,
     });
   });

diff --git a/packages/form-js-viewer/src/render/components/form-fields/parts/SearchableSelect.js b/packages/form-js-viewer/src/render/components/form-fields/parts/SearchableSelect.js
@@ -54,9 +54,9 @@ export function SearchableSelect(props) {
   const setValue = useCallback(
     (option) => {
       setFilter((option && option.label) || '');
-      props.onChange({ value: (option && option.value) || null, field });
+      props.onChange({ value: (option && option.value) || null });
     },
-    [field, props],
+    [props],
   );
 
   const displayState = useMemo(() => {

diff --git a/packages/form-js-viewer/src/render/components/form-fields/parts/SimpleSelect.js b/packages/form-js-viewer/src/render/components/form-fields/parts/SimpleSelect.js
@@ -33,9 +33,9 @@ export function SimpleSelect(props) {
 
   const setValue = useCallback(
     (option) => {
-      props.onChange({ value: (option && option.value) || null, field });
+      props.onChange({ value: (option && option.value) || null });
     },
-    [field, props],
+    [props],
   );
 
   const displayState = useMemo(() => {

diff --git a/packages/form-js-viewer/test/spec/custom/index.js b/packages/form-js-viewer/test/spec/custom/index.js
@@ -41,7 +41,6 @@ function Range(props) {
 
   const onChange = ({ target }) => {
     props.onChange({
-      field,
       value: Number(target.value),
     });
   };
-Original file line number
+Diff line change
@@ Expand Up / @@ -21,7 +21,6 @@ function RangeRenderer(props) { @@
       const onChange = ({ target }) => {
         props.onChange({
-          field,
           value: Number(target.value),
         });
       };
@@ Expand Down @@