Merge pull request #3798 from alisman/fxSum

Query page sample count should show sum of each study's allSampleCount property
cBioPortal · Jun 14, 2021 · 064cb6a · 064cb6a
2 parents 6ba1ace + d1ac275
commit 064cb6a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/shared/components/query/QueryStore.ts b/src/shared/components/query/QueryStore.ts
@@ -122,7 +122,10 @@ export enum Focus {
 // mobx observable
 export class QueryStore {
     constructor(urlWithInitialParams?: string) {
+        getBrowserWindow().activeQueryStore = this;
+
         makeObservable(this);
+
         this.initialize(urlWithInitialParams);
     }
 
@@ -1018,6 +1021,10 @@ export class QueryStore {
         },
     });
 
+    @computed get sampleCountForSelectedStudies() {
+        return _.sumBy(this.selectableSelectedStudies, s => s.allSampleCount);
+    }
+
     readonly sampleLists = remoteData({
         invoke: async () => {
             if (!this.isSingleNonVirtualStudySelected) {

diff --git a/src/shared/components/query/StudySelectorStats.tsx b/src/shared/components/query/StudySelectorStats.tsx
@@ -31,10 +31,7 @@ export const StudySelectorStats: React.FunctionComponent<{
                                 {numSelectedStudies === 1 ? 'study' : 'studies'}{' '}
                                 selected (
                                 <b>
-                                    {
-                                        props.store.profiledSamplesCount.result
-                                            .all
-                                    }
+                                    {props.store.sampleCountForSelectedStudies}
                                 </b>{' '}
                                 samples)
                             </a>