@5minds/node-red-contrib-processcube 1.6.0-process-instance-delete-hotfix-babe1e-m4ve9aj6 → 1.6.0-process-instance-delete-hotfix-5dd524-m4vf68wc

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@5minds/node-red-contrib-processcube",
-    "version": "1.6.0-process-instance-delete-hotfix-babe1e-m4ve9aj6",
+    "version": "1.6.0-process-instance-delete-hotfix-5dd524-m4vf68wc",
     "license": "MIT",
     "description": "Node-RED nodes for ProcessCube",
     "scripts": {

package/processinstance-delete.html

@@ -7,7 +7,7 @@
             engine: { value: '', type: 'processcube-engine-config' },
             modelid: { value: '' },
             duration: { value: '', type: 'number' },
-            time_unit: { value: 'hours' },
+            time_type: { value: '' },
             batch_size: { value: '100', type: 'number' },
         },
         inputs: 1,
@@ -37,8 +37,8 @@
         <input type="text" id="node-input-duration" />
     </div>
     <div class="form-row">
-        <label for="node-input-time_unit"><i class="fa fa-sliders"></i> Time Unit</label>
-        <select id="node-input-time_unit" style="width: 70%;">
+        <label for="node-input-time_type"><i class="fa fa-sliders"></i> Time Unit</label>
+        <select id="node-input-time_type" style="width: 70%;">
             <option value="hours">Hours</option>
             <option value="days">Days</option>
         </select>
@@ -55,7 +55,7 @@ Delete old instances of a process model in the ProcessCube.
 ## Inputs
 
 : payload.duration (number): The number of given time periods.
-: payload.time_unit ('hours' | 'days'): The type of time period to use.
+: payload.time_type ('hours' | 'days'): The type of time period to use.
 : payload.batch_size (number): The number of instances to be deleted simultaneously. (default 100)
 
 ## Outputs

package/processinstance-delete.js

@@ -18,14 +18,26 @@ module.exports = function (RED) {
                 return;
             }
 
-            const isHours = msg.payload.time_unit
-                ? msg.payload.time_unit.toLowerCase() === 'hours'
-                : config.time_unit.toLowerCase() === 'hours';
-            const multiplier = isHours ? 1 : 24;
-            node.log(`Multiplikator: ${multiplier}`);
+            // Gültige Werte für time_type
+            const validTimeTypes = ['days', 'hours'];
+            const timeType = msg.payload.time_type 
+                ? msg.payload.time_type.toLowerCase() 
+                : config.time_type?.toLowerCase();
+
+            // time_type validieren
+            if (!timeType || !validTimeTypes.includes(timeType)) {
+                node.error(`Invalid time_type provided: ${timeType}. Allowed values are 'days' or 'hours'.`);
+                return;
+            }
+            
+            // Zeitmultiplikator berechnen
+            const multiplier = timeType === 'hours' ? 1 : 24;
+            node.log(`Time type: ${timeType}, multiplier: ${multiplier}`);
+            
             const deletionDate = new Date(Date.now() - timeToUse * multiplier * 60 * 60 * 1000);
             node.log(`Errechnetes Datum: ${deletionDate}`);
             const modelId = msg.payload.processModelId?.trim() || config.modelid?.trim();
+            
             if (!modelId) {
                 node.error('processModelId is not defined or empty.');
                 return;