From 3f6d7a727905df5f1cd9a509fe67728b50330abc Mon Sep 17 00:00:00 2001
From: Onur Sumer <s.onur.sumer@gmail.com>
Date: Wed, 20 Nov 2019 15:48:47 -0500
Subject: [PATCH] Merge pull request #128 from onursumer/gnomad-no-data-fix

Fix the broken tooltip for the no gnomad data case
---
 packages/react-mutation-mapper/package.json   |  2 +-
 .../src/component/gnomad/GnomadFrequency.tsx  | 39 +++++++------------
 2 files changed, 15 insertions(+), 26 deletions(-)

diff --git a/packages/react-mutation-mapper/package.json b/packages/react-mutation-mapper/package.json
index e227e048155..3860c411a6e 100644
--- a/packages/react-mutation-mapper/package.json
+++ b/packages/react-mutation-mapper/package.json
@@ -1,6 +1,6 @@
 {
   "name": "react-mutation-mapper",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Generic Mutation Mapper",
   "author": "cBioPortal",
   "license": "GNU Affero General Public License v3.0",
diff --git a/packages/react-mutation-mapper/src/component/gnomad/GnomadFrequency.tsx b/packages/react-mutation-mapper/src/component/gnomad/GnomadFrequency.tsx
index 939176fbe8a..843b8663b3e 100644
--- a/packages/react-mutation-mapper/src/component/gnomad/GnomadFrequency.tsx
+++ b/packages/react-mutation-mapper/src/component/gnomad/GnomadFrequency.tsx
@@ -89,7 +89,6 @@ export default class GnomadFrequency extends React.Component<GnomadFrequencyProp
         let gnomadUrl = "";
         let display: JSX.Element;
         let overlay: (() => JSX.Element) | null = null;
-        let content: JSX.Element | null = null;
         let result : {[key:string]: GnomadSummary} = {};
 
         // Checking if gnomad data is valid
@@ -144,37 +143,27 @@ export default class GnomadFrequency extends React.Component<GnomadFrequencyProp
             // The column will show the total frequency
             // Column will show 0 if the total frequency is 0, still has the tooltip to show the gnomad table (since gnomad data is still available)
             if (result["Total"].alleleFrequency === 0) {
-                display = <span>0</span>
+                display = <span>0</span>;
             } else {
-                display = <span>{parseFloat(result["Total"].alleleFrequency.toString()).toExponential(1)}</span>
+                display = <span>{parseFloat(result["Total"].alleleFrequency.toString()).toExponential(1)}</span>;
             }
 
             overlay = () => <GnomadFrequencyTable data={sorted} gnomadUrl={gnomadUrl} />;
         }
         else {
-            display =
-                <DefaultTooltip
-                    placement="topRight"
-                    overlay={(<span>Variant has no data in gnomAD.</span>)}
-                >
-                    <span style={{height: '100%', width: '100%', display: 'block', overflow: 'hidden'}}>&nbsp;</span>
-                </DefaultTooltip>
+            display = <span style={{height: '100%', width: '100%', display: 'block', overflow: 'hidden'}}>&nbsp;</span>;
+            overlay = () => <span>Variant has no data in gnomAD.</span>;
         }
 
-        // add a tooltip if the gnomad value is valid
-        if (overlay) {
-            content = (
-                <DefaultTooltip
-                    overlay={overlay}
-                    placement="topRight"
-                    trigger={['hover', 'focus']}
-                    destroyTooltipOnHide={true}
-                >
-                    {display}
-                </DefaultTooltip>
-            );
-        }
-
-        return content;
+        return (
+            <DefaultTooltip
+                overlay={overlay}
+                placement="topRight"
+                trigger={['hover', 'focus']}
+                destroyTooltipOnHide={true}
+            >
+                {display}
+            </DefaultTooltip>
+        );
     }
 }