From fd42e2e2b652054ae5a68561b3a03fbd5e57c52a Mon Sep 17 00:00:00 2001
From: Joel Collins <git@jtcollins.net>
Date: Tue, 1 Dec 2020 17:13:20 +0000
Subject: [PATCH] Fixed broken dataset rendering if key exists but is empty

---
 .../tabContentComponents/galleryContent.vue   | 62 ++++++++++---------
 1 file changed, 33 insertions(+), 29 deletions(-)

diff --git a/openflexure_microscope/api/static/src/components/tabContentComponents/galleryContent.vue b/openflexure_microscope/api/static/src/components/tabContentComponents/galleryContent.vue
index 3ae91b76..c00afafc 100644
--- a/openflexure_microscope/api/static/src/components/tabContentComponents/galleryContent.vue
+++ b/openflexure_microscope/api/static/src/components/tabContentComponents/galleryContent.vue
@@ -24,7 +24,7 @@
             <a href="#">Filter</a>
             <div
               :class="{
-                'uk-light uk-background-secondary': $store.state.darkMode
+                'uk-light uk-background-secondary': $store.state.darkMode,
               }"
               class="uk-navbar-dropdown"
             >
@@ -159,10 +159,10 @@ export default {
     captureCard,
     scanCard,
     ZipDownloader,
-    Paginate
+    Paginate,
   },
 
-  data: function() {
+  data: function () {
     return {
       captures: [],
       checkedTags: [],
@@ -171,15 +171,15 @@ export default {
       scanTag: "scan",
       unwatchStoreFunction: null,
       maxitems: 10,
-      page: 1
+      page: 1,
     };
   },
 
   computed: {
-    capturesUri: function() {
+    capturesUri: function () {
       return `${this.$store.getters.baseUri}/api/v2/captures`;
     },
-    allTags: function() {
+    allTags: function () {
       // Return an array of unique tags across all captures
       var tags = [];
       for (var capture of this.captures) {
@@ -192,12 +192,16 @@ export default {
       return tags.sort();
     },
 
-    noScanCaptures: function() {
+    noScanCaptures: function () {
       // List of captures that are not part of a scan
       var captures = [];
       for (var capture of this.captures) {
         // Add to capture list if matched
-        if (!capture.dataset) {
+        if (
+          !capture.dataset || // If no dataset key
+          (capture.dataset.constructor === Object && // Or dataset is an object...
+            Object.keys(capture.dataset).length === 0) // ...but it's empty
+        ) {
           captures.push(capture);
         }
       }
@@ -205,7 +209,7 @@ export default {
       return captures;
     },
 
-    allScans: function() {
+    allScans: function () {
       // List of scans as capture-like objects
       var scans = {};
 
@@ -248,12 +252,12 @@ export default {
       return scans;
     },
 
-    scanList: function() {
+    scanList: function () {
       // List of scans, obtained from this.allScans values
       return Object.values(this.allScans);
     },
 
-    itemList: function() {
+    itemList: function () {
       // Get list of current items to show
       // If galleryFolder (ie inside a scan folder), show scan captures
       // Otherwise, show root captures and scan cards
@@ -264,12 +268,12 @@ export default {
       }
     },
 
-    filteredItems: function() {
+    filteredItems: function () {
       // Filter itemList by checkedTags
       return this.filterCaptures(this.itemList, this.checkedTags);
     },
 
-    filteredCaptures: function() {
+    filteredCaptures: function () {
       var captures = {};
 
       for (var item of this.filteredItems) {
@@ -288,19 +292,19 @@ export default {
       return captures;
     },
 
-    sortedItems: function() {
+    sortedItems: function () {
       // Sort filteredItems using sortCaptures function
       return this.sortCaptures(this.filteredItems);
     },
 
-    pagedItems: function() {
+    pagedItems: function () {
       let startIndex = (this.page - 1) * this.maxitems;
       return this.sortedItems.slice(startIndex, startIndex + this.maxitems);
     },
 
-    numberOfPages: function() {
+    numberOfPages: function () {
       return Math.floor(this.sortedItems.length / this.maxitems);
-    }
+    },
   },
 
   mounted() {
@@ -312,13 +316,13 @@ export default {
     });
   },
 
-  created: function() {
+  created: function () {
     // Watch for host 'ready', then update status
     this.unwatchStoreFunction = this.$store.watch(
       (state, getters) => {
         return getters.ready;
       },
-      ready => {
+      (ready) => {
         if (ready) {
           // If the connection is now ready, update capture list
           this.updateCaptures();
@@ -348,20 +352,20 @@ export default {
       }
     },
 
-    updateCaptures: function() {
+    updateCaptures: function () {
       if (this.$store.state.available) {
         axios
           .get(this.capturesUri)
-          .then(response => {
+          .then((response) => {
             this.captures = response.data;
           })
-          .catch(error => {
+          .catch((error) => {
             this.modalError(error); // Let mixin handle error
           });
       }
     },
 
-    filterCaptures: function(list, filterTags) {
+    filterCaptures: function (list, filterTags) {
       // Filter a list of captures by an array of tags
       var result = [];
       for (var capture of list) {
@@ -370,7 +374,7 @@ export default {
 
         // Filter by selected tags
         var tags = capture.tags;
-        let checker = (arr, target) => target.every(v => arr.includes(v));
+        let checker = (arr, target) => target.every((v) => arr.includes(v));
         // True if all tags match
         includeCapture = checker(tags, filterTags);
 
@@ -383,7 +387,7 @@ export default {
       return result;
     },
 
-    sortCaptures: function(list) {
+    sortCaptures: function (list) {
       // Sort a list of captures by metadata time
       function compare(a, b) {
         if (a.time < b.time) return -1;
@@ -398,15 +402,15 @@ export default {
       }
     },
 
-    galleryBack: function() {
+    galleryBack: function () {
       this.galleryFolder = "";
       this.page = 1;
     },
 
-    selectFolder: function(folderID) {
+    selectFolder: function (folderID) {
       this.galleryFolder = folderID;
-    }
-  }
+    },
+  },
 };
 </script>