Fixes table display bug when collection has one attribute and it is hidden

v3/src/data-interactive/handlers/collection-handler.test.ts

@@ -84,11 +84,11 @@ describe("DataInteractive CollectionHandler", () => {
 
   it("get works", () => {
     const documentContent = appState.document.content!
-    const { dataset } = setupTestDataset()
+    const { dataset, a3 } = setupTestDataset()
     documentContent.createDataSet(getSnapshot(dataset))
     const dataContext = getSharedDataSets(documentContent)[0].dataSet
     const c1 = dataContext.collections[0]
-    expect(handler.get?.({}).success).toBe(false)
+    dataContext.moveAttribute(a3.id, { collection: c1.id })
     expect(handler.get?.({ dataContext }).success).toBe(false)
 
     // Grouped collection

v3/src/models/shared/shared-case-metadata.ts

@@ -1,5 +1,5 @@
-import { observable } from "mobx"
-import { getSnapshot, getType, Instance, ISerializedActionCall, types } from "mobx-state-tree"
+import { observable, reaction, comparer } from "mobx"
+import { getSnapshot, getType, Instance, ISerializedActionCall, types, addDisposer } from "mobx-state-tree"
 import { onAnyAction } from "../../utilities/mst-utils"
 import { CategorySet, createProvisionalCategorySet, ICategorySet } from "../data/category-set"
 import { DataSet, IDataSet } from "../data/data-set"
@@ -194,6 +194,31 @@ export const SharedCaseMetadata = SharedModel
       return categorySet
     }
   }))
+  .actions(self => ({
+    afterCreate() {
+      // respond to change of collections by showing hidden attributes if all attributes are hidden
+      addDisposer(self, reaction(
+        () => {
+          return {
+            hiddenAttrs: Array.from(self.hidden.keys()),
+            collections: self.data?.collections.map(collection => collection.attributes.map(attr => attr?.id)) || []
+          }
+        },
+        ({hiddenAttrs}) => {
+          self.data?.collections.forEach(collection => {
+            const attrs = collection.attributes
+            if (attrs && attrs.every(attr => attr && hiddenAttrs.includes(attr.id))) {
+              attrs.forEach(attr => attr && self.setIsHidden(attr.id, false))
+            }
+          })
+        },
+        { name: "DataConfigurationModel.afterCreate.reaction [show all hidden attributes in a collection]",
+          fireImmediately: true,
+          equals: comparer.structural
+        }
+      ))
+    }
+  }))
   .actions(applyModelChange)
 
 export interface ISharedCaseMetadata extends Instance<typeof SharedCaseMetadata> {}