Merge pull request #4083 from dippindots/fix-9076

Returns an empty list for undefined profileIds in isSampleProfiledInMultiple function
cBioPortal · Dec 14, 2021 · 55d2301 · 55d2301
2 parents 4872f7e + ce73db4
commit 55d2301
Showing 1 changed file with 15 additions and 10 deletions.
diff --git a/src/shared/lib/isSampleProfiled.ts b/src/shared/lib/isSampleProfiled.ts
@@ -62,17 +62,22 @@ export function isSampleProfiledInSomeMolecularProfile(
 
 export function isSampleProfiledInMultiple(
     uniqueSampleKey: string,
-    molecularProfileIds: string[],
+    molecularProfileIds: string[] | undefined,
     coverageInformation: CoverageInformation,
     hugoGeneSymbol?: string
 ): boolean[] {
-    // returns boolean[] in same order as molecularProfileIds
-    const profiledReport = getSampleProfiledReport(
-        uniqueSampleKey,
-        coverageInformation,
-        hugoGeneSymbol
-    );
-    return molecularProfileIds.map(
-        molecularProfileId => !!profiledReport[molecularProfileId]
-    );
+    // returns empty list if molecularProfileIds is undefined
+    if (!molecularProfileIds) {
+        return [];
+    } else {
+        // returns boolean[] in same order as molecularProfileIds
+        const profiledReport = getSampleProfiledReport(
+            uniqueSampleKey,
+            coverageInformation,
+            hugoGeneSymbol
+        );
+        return molecularProfileIds.map(
+            molecularProfileId => !!profiledReport[molecularProfileId]
+        );
+    }
 }