Merge pull request #4089 from dippindots/fix-8920

Do not fetch structural variants data when there are no applicable samples
cBioPortal · Dec 14, 2021 · 4872f7e · 4872f7e
2 parents 92842f7 + ab9c6ea
commit 4872f7e
Showing 1 changed file with 17 additions and 10 deletions.
diff --git a/src/pages/resultsView/ResultsViewPageStore.ts b/src/pages/resultsView/ResultsViewPageStore.ts
@@ -3177,17 +3177,24 @@ export class ResultsViewPageStore
                 [] as StructuralVariantFilter['sampleMolecularIdentifiers']
             );
 
-            const data = {
-                entrezGeneIds: _.map(
-                    this.genes.result,
-                    (gene: Gene) => gene.entrezGeneId
-                ),
-                sampleMolecularIdentifiers: filters,
-            } as StructuralVariantFilter;
+            // filters can be an empty list
+            // when all selected samples are coming from studies that don't have structural variant profile
+            // in this case, we should not fetch structural variants data
+            if (_.isEmpty(filters)) {
+                return [];
+            } else {
+                const data = {
+                    entrezGeneIds: _.map(
+                        this.genes.result,
+                        (gene: Gene) => gene.entrezGeneId
+                    ),
+                    sampleMolecularIdentifiers: filters,
+                } as StructuralVariantFilter;
 
-            return await client.fetchStructuralVariantsUsingPOST({
-                structuralVariantFilter: data,
-            });
+                return await client.fetchStructuralVariantsUsingPOST({
+                    structuralVariantFilter: data,
+                });
+            }
         },
     });