From fab480816b90b624c64d297f7350a2a0f8d3b6a1 Mon Sep 17 00:00:00 2001
From: elsiosanchez <45974454+elsiosanchez@users.noreply.github.com>
Date: Wed, 11 Mar 2020 21:23:05 -0400
Subject: [PATCH] unnecessary requests (#383)

* unnecessary requests

* validate isDocument

* validate when new registration
---
 src/components/ADempiere/Panel/index.vue             | 7 +++----
 src/components/ADempiere/WorkflowStatusBar/index.vue | 2 +-
 src/views/ADempiere/Window/index.vue                 | 9 +++++----
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/components/ADempiere/Panel/index.vue b/src/components/ADempiere/Panel/index.vue
index 6c8d6e99..1c14b95f 100644
--- a/src/components/ADempiere/Panel/index.vue
+++ b/src/components/ADempiere/Panel/index.vue
@@ -315,7 +315,6 @@ export default {
   },
   created() {
     // get fields with uuid
-    this.$store.dispatch('listWorkflows', this.metadata.tableName)
     this.getPanel()
   },
   methods: {
@@ -615,11 +614,11 @@ export default {
       return groupsList
     },
     setTagsViewTitle(actionValue) {
-      if (!this.isEmptyValue(this.$route.params.recordId) && this.getterPanel.isDocument) {
+      if (this.getterPanel.isDocument && this.getterDataStore.isLoaded) {
+        this.$store.dispatch('listWorkflows', this.metadata.tableName)
         this.$store.dispatch('listDocumentStatus', {
           recordUuid: this.$route.query.action,
-          recordId: this.$route.params.recordId,
-          tableName: this.$route.params.tableName
+          tableName: this.metadata.tableName
         })
       }
       if (actionValue === 'create-new' || this.isEmptyValue(actionValue)) {
diff --git a/src/components/ADempiere/WorkflowStatusBar/index.vue b/src/components/ADempiere/WorkflowStatusBar/index.vue
index 10adff75..a43f7ec0 100644
--- a/src/components/ADempiere/WorkflowStatusBar/index.vue
+++ b/src/components/ADempiere/WorkflowStatusBar/index.vue
@@ -1,5 +1,5 @@
 <template>
-  <el-steps v-if="!isEmptyValue(gettersNodeList)" :active="getActive" finish-status="success" simple :style="styleSteps">
+  <el-steps :active="getActive" finish-status="success" simple :style="styleSteps">
     <el-step
       v-for="(node, index) in listDocumentStatus"
       :key="index"
diff --git a/src/views/ADempiere/Window/index.vue b/src/views/ADempiere/Window/index.vue
index 42204397..afc1726d 100644
--- a/src/views/ADempiere/Window/index.vue
+++ b/src/views/ADempiere/Window/index.vue
@@ -58,11 +58,12 @@
                   <resize-observer @notify="handleResize" />
                   <Split v-shortkey="['f8']" direction="vertical" @onDrag="onDrag" @shortkey.native="handleChangeShowedRecordNavigation(!isShowedRecordNavigation)">
                     <SplitArea :size="sizeAreaStyle" :style="splitAreaStyle">
-                      <el-header :style="gettersNodeList ? 'height: 45px; background: #F5F7FA' : 'height: 40px'">
+                      <el-header :style="isWorkflowBarStatus ? 'height: 45px; background: #F5F7FA' : 'height: 40px'">
                         <el-container>
                           <el-aside width="100%" style="width: 78vw;overflow: hidden;">
                             <el-scrollbar>
                               <workflow-status-bar
+                                v-if="isWorkflowBarStatus"
                                 :style-steps="styleStepsSimple"
                                 :container-uuid="windowMetadata.currentTabUuid"
                                 :parent-uuid="windowUuid"
@@ -528,9 +529,9 @@ export default {
       })
       return record
     },
-    gettersNodeList() {
-      var node = this.$store.getters.getNodeWorkflow
-      if (!this.isEmptyValue(node.workflowsList)) {
+    isWorkflowBarStatus() {
+      const panel = this.$store.getters.getPanel(this.windowMetadata.currentTabUuid)
+      if (!this.isEmptyValue(panel) && panel.isDocument && this.$route.meta.type === 'window' && this.$route.query.action !== 'create-new') {
         return true
       }
       return false