@processmaker/screen-builder 2.28.1-RC4B → 2.30.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@processmaker/screen-builder",
-  "version": "2.28.1-RC4B",
+  "version": "2.30.0",
   "scripts": {
     "serve": "vue-cli-service serve",
     "build": "vue-cli-service build",
@@ -22,7 +22,7 @@
   ],
   "dependencies": {
     "is-proxy": "^1.0.6",
-    "lodash": "^4.17.20",
+    "lodash": "^4.17.21",
     "moment": "^2.29.1",
     "moment-timezone": "^0.5.27",
     "scrollparent": "^2.0.1",
@@ -38,7 +38,7 @@
     "@cypress/code-coverage": "^3.8.1",
     "@fortawesome/fontawesome-free": "^5.6.1",
     "@panter/vue-i18next": "^0.15.2",
-    "@processmaker/vue-form-elements": "0.29.0",
+    "@processmaker/vue-form-elements": "0.29.1",
     "@processmaker/vue-multiselect": "^2.2.0",
     "@vue/cli-plugin-babel": "^3.6.0",
     "@vue/cli-plugin-e2e-cypress": "^4.0.3",
@@ -82,7 +82,7 @@
   },
   "peerDependencies": {
     "@panter/vue-i18next": "^0.15.0",
-    "@processmaker/vue-form-elements": "0.29.0",
+    "@processmaker/vue-form-elements": "0.29.1",
     "i18next": "^15.0.8",
     "vue": "^2.6.12",
     "vuex": "^3.1.1"

package/src/ValidationsFactory.js

@@ -35,7 +35,7 @@ class Validations {
   isVisible() {
     // Disable validations if field is hidden
     let visible = true;
-    if (!this.data.noData && this.element.config.conditionalHide) {
+    if (this.element.config.conditionalHide) {
       try {
         visible = !!Parser.evaluate(this.element.config.conditionalHide, this.data);
       } catch (error) {
@@ -43,7 +43,7 @@ class Validations {
       }
     }
     return visible;
-  }    
+  }
 }
 
 /**
@@ -115,7 +115,8 @@ class FormLoopValidations extends Validations {
     const loopField = get(validations, this.element.config.name);
     loopField['$each'] = {};
     this.checkForSiblings(validations);
-    await ValidationsFactory(this.element.items, { screen: this.screen, data: {_parent: this.data, noData:true }, parentVisibilityRule: this.element.config.conditionalHide, insideLoop: true }).addValidations(loopField['$each']);
+    const firstRow = (get(this.data, this.element.config.name) || [{}])[0];
+    await ValidationsFactory(this.element.items, { screen: this.screen, data: {_parent: this.data, ...firstRow }, parentVisibilityRule: this.element.config.conditionalHide, insideLoop: true }).addValidations(loopField['$each']);
   }
   checkForSiblings(validations) {
     const siblings = [];

package/src/components/inspector/form-multiselect.vue

@@ -45,7 +45,7 @@ export default {
   computed: {
     classList() {
       return {
-        'is-invalid': (this.validator && this.validator.errorCount) || this.error,
+        'has-errors': (this.validator && this.validator.errorCount) || this.error,
       };
     },
   },
@@ -59,7 +59,7 @@ export default {
 </script>
 
 <style lang="scss">
-  .is-invalid .multiselect__tags {
-    border-color: red !important;
+  .has-errors .multiselect__tags {
+    border-color: red;
   }
-</style>
+</style>

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

@@ -1,5 +1,5 @@
 <template>
-  <input type="text" class="form-control" v-model="currencyValue" ref="currencyInput" >
+  <input type="text" class="form-control" @keyup="keyup" v-model="currencyValue" ref="currencyInput" >
 </template>
 
 <script>
@@ -66,6 +66,13 @@ export default {
         this.currencyInput.inputmask.setValue(this.value);
       }
     },
+    keyup(event) {
+      // Workaround for a bug in inputmask where backspacing after
+      // the decimal does not trigger an input event
+      if (this.currencyValue !== event.target.value) {
+        this.currencyValue = event.target.value;
+      }
+    },
   },
   mounted() {
     this.loadMasks();

package/src/mixins/ScreenBase.js

@@ -103,15 +103,7 @@ export default {
     },
     mustache(text) {
       try {
-        const data = new Proxy(this, {
-          get(target, name) {
-            if (name === '_parent') {
-              return target._parent;
-            } else {
-              return target.vdata[name];
-            }
-          },
-        });
+        const data = Object.assign({_parent: this._parent}, this.vdata);
         return text && Mustache.render(text, data);
       } catch (e) {
         return 'MUSTACHE: ' + e.message;
@@ -187,7 +179,7 @@ export default {
           this.$set(
             object,
             attr,
-            setValue
+            setValue,
           );
 
           object = get(object, attr);

package/src/mixins/VisibilityRule.js

@@ -3,7 +3,7 @@ import { debounce } from 'lodash';
 
 export default {
   mounted() {
-    this.refreshValidationRulesByName = debounce(this.refreshValidationRulesByName, 1000);
+    this.refreshValidationRulesByName = debounce(this.refreshValidationRulesByName, 500);
 
     this.$root.$on('refresh-validation-rules', () => {
       this.loadValidationRules();
@@ -30,7 +30,7 @@ export default {
 
         window.setTimeout(() => {
           this.refreshValidationRulesByName(fieldName, isVisible);
-        }, 1000);
+        }, 250);
         return isVisible;
       } catch (e) {
         return false;