Action Process DUPLICATES if double Click is made (#2633)

src/components/ADempiere/ModalDialog/index.vue

@@ -55,6 +55,7 @@
         icon="el-icon-check"
         class="button-base-icon"
         :disabled="isDisabledDone"
+        :loading="isDisabledDone"
         @click="doneButton"
       />
     </span>
@@ -101,6 +102,7 @@
         icon="el-icon-check"
         class="button-base-icon"
         :disabled="isDisabledDone"
+        :loading="isDisabledDone"
         @click="doneButton"
       />
     </div>

src/components/ADempiere/TabManager/convenienceButtons/PrintProcess.vue

@@ -30,9 +30,15 @@
       @command="handleCommandActions"
     >
       <svg-icon
+        v-if="!isLoading"
         style="font-size: 23px;"
         icon-class="print"
       />
+      <i
+        v-else
+        style="font-size: 21px;"
+        class="el-icon-loading"
+      />
 
       <el-dropdown-menu slot="dropdown">
         <el-dropdown-item
@@ -50,18 +56,26 @@
       type="info"
       size="small"
       style="margin-left: 5px;padding-top: 1px;padding-right: 5px;padding-bottom: 8px;padding-left: 5px;"
+      :disabled="isLoading"
+      :loading="isLoading"
       @click="printProcess()"
     >
       <svg-icon
+        v-if="!isLoading"
         style="font-size: 21px;"
         icon-class="print"
       />
+      <i
+        v-else
+        style="font-size: 21px;"
+        class="el-icon-loading"
+      />
     </el-button>
   </span>
 </template>
 
 <script>
-import { defineComponent, computed } from '@vue/composition-api'
+import { defineComponent, computed, ref } from '@vue/composition-api'
 
 import language from '@/lang'
 import store from '@/store'
@@ -89,6 +103,10 @@ export default defineComponent({
   },
 
   setup(props) {
+    /**
+     * Ref
+     */
+    const isLoading = ref(false)
     /**
      * Const
      */
@@ -134,13 +152,16 @@ export default defineComponent({
         })
         return
       }
-
+      isLoading.value = true
       store.dispatch('runReport', {
         containerUuid: process.uuid,
         recordId: recordId.value,
         reportId: process.internal_id,
         tableName: table_name
       })
+        .finally(() => {
+          isLoading.value = false
+        })
     }
 
     function handleCommandActions(command) {
@@ -178,6 +199,8 @@ export default defineComponent({
     loadProcessData()
 
     return {
+      // Ref
+      isLoading,
       // Const
       process,
       is_document,

src/store/modules/ADempiere/dictionary/browser/actions.js

@@ -129,6 +129,11 @@ export default {
           resolve(browserDefinition)
           const { process } = browserDefinition
           if (!isEmptyValue(process)) {
+            store.commit('setIsloadingProcessOfBrowser', {
+              isLoading: false,
+              parentUuid: process.id,
+              containerUuid: browserDefinition.containerUuid
+            })
             dispatch('setModalDialog', {
               containerUuid: process.uuid,
               title: process.name,
@@ -148,6 +153,12 @@ export default {
 
                 const isAllSelection = rootGetters.getStoredBrowserProcessAll(browserDefinition.uuid)
 
+                store.commit('setIsloadingProcessOfBrowser', {
+                  isLoading: true,
+                  parentUuid: process.id,
+                  containerUuid: browserDefinition.containerUuid
+                })
+
                 store.dispatch('startProcessOfBrowser', {
                   parentUuid: browserDefinition.uuid,
                   containerUuid: process.uuid,
@@ -164,6 +175,12 @@ export default {
                       path: oldRouter.path
                     }, () => {})
                   }
+                }).finally(() => {
+                  store.commit('setIsloadingProcessOfBrowser', {
+                    isLoading: false,
+                    parentUuid: process.id,
+                    containerUuid: browserDefinition.containerUuid
+                  })
                 })
               },
               beforeOpen: ({ parentUuid: browserUuid, containerUuid }) => {
@@ -187,6 +204,13 @@ export default {
                   containerUuidAssociated: browserDefinition.uuid
                 })
               },
+              isDisabledDone() {
+                // validate document status and Processing flag
+                return store.getters.getIsloadingProcessOfBrowser({
+                  parentUuid: process.id,
+                  containerUuid: browserDefinition.containerUuid
+                })
+              },
               ...process,
               // TODO: Change to string and import dynamic in component
               componentPath: () => import('@/components/ADempiere/PanelDefinition/index.vue'),

src/store/modules/ADempiere/processManager.js

@@ -18,7 +18,7 @@
 
 import lang from '@/lang'
 import router from '@/router'
-
+import Vue from 'vue'
 // API Request Methods
 import {
   requestRunBusinessProcess,
@@ -39,9 +39,20 @@ import {
 } from '@/utils/ADempiere/dictionary/process.js'
 
 const processManager = {
-  state: {},
+  state: {
+    isLoadingProcessOfBrowser: {}
+  },
 
-  mutations: {},
+  mutations: {
+    setIsloadingProcessOfBrowser(state, {
+      containerUuid,
+      isLoading
+    }) {
+      Vue.set(state.isLoadingProcessOfBrowser, containerUuid, {
+        isLoading
+      })
+    }
+  },
 
   actions: {
     processActionPerformed({ dispatch, getters }, {
@@ -451,6 +462,12 @@ const processManager = {
   },
 
   getters: {
+    getIsloadingProcessOfBrowser: (state) => ({
+      containerUuid
+    }) => {
+      if (isEmptyValue(containerUuid)) return false
+      return state.isLoadingProcessOfBrowser[containerUuid].isLoading
+    }
   }
 }