cleanups

bpmn-io · Nov 23, 2023 · 3017a1a · 3017a1a
1 parent e00d49d
commit 3017a1a
Show file tree

Hide file tree

Showing 7 changed files with 12 additions and 11 deletions.
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
@@ -1,6 +1,6 @@
 import { useContext, useRef } from 'preact/hooks';
 import useOptionsAsync, { LOAD_STATES } from '../../hooks/useOptionsAsync';
-import useRestrictedMultiSelectValues from '../../hooks/useRestrictedMultiSelectValues';
+import useCleanupMultiSelectValues from '../../hooks/useCleanupMultiSelectValues';
 import classNames from 'classnames';
 import { FormContext } from '../../context';
 
@@ -81,7 +81,7 @@ export default function Checklist(props) {
     options
   } = useOptionsAsync(field);
 
-  useRestrictedMultiSelectValues({
+  useCleanupMultiSelectValues({
     field,
     loadState,
     options,

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
@@ -1,6 +1,6 @@
 import { useContext, useRef } from 'preact/hooks';
 import useOptionsAsync, { LOAD_STATES } from '../../hooks/useOptionsAsync';
-import useRestrictedSingleSelectValue from '../../hooks/useRestrictedSingleSelectValue';
+import useCleanupSingleSelectValue from '../../hooks/useCleanupSingleSelectValue';
 import classNames from 'classnames';
 import { FormContext } from '../../context';
 
@@ -70,7 +70,7 @@ export default function Radio(props) {
     options
   } = useOptionsAsync(field);
 
-  useRestrictedSingleSelectValue({
+  useCleanupSingleSelectValue({
     field,
     loadState,
     options,

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
@@ -2,7 +2,7 @@ import { useContext, useMemo, useRef, useState } from 'preact/hooks';
 
 import { useService } from '../../hooks';
 import useOptionsAsync, { LOAD_STATES } from '../../hooks/useOptionsAsync';
-import useRestrictedMultiSelectValues from '../../hooks/useRestrictedMultiSelectValues';
+import useCleanupMultiSelectValues from '../../hooks/useCleanupMultiSelectValues';
 
 import { FormContext } from '../../context';
 import classNames from 'classnames';
@@ -60,7 +60,7 @@ export default function Taglist(props) {
     options
   } = useOptionsAsync(field);
 
-  useRestrictedMultiSelectValues({
+  useCleanupMultiSelectValues({
     field,
     loadState,
     options,

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
@@ -1,7 +1,7 @@
 import { useCallback, useContext, useEffect, useMemo, useRef, useState } from 'preact/hooks';
 import useOptionsAsync, { LOAD_STATES } from '../../../hooks/useOptionsAsync';
 import { useService } from '../../../hooks';
-import useRestrictedSingleSelectValue from '../../../hooks/useRestrictedSingleSelectValue';
+import useCleanupSingleSelectValue from '../../../hooks/useCleanupSingleSelectValue';
 
 import { FormContext } from '../../../context';
 
@@ -42,7 +42,7 @@ export default function SearchableSelect(props) {
     options
   } = useOptionsAsync(field);
 
-  useRestrictedSingleSelectValue({
+  useCleanupSingleSelectValue({
     field,
     loadState,
     options,

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
@@ -1,6 +1,6 @@
 import { useCallback, useContext, useMemo, useRef, useState } from 'preact/hooks';
 import useOptionsAsync, { LOAD_STATES } from '../../../hooks/useOptionsAsync';
-import useRestrictedSingleSelectValue from '../../../hooks/useRestrictedSingleSelectValue';
+import useCleanupSingleSelectValue from '../../../hooks/useCleanupSingleSelectValue';
 
 import { FormContext } from '../../../context';
 
@@ -39,7 +39,7 @@ export default function SimpleSelect(props) {
     options
   } = useOptionsAsync(field);
 
-  useRestrictedSingleSelectValue({
+  useCleanupSingleSelectValue({
     field,
     loadState,
     options,

diff --git a/...r/hooks/useRestrictedMultiSelectValues.js → ...nder/hooks/useCleanupMultiSelectValues.js b/...r/hooks/useRestrictedMultiSelectValues.js → ...nder/hooks/useCleanupMultiSelectValues.js
@@ -11,7 +11,7 @@ export default function(props) {
     values
   } = props;
 
-  // Ensures that the values are always a subset of the options
+  // Ensures that the values are always a subset of the possible options
   useEffect(() => {
 
     if (loadState !== LOAD_STATES.LOADED) {

diff --git a/...r/hooks/useRestrictedSingleSelectValue.js → ...nder/hooks/useCleanupSingleSelectValue.js b/...r/hooks/useRestrictedSingleSelectValue.js → ...nder/hooks/useCleanupSingleSelectValue.js
@@ -11,6 +11,7 @@ export default function(props) {
     value
   } = props;
 
+  // Ensures that the value is always one of the possible options
   useEffect(() => {
 
     if (loadState !== LOAD_STATES.LOADED) {