SONAR-7809 fix bad rendering when there is no new coverage on the Overview page

diff --git a/server/sonar-web/src/main/js/apps/overview/main/Coverage.js b/server/sonar-web/src/main/js/apps/overview/main/Coverage.js
index d6aee6495a1af4a8a2d5c0579efa43088abd358b..46da2f17b8a308cc20e11f75585e6c71037e39fe 100644 (file)
--- a/server/sonar-web/src/main/js/apps/overview/main/Coverage.js
+++ b/server/sonar-web/src/main/js/apps/overview/main/Coverage.js
@@ -117,14 +117,19 @@ class Coverage extends React.Component {
     const newCoverageMeasure = this.getNewCoverageMeasure(coverageMetricPrefix);
     const newLinesToCover = this.getNewLinesToCover(coverageMetricPrefix);
 
-    const value = newCoverageMeasure ? (
+    const newCoverageValue = newCoverageMeasure ?
+        getPeriodValue(newCoverageMeasure, leakPeriod.index) : null;
+    const newLinesToCoverValue = newLinesToCover ?
+        getPeriodValue(newLinesToCover, leakPeriod.index) : null;
+
+    const formattedValue = newCoverageValue != null ? (
         <div>
           <DrilldownLink
               component={component.key}
               metric={newCoverageMeasure.metric.key}
               period={leakPeriod.index}>
             <span className="js-overview-main-new-coverage">
-              {formatMeasure(getPeriodValue(newCoverageMeasure, leakPeriod.index), 'PERCENT')}
+              {formatMeasure(newCoverageValue, 'PERCENT')}
             </span>
           </DrilldownLink>
         </div>
@@ -132,7 +137,7 @@ class Coverage extends React.Component {
         <span>—</span>
     );
 
-    const label = newLinesToCover ? (
+    const label = newLinesToCoverValue != null ? (
         <div className="overview-domain-measure-label">
           {translate('overview.coverage_on')}
           <br/>
@@ -142,7 +147,7 @@ class Coverage extends React.Component {
               metric={newLinesToCover.metric.key}
               period={leakPeriod.index}>
             <span className="js-overview-main-new-coverage">
-              {formatMeasure(getPeriodValue(newLinesToCover, leakPeriod.index), 'SHORT_INT')}
+              {formatMeasure(newLinesToCoverValue, 'SHORT_INT')}
             </span>
           </DrilldownLink>
           {getMetricName('new_ncloc')}
@@ -156,7 +161,7 @@ class Coverage extends React.Component {
     return (
         <div className="overview-domain-measure">
           <div className="overview-domain-measure-value">
-            {value}
+            {formattedValue}
           </div>
           {label}
         </div>