@processmaker/screen-builder 3.8.16 → 3.8.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@processmaker/screen-builder",
-  "version": "3.8.16",
+  "version": "3.8.18",
   "scripts": {
     "dev": "VITE_COVERAGE=true vite",
     "build": "vite build",

package/src/components/accordions.js

@@ -70,26 +70,6 @@ export default [
       "defaultValue",
       "showForDesktop",
       {name: "customFormatter", showFor: "FormInput"},
-      {name: "ariaLabel", showFor: "FormInput"},
-      {name: "ariaLabel", showFor: "FormSelectList"},
-      {name: "ariaLabel", showFor: "FormDatePicker"},
-      {name: "ariaLabel", showFor: "FormCheckbox"},
-      {name: "ariaLabel", showFor: "FormDatePicker"},
-      {name: "ariaLabel", showFor: "FileUpload"},
-      {name: "ariaLabel", showFor: "FileDownload"},
-      {name: "ariaLabel", showFor: "FormSelectList"},
-      {name: "ariaLabel", showFor: "FormButton"},
-      {name: "ariaLabel", showFor: "FormTextArea"},
-      {name: "tabindex", showFor: "FormInput"},
-      {name: "tabindex", showFor: "FormSelectList"},
-      {name: "tabindex", showFor: "FormDatePicker"},
-      {name: "tabindex", showFor: "FormCheckbox"},
-      {name: "tabindex", showFor: "FormDatePicker"},
-      {name: "tabindex", showFor: "FileUpload"},
-      {name: "tabindex", showFor: "FileDownload"},
-      {name: "tabindex", showFor: "FormSelectList"},
-      {name: "tabindex", showFor: "FormButton"},
-      {name: "tabindex", showFor: "FormTextArea"},
       {name: "encryptedConfig", showFor: "FormInput"},
       {name: "variablesToSubmit", showFor: "FormButton"},
     ],

package/src/components/inspector/variables-to-submit.vue

@@ -43,15 +43,16 @@
         
         <!-- Select All and Search Section -->
         <div class="controls-section">
-          <button
-            type="button"
-            class="select-all-button"
-            @click="selectAll"
-            :disabled="filteredVariables.length === 0 || selectedVariables.length === filteredVariables.length"
+          <b-form-checkbox
+            :checked="allSelected"
+            :indeterminate="someSelected"
+            @change="toggleSelectAll"
+            :disabled="filteredVariables.length === 0"
+            class="select-all-checkbox"
             data-cy="variables-to-submit-select-all"
           >
             {{ $t('Select All') }}
-          </button>
+          </b-form-checkbox>
           <button
             type="button"
             class="search-button"
@@ -164,10 +165,13 @@ export default {
       
       // Extract calculated variables (computed properties)
       Object.assign(variables, this.extractCalculatedVariables());
+
+      // Extract watcher output variables
+      Object.assign(variables, this.extractWatcherVariables());
       
-      // Filter: exclude _parent variables, include all others
+      // Filter: exclude _parent variables and invalid variable names
       return Object.keys(variables)
-        .filter(variable => !variable.startsWith('_parent.'))
+        .filter(variable => this.isValidVariableName(variable))
         .sort();
     },
     
@@ -211,6 +215,22 @@ export default {
       return this.requiredVariables.filter(
         variable => !this.selectedVariables.includes(variable)
       );
+    },
+    
+    /**
+     * Check if all filtered variables are selected
+     */
+    allSelected() {
+      return this.filteredVariables.length > 0 && 
+        this.filteredVariables.every(v => this.selectedVariables.includes(v));
+    },
+    
+    /**
+     * Check if some (but not all) filtered variables are selected
+     */
+    someSelected() {
+      const selectedCount = this.filteredVariables.filter(v => this.selectedVariables.includes(v)).length;
+      return selectedCount > 0 && selectedCount < this.filteredVariables.length;
     }
   },
   watch: {
@@ -231,8 +251,13 @@ export default {
         this.$emit('change', []);
       }
     },
-    isEnabled(newValue) {
-      if (!newValue) {
+    isEnabled(newValue, oldValue) {
+      if (newValue && !oldValue) {
+        // When enabled for the first time, select all variables
+        this.selectedVariables = [...this.availableVariables];
+        this.$emit('input', this.selectedVariables);
+        this.$emit('change', this.selectedVariables);
+      } else if (!newValue) {
         // When disabled, clear selection to submit all variables
         this.selectedVariables = [];
         this.$emit('input', []);
@@ -250,6 +275,11 @@ export default {
     '$root.computed'() {
       // Force recomputation when computed properties change
       this.$forceUpdate();
+    },
+    // Watch for watchers changes in App.vue
+    '$root.watchers'() {
+      // Force recomputation when watchers change
+      this.$forceUpdate();
     }
   },
   methods: {
@@ -289,6 +319,14 @@ export default {
       this.selectedVariables = this.selectedVariables.filter(v => !filteredSet.has(v));
     },
     
+    toggleSelectAll(checked) {
+      if (checked) {
+        this.selectAll();
+      } else {
+        this.deselectAll();
+      }
+    },
+    
     toggleSearch() {
       this.showSearch = !this.showSearch;
       if (!this.showSearch) {
@@ -296,6 +334,23 @@ export default {
       }
     },
     
+    /**
+     * Check if a variable name is valid
+     * Uses same logic as dot_notation validation rule
+     */
+    isValidVariableName(name) {
+      if (!name || typeof name !== 'string') {
+        return false;
+      }
+      if (name.startsWith('_parent.')) {
+        return false;
+      }
+      // Same regex as dot_notation: starts with letter, followed by letters, numbers, or underscores
+      const validPartRegex = /^[a-zA-Z][a-zA-Z0-9_]*$/;
+      const parts = name.split('.');
+      return parts.every(part => validPartRegex.test(part));
+    },
+    
     /**
      * Extract calculated variables (computed properties) from the screen
      * Searches in multiple locations: App.vue, builder, or parent components
@@ -348,6 +403,47 @@ export default {
       
       return [];
     },
+
+    /**
+     * Extract watcher output variables from the screen
+     */
+    extractWatcherVariables() {
+      const watcherVars = {};
+      const watchers = this.getWatchers() || [];
+      
+      watchers.forEach(watcher => {
+        if (watcher.byPass) return;
+        
+        // Output variable (for scripts)
+        if (watcher.output_variable) {
+          watcherVars[watcher.output_variable] = null;
+        }
+        
+        // Data mapping variables (for data sources)
+        try {
+          const config = typeof watcher.script_configuration === 'string'
+            ? JSON.parse(watcher.script_configuration)
+            : watcher.script_configuration;
+          (config?.dataMapping || []).forEach(m => {
+            if (m.key) watcherVars[m.key] = null;
+          });
+        } catch {
+          console.error('Invalid JSON in script_configuration for watcher:', watcher.name);
+        }
+      });
+      
+      return watcherVars;
+    },
+    
+    /**
+     * Get watchers from various sources
+     */
+    getWatchers() {
+      return this.$root?.$data?.watchers 
+        || this.$root?.$children?.[0]?.watchers 
+        || this.$root?.$children?.[0]?.$data?.watchers 
+        || [];
+    },
     
     /**
      * Extract variables from form config
@@ -392,6 +488,23 @@ export default {
      * Single Responsibility: Only handles variable name extraction
      */
     extractVariableFromItem(item, variables) {
+      // Components that don't have submittable variables
+      const displayOnlyComponents = [
+        'FormNestedScreen',
+        'FormHtmlViewer',
+        'FormMultiColumn',
+        'FormCollectionRecordControl',
+        'FormCollectionViewControl',
+        'FormAvatar',
+        'FormListTable',
+        'FormAnalyticsChart',
+        'CaseProgressBar',
+        'FileDownload',
+      ];
+      // Skip display-only and container components that don't have submittable variables
+      if (displayOnlyComponents.includes(item.component)) {
+        return;
+      }
       const variableName = item.config?.name;
       if (variableName && !variableName.startsWith('_parent.')) {
         variables[variableName] = null;
@@ -617,25 +730,16 @@ export default {
   margin-top: 0;
 }
 
-.select-all-button {
-  background: none;
-  border: none;
-  padding: 0;
-  color: #0d6efd;
-  font-size: 15px;
+.select-all-checkbox {
+  margin: 0;
+  font-size: 14px;
   font-weight: 600;
-  cursor: pointer;
-  text-decoration: none;
-}
-
-.select-all-button:hover:not(:disabled) {
-  color: #0a58ca;
-  text-decoration: none;
+  color: #495057;
 }
 
-.select-all-button:disabled {
-  color: #adb5bd;
-  cursor: not-allowed;
+.select-all-checkbox >>> .custom-control-label {
+  cursor: pointer;
+  user-select: none;
 }
 
 .search-button {

package/src/components/task.vue

@@ -692,7 +692,7 @@ export default {
         return null;
       }
     },
-
+    
     /**
      * Handles redirection upon process completion, considering destination type and user task validation.
      * @async
@@ -831,20 +831,7 @@ export default {
      * @param {Object} data - The event data received from the socket listener.
      */
     handleProcessUpdate(data) {
-      const { event, elementDestination, tokenId } = data;
-
-      // If the activity is completed and there is an element destination, set the element destination to the task
-      if (
-        event === "ACTIVITY_COMPLETED" &&
-        this.task.id === tokenId &&
-        elementDestination
-      ) {
-        this.task.elementDestination = elementDestination;
-        // update allow_interstitial based on the element destination change after the submit
-        this.task.allow_interstitial = elementDestination.type === "displayNextAssignedTask";
-      }
-
-      if (event === 'ACTIVITY_EXCEPTION') {
+      if (data.event === 'ACTIVITY_EXCEPTION') {
         this.$emit('error', this.requestId);
         window.location.href = `/requests/${this.requestId}`;
       }