@processmaker/screen-builder 3.8.15-patch.a.1 → 3.8.15-patch.c

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@processmaker/screen-builder",
-  "version": "3.8.15-patch.a.1",
+  "version": "3.8.15-patch.c",
   "scripts": {
     "dev": "VITE_COVERAGE=true vite",
     "build": "vite build",
@@ -12,9 +12,6 @@
     "storybook": "storybook dev -p 6006",
     "build-storybook": "storybook build",
     "test-storybook": "test-storybook",
-    "test": "jest tests/unit --no-coverage",
-    "test:coverage": "jest tests/unit --coverage",
-    "test:variables": "jest tests/unit/VariablesToSubmit.spec.js --no-coverage",
     "serve": "npm run dev"
   },
   "main": "./dist/vue-form-builder.umd.js",
@@ -75,7 +72,6 @@
     "@vue/test-utils": "^1.1.1",
     "axios": "^1.6.0",
     "babel-core": "7.0.0-bridge.0",
-    "babel-jest": "^26.6.3",
     "babel-plugin-istanbul": "^6.1.1",
     "bootstrap": "^4.5.3",
     "bootstrap-vue": "^2.19.0",
@@ -97,9 +93,6 @@
     "i18next": "^15.0.8",
     "identity-obj-proxy": "^3.0.0",
     "inputmask": "^5.0.7",
-    "jest-environment-jsdom": "^30.2.0",
-    "jest-serializer-vue": "^2.0.2",
-    "jest-transform-stub": "^2.0.0",
     "mustache": "^3.0.1",
     "nyc": "^15.1.0",
     "prettier": "2.7.1",
@@ -114,7 +107,6 @@
     "vite-plugin-monaco-editor": "^1.1.0",
     "vue": "^2.6.12",
     "vue-deepset": "^0.6.3",
-    "vue-jest": "^3.0.7",
     "vue-template-compiler": "^2.6.14",
     "vue-uniq-ids": "^1.0.0",
     "vue-upload-component": "^2.8.14",

package/src/components/accordions.js

@@ -86,7 +86,6 @@ export default [
       {name: 'tabindex', showFor: 'FormButton'},
       {name: 'tabindex', showFor: 'FormTextArea'},
       {name: 'encryptedConfig', showFor: 'FormInput'},
-      {name: 'variablesToSubmit', showFor: 'FormButton'},
     ],
     open: false,
   },

package/src/components/inspector/index.js

@@ -32,4 +32,3 @@ export { default as LabelSubmitButton } from "./label-submit-button.vue";
 export { default as AnalyticsSelector } from "./analytics-selector.vue";
 export { default as EncryptedConfig } from "./encrypted-config.vue";
 export { default as CodeEditor } from "./code-editor.vue";
-export { default as VariablesToSubmit } from "./variables-to-submit.vue";

package/src/components/renderer/form-button.vue

@@ -41,8 +41,7 @@ export default {
     "loading",
     "loadingLabel",
     "handler",
-    "handlerSecurityEnabled",
-    "config"
+    "handlerSecurityEnabled"
   ],
   data() {
     return {
@@ -87,8 +86,7 @@ export default {
       return {
         name: this.name,
         label: this.label,
-        value: this.fieldValue,
-        variablesToSubmit: this.config?.variablesToSubmit
+        value: this.fieldValue
       };
     }
   },

package/src/components/task.vue

@@ -586,9 +586,7 @@ export default {
       }
       this.disabled = true;
 
-      // Ensure formData is always a valid object (never null, undefined, or false)
-      const safeFormData = (formData && typeof formData === 'object') ? formData : (this.requestData || {});
-      if (formData && typeof formData === 'object') {
+      if (formData) {
         this.onUpdate(Object.assign({}, this.requestData, formData));
       }
 
@@ -597,7 +595,7 @@ export default {
       } else {
         this.loadingButton = false;
       }
-      this.$emit('submit', this.task, safeFormData, loading, buttonInfo);
+      this.$emit('submit', this.task, loading, buttonInfo);
 
       if (this.task?.allow_interstitial && !this.loadingButton && !this.disableInterstitial) {
         this.task.interstitial_screen['_interstitial'] = true;
@@ -900,7 +898,11 @@ export default {
      * @param {Object} data - The event data containing the tokenId of the task.
      */
     async handleRedirectToTask(data) {
-      if (data?.params[0]?.tokenId) {
+      if (
+        (data?.params[0]?.tokenId &&
+        this.task.user?.id === data.params[0]?.userId) ||
+        this.task.elementDestination?.type === 'taskSource'
+      ) {
         this.loadingTask = true;
         // Check if interstitial tasks are allowed for this task.
         if (this.task && !(this.task.allow_interstitial || this.isSameUser(this.task, data))) {

package/src/components/vue-form-renderer.vue

@@ -38,13 +38,12 @@ import { getItemsFromConfig } from "../itemProcessingUtils";
 import { ValidatorFactory } from "../factories/ValidatorFactory";
 import CurrentPageProperty from "../mixins/CurrentPageProperty";
 import DeviceDetector, { MAX_MOBILE_WIDTH } from "../mixins/DeviceDetector";
-import VariablesToSubmitFilter from "../mixins/VariablesToSubmitFilter";
 import ScreenRenderer from "@/components/screen-renderer.vue";
 
 export default {
   name: "VueFormRenderer",
   components: { ScreenRenderer, CustomCssOutput },
-  mixins: [CurrentPageProperty, DeviceDetector, VariablesToSubmitFilter],
+  mixins: [CurrentPageProperty, DeviceDetector],
   model: {
     prop: "data",
     event: "update"
@@ -258,8 +257,7 @@ export default {
       this.$emit('after-submit', ...arguments);
     },
     submit(eventData, loading = false, buttonInfo = null) {
-      // eventData is already filtered by ScreenBase.submitForm, just pass it through
-      this.$emit("submit", eventData, loading, buttonInfo);
+      this.$emit("submit", this.data, loading, buttonInfo);
     },
     parseCss() {
       const containerSelector = `.${this.containerClass}`;

package/src/form-builder-controls.js

@@ -800,14 +800,6 @@ export default [
           },
         },
         buttonVariantStyleProperty,
-        {
-          type: 'VariablesToSubmit',
-          field: 'variablesToSubmit',
-          config: {
-            label: 'Variables to Submit',
-            helper: 'Select which variables should be included in the submission payload. Only variables from the parent (top-level) request can be used.',
-          },
-        },
       ],
     },
   },

package/src/mixins/Json2Vue.js

@@ -57,8 +57,7 @@ export default {
       if (buttonInfo && this.loopContext) {
         buttonInfo.loopContext = this.loopContext;
       }
-      // Use eventData (already filtered) instead of this.value (unfiltered)
-      this.$emit('submit', eventData, loading, buttonInfo);
+      this.$emit('submit', this.value, loading, buttonInfo);
     },
     buildComponent(definition) {
       if (window.ProcessMaker && window.ProcessMaker.EventBus) {

package/src/mixins/ScreenBase.js

@@ -4,7 +4,6 @@ import { mapActions, mapGetters, mapState } from 'vuex';
 import { ValidationMsg } from './ValidationRules';
 import DataReference from "./DataReference";
 import computedFields from "./computedFields";
-import VariablesToSubmitFilter from "./VariablesToSubmitFilter";
 import { findRootScreen } from "./DataReference";
 
 const stringFormats = ['string', 'datetime', 'date', 'password'];
@@ -12,7 +11,7 @@ const parentReference = [];
 
 export default {
   name: "ScreenContent",
-  mixins: [DataReference, computedFields, VariablesToSubmitFilter],
+  mixins: [DataReference, computedFields],
   schema: [
     function() {
       if (window.ProcessMaker && window.ProcessMaker.packages && window.ProcessMaker.packages.includes('package-vocabularies')) {
@@ -159,19 +158,7 @@ export default {
       };
       this.$emit('after-submit', event, ...arguments);
       if (event.validation === false) {
-        // Filter data based on variablesToSubmit configuration (LAYER 1 protection)
-        const dataToSubmit = this.filterDataForSubmission(this.vdata, buttonInfo);
-        
-        // Debug logging for variables filtering
-        if (buttonInfo?.variablesToSubmit?.length > 0) {
-          console.log('[VariablesToSubmit] Filtering enabled (validation bypassed)');
-          console.log('[VariablesToSubmit] Original data keys:', Object.keys(this.vdata || {}));
-          console.log('[VariablesToSubmit] Selected variables:', buttonInfo.variablesToSubmit);
-          console.log('[VariablesToSubmit] Filtered data keys:', Object.keys(dataToSubmit));
-          console.log('[VariablesToSubmit] Data to submit:', dataToSubmit);
-        }
-        
-        this.$emit('submit', dataToSubmit, loading, buttonInfo);
+        this.$emit('submit', this.vdata, loading, buttonInfo);
         return;
       }
       await this.validateNow(findRootScreen(this));
@@ -183,19 +170,7 @@ export default {
         // if the form is not valid the data is not emitted
         return;
       }
-      // Filter data based on variablesToSubmit configuration (LAYER 1 protection)
-      const dataToSubmit = this.filterDataForSubmission(this.vdata, buttonInfo);
-      
-      // Debug logging for variables filtering
-      if (buttonInfo?.variablesToSubmit?.length > 0) {
-        console.log('[VariablesToSubmit] Filtering enabled');
-        console.log('[VariablesToSubmit] Original data keys:', Object.keys(this.vdata || {}));
-        console.log('[VariablesToSubmit] Selected variables:', buttonInfo.variablesToSubmit);
-        console.log('[VariablesToSubmit] Filtered data keys:', Object.keys(dataToSubmit));
-        console.log('[VariablesToSubmit] Data to submit:', dataToSubmit);
-      }
-      
-      this.$emit('submit', dataToSubmit, loading, buttonInfo);
+      this.$emit('submit', this.vdata, loading, buttonInfo);;
     },
     resetValue(safeDotName, variableName) {
       this.setValue(safeDotName, null);

package/src/mixins/index.js

@@ -16,7 +16,6 @@ export { default as ScreenBase } from "./ScreenBase";
 export { default as shouldElementBeVisible } from "./shouldElementBeVisible";
 export { default as testing } from "./testing";
 export { ValidationMsg, validators } from "./ValidationRules";
-export { default as VariablesToSubmitFilter } from "./VariablesToSubmitFilter";
 export { default as VisibilityRule } from "./VisibilityRule";
 export { default as watchers } from "./watchers";
 export { default as Clipboard } from "./Clipboard";