Fix Record Access (#832)

* Fix Record Access

* correct observation

Co-authored-by: elsiosanchez <elsiossanches@gmail.com>

src/components/ADempiere/ContextMenu/contextMenuMobile.vue

@@ -243,8 +243,15 @@ export default {
       } else if (action === 'shareLink') {
         this.setShareLink()
       } else if (action.action === 'recordAccess') {
-        this.$store.commit('changeShowRigthPanel', true)
+        this.$store.commit('changeShowRigthPanel', false)
         this.$store.commit('setRecordAccess', true)
+        this.$router.push({
+          name: this.$route.name,
+          query: {
+            ...this.$route.query,
+            typeAction: action.action
+          }
+        }, () => {})
       } else {
         this.runAction(action)
       }

src/components/ADempiere/RecordAccess/recordAccessMobile.vue

@@ -349,10 +349,6 @@ export default {
       }
     },
     SendRecorAccess(list) {
-      // list.forEach(element => {
-      //   element.isReadOnly = this.isReadonly
-      //   element.isDependentEntities = this.isDependentEntities
-      // })
       this.saveRecordAccess(list)
     }
   }

src/components/ADempiere/RightPanel/index.vue

@@ -28,7 +28,7 @@
           <div
             class="handle-button"
             :style="{'top':buttonTop+'px','background-color':theme}"
-            @click="isShowRightPanel=!isShowRightPanel"
+            @click="closePanel"
           >
             <i :class="icon" style="color: white;" />
           </div>
@@ -108,6 +108,17 @@ export default {
         window.removeEventListener('click', this.closeSidebar)
       }
     },
+    closePanel() {
+      this.$router.push({
+        name: this.$route.name,
+        query: {
+          ...this.$route.query,
+          typeAction: ''
+        }
+      }, () => {})
+      this.$store.commit('changeShowRigthPanel', false)
+      this.isShowRightPanel = !this.isShowRightPanel
+    },
     insertToBody() {
       const elx = this.$refs.rightMenu
       const body = document.querySelector('body')

src/store/modules/ADempiere/contextMenu.js

@@ -1,4 +1,4 @@
-import { recursiveTreeSearch } from '@/utils/ADempiere/valueUtils.js'
+import { isEmptyValue, recursiveTreeSearch } from '@/utils/ADempiere/valueUtils.js'
 import { requestListDocumentActions, requestListDocumentStatuses } from '@/api/ADempiere/window'
 
 // Store used for set all related to context menu
@@ -50,8 +50,11 @@ const contextMenu = {
     addlistDocumentStatus(state, payload) {
       state.listDocumentStatus = payload
     },
-    changeShowRigthPanel(state) {
+    changeShowRigthPanel(state, params) {
-      state.isShowRightPanel = !state.isShowRightPanel
+      if (isEmptyValue(params)) {
+        state.isShowRightPanel = !state.isShowRightPanel
+      }
+      state.isShowRightPanel = params
     },
     changeShowPopoverField(state) {
       state.isShowPopoverField = !state.isShowPopoverField

src/views/ADempiere/Window/windowLogicComponent.js

@@ -117,6 +117,10 @@ export default {
       return component
     },
     showRecordAccess() {
+      if (this.$route.query.typeAction === 'recordAccess') {
+        this.$store.commit('changeShowRigthPanel', true)
+        return true
+      }
       return this.$store.getters.getShowRecordAccess
     },
     isNewRecord() {