10467: add "Files & Links" tab of the Patient View to the Study View

fixing a bug
cBioPortal · Aug 9, 2024 · 0cecacb · 0cecacb
1 parent 23d4233
commit 0cecacb
Show file tree

Hide file tree

Showing 5 changed files with 428 additions and 12 deletions.
diff --git a/src/pages/studyView/StudyViewPage.tsx b/src/pages/studyView/StudyViewPage.tsx
@@ -377,6 +377,17 @@ export default class StudyViewPage extends React.Component<
         }
     }
 
+    @computed get shouldShowSampleResources() {
+        if (this.store.sampleResourceData.isComplete) {
+            return _.some(
+                this.store.sampleResourceData.result,
+                data => data.length > 0
+            );
+        } else {
+            return false;
+        }
+    }
+
     @computed get shouldShowResources() {
         if (this.store.resourceIdToResourceData.isComplete) {
             return _.some(
@@ -725,7 +736,7 @@ export default class StudyViewPage extends React.Component<
                                             StudyViewPageTabKeyEnum.FILES_AND_LINKS
                                         }
                                         linkText={RESOURCES_TAB_NAME}
-                                        hide={!this.shouldShowResources}
+                                        hide={!this.shouldShowSampleResources}
                                     >
                                         <div>
                                             <ResourcesTab

diff --git a/src/pages/studyView/StudyViewPageStore.ts b/src/pages/studyView/StudyViewPageStore.ts
@@ -5784,6 +5784,43 @@ export class StudyViewPageStore
         },
     });
 
+    readonly sampleResourceData = remoteData<{
+        [sampleId: string]: ResourceData[];
+    }>({
+        await: () => [this.resourceDefinitions, this.samples],
+        invoke: () => {
+            const sampleResourceDefinitions = this.resourceDefinitions.result!.filter(
+                d => d.resourceType === 'SAMPLE'
+            );
+            if (!sampleResourceDefinitions.length) {
+                return Promise.resolve({});
+            }
+
+            const samples = this.samples.result!;
+            const ret: { [sampleId: string]: ResourceData[] } = {};
+            const promises = [];
+            for (const sample of samples) {
+                for (const resource of sampleResourceDefinitions) {
+                    promises.push(
+                        internalClient
+                            .getAllResourceDataOfSampleInStudyUsingGET({
+                                sampleId: sample.sampleId,
+                                studyId: sample.studyId, // TODO:
+                                resourceId: resource.resourceId,
+                                projection: 'DETAILED',
+                            })
+                            .then(data => {
+                                ret[sample.sampleId] =
+                                    ret[sample.sampleId] || [];
+                                ret[sample.sampleId].push(...data);
+                            })
+                    );
+                }
+            }
+            return Promise.all(promises).then(() => ret);
+        },
+    });
+
     readonly resourceIdToResourceData = remoteData<{
         [resourceId: string]: ResourceData[];
     }>({