updated type of sampleToMutationGenePanelId

src/pages/patientView/copyNumberAlterations/CopyNumberTableWrapper.tsx

@@ -55,7 +55,7 @@ type ICopyNumberTableWrapperProps = {
     status:"loading"|"available"|"unavailable";
     currentGeneFilter:GeneFilterOption;
     onFilterGenes?:(option:GeneFilterOption)=>void;
-    sampleToMutationGenePanelId?: any;
+    sampleToMutationGenePanelId?: {[sampleId:string]:string};
 };
 
 @observer

src/shared/components/mutationTable/MutationTable.tsx

@@ -95,7 +95,7 @@ export interface IMutationTableProps {
     showCountHeader?:boolean;
     columnVisibility?: {[columnId: string]: boolean};
     columnVisibilityProps?: IColumnVisibilityControlsProps;
-    sampleToMutationGenePanelId?: any;
+    sampleToMutationGenePanelId?: {[sampleId:string]: string};
 }
 
 export enum MutationTableColumnType {

src/shared/components/mutationTable/column/PanelColumnFormatter.tsx

@@ -1,40 +1,67 @@
 import * as React from 'react';
-import {Mutation, DiscreteCopyNumberData} from "shared/api/generated/CBioPortalAPI";
+import {
+	Mutation,
+	DiscreteCopyNumberData
+} from 'shared/api/generated/CBioPortalAPI';
 
 interface PanelColumnFormatterProps {
-  data: Mutation[] | DiscreteCopyNumberData[],
-  sampleToMutationGenePanelId: any,
+	data: Mutation[] | DiscreteCopyNumberData[];
+	sampleToMutationGenePanelId: {[sampleId:string]: string} | undefined;
 }
 
-class PanelColumn extends React.Component<PanelColumnFormatterProps,{}> {
-  constructor(props:PanelColumnFormatterProps) {
-    super(props);
-  }
-
-  render() {
-    const {data, sampleToMutationGenePanelId} = this.props;
-
-    if (!Object.keys(sampleToMutationGenePanelId).length) {
-      return <i className='fa fa-spinner fa-pulse'/>;
-    }
-
-    const genePanelIds:string[] = getGenePanelIds(data, sampleToMutationGenePanelId);
+class PanelColumn extends React.Component<PanelColumnFormatterProps, {}> {
+	constructor(props: PanelColumnFormatterProps) {
+		super(props);
+	}
+
+	render() {
+    const { data, sampleToMutationGenePanelId } = this.props;
 
-    return (
-      <div style={{position:'relative'}}>
-        <ul  style={{marginBottom:0}} className="list-inline list-unstyled">{ genePanelIds.join(', ') }</ul>
-      </div>
-    )
-  }
-}
+    if (!sampleToMutationGenePanelId) return;
 
-const getGenePanelIds = (data:any[], sampleToMutationGenePanelId:any) => {
-  const sampleIds = data.map(datum => datum.sampleId);
-  return sampleIds.map(id => sampleToMutationGenePanelId[id]);
+		if (sampleToMutationGenePanelId && !Object.keys(sampleToMutationGenePanelId).length) {
+			return <i className='fa fa-spinner fa-pulse' />;
+		}
+
+		const genePanelIds: string[] = getGenePanelIds(
+			data,
+			sampleToMutationGenePanelId
+		);
+
+		return (
+			<div style={{ position: 'relative' }}>
+				<ul style={{ marginBottom: 0 }} className='list-inline list-unstyled'>
+					{genePanelIds.join(', ')}
+				</ul>
+			</div>
+		);
+	}
 }
 
+const getGenePanelIds = (
+	data: any[],
+	sampleToMutationGenePanelId: {[sampleId:string]: string} | undefined
+) => {
+  if (sampleToMutationGenePanelId) {
+    const sampleIds = data.map((datum) => datum.sampleId);
+    return sampleIds.map((id) => sampleToMutationGenePanelId[id]);
+  }
+  return [];
+};
+
 export default {
-  renderFunction: (data:Mutation[] | DiscreteCopyNumberData[], sampleToMutationGenePanelId:any) => <PanelColumn data={data} sampleToMutationGenePanelId={sampleToMutationGenePanelId}/>,
-  download: (data:Mutation[] | DiscreteCopyNumberData[], sampleToMutationGenePanelId:any) => getGenePanelIds(data, sampleToMutationGenePanelId),
-  getGenePanelIds
-}
+	renderFunction: (
+		data: Mutation[] | DiscreteCopyNumberData[],
+		sampleToMutationGenePanelId: {[sampleId:string]: string} | undefined
+	) => (
+		<PanelColumn
+			data={data}
+			sampleToMutationGenePanelId={sampleToMutationGenePanelId}
+		/>
+	),
+	download: (
+		data: Mutation[] | DiscreteCopyNumberData[],
+		sampleToMutationGenePanelId: {[sampleId:string]: string} | undefined
+	) => getGenePanelIds(data, sampleToMutationGenePanelId),
+	getGenePanelIds
+};