Trim and remove spaces from ID with user alert (#94)

sartography · Jun 25, 2024 · 91aa2f5 · 91aa2f5
1 parent bf7ea0c
commit 91aa2f5
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 3 deletions.
diff --git a/app/spiffworkflow/DataObject/propertiesPanel/DataObjectArray.js b/app/spiffworkflow/DataObject/propertiesPanel/DataObjectArray.js
@@ -13,6 +13,7 @@ import {
   idToHumanReadableName,
   findDataObject,
 } from '../DataObjectHelpers';
+import { processId } from '../../helpers';
 
 /**
  * Provides a list of data objects, and allows you to add / remove data objects, and change their ids.
@@ -146,7 +147,7 @@ function DataObjectTextField(props) {
         element,
         moddleElement: dataObject,
         properties: {
-          id: value,
+          id: processId(value),
           // name: doName
         },
       });

diff --git a/app/spiffworkflow/eventList.js b/app/spiffworkflow/eventList.js
@@ -5,7 +5,7 @@ import {
   TextFieldEntry,
   isTextFieldEntryEdited
 } from '@bpmn-io/properties-panel';
-import { getRoot } from './helpers';
+import { getRoot, processId } from './helpers';
 
 /* This function creates a list of a particular event type at the process level using the item list
  * and add function provided by `getArray`.
@@ -142,7 +142,7 @@ function ItemTextField(props) {
       element,
       moddleElement: item,
       properties: {
-        id: value,
+        id: processId(value),
         name: value,
       },
     });

diff --git a/app/spiffworkflow/helpers.js b/app/spiffworkflow/helpers.js
@@ -36,3 +36,13 @@ export function getRoot(businessObject, moddle) {
   }
   return businessObject;
 }
+
+
+export function processId(id) {
+  let trimmedId = id.trim();
+  let processedId = trimmedId.replace(/\s+/g, '');
+  if (id !== processedId) {
+    alert('ID should not contain spaces. It has been adjusted.');
+  }
+  return processedId;
+}