From c039e28db8b40417d07fdc5e1acb1bf252e7478c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C5=BEe=20Fortun?= <joze.fortun@gmail.com>
Date: Thu, 16 Nov 2023 14:08:01 +0100
Subject: [PATCH] Fix all records view processing and navigation

---
 .../src/views/Admin/Modules/Records/View.vue  | 31 ++++++++++++++++---
 1 file changed, 27 insertions(+), 4 deletions(-)

diff --git a/client/web/compose/src/views/Admin/Modules/Records/View.vue b/client/web/compose/src/views/Admin/Modules/Records/View.vue
index 9b8da5ed6..a2170abd6 100644
--- a/client/web/compose/src/views/Admin/Modules/Records/View.vue
+++ b/client/web/compose/src/views/Admin/Modules/Records/View.vue
@@ -51,7 +51,7 @@
     </b-alert>
 
     <b-row
-      v-if="module && record"
+      v-if="module"
       no-gutters
     >
       <b-col
@@ -59,6 +59,7 @@
         :key="index"
         md="3"
         cols="12"
+        style="max-height: 650px; height: 650px;"
       >
         <component
           :is="getRecordComponent"
@@ -148,6 +149,11 @@ export default {
         namespace: this.$attrs.namespace,
       },
 
+      recordNavigation: {
+        prev: undefined,
+        next: undefined,
+      },
+
       abortableRequests: [],
     }
   },
@@ -219,7 +225,7 @@ export default {
       return undefined
     },
 
-    recordNavigation () {
+    currentRecordNavigation () {
       const { recordID } = this.record || {}
       return this.getNextAndPrevRecord(recordID)
     },
@@ -229,15 +235,32 @@ export default {
     '$attrs.recordID': {
       immediate: true,
       handler () {
+        this.record = undefined
+        this.initialRecordState = undefined
         this.loadRecord()
       },
     },
+
+    currentRecordNavigation: {
+      handler (rn, oldRn) {
+        // To prevent hiding and then showing the record navigation
+        // We use the old value if its valid and the current one isn't
+        if (rn.prev || rn.next) {
+          this.recordNavigation = rn
+        } else if (this.recordID !== NoID && (oldRn.prev || oldRn.next)) {
+          this.recordNavigation = oldRn
+        } else {
+          this.recordNavigation = {
+            prev: undefined,
+            next: undefined,
+          }
+        }
+      },
+    },
   },
 
   created () {
     this.createBlocks()
-    this.record = new compose.Record(this.module, { values: this.values })
-    this.initialRecordState = this.record.clone()
   },
 
   beforeDestroy () {