fix measure formatting when empty string is passed

diff --git a/server/sonar-web/src/main/js/helpers/measures.js b/server/sonar-web/src/main/js/helpers/measures.js
index 23a27e9270e280bb22bf168d2ab32eeb8758a00b..dd8b4ea58547823beaaeddcc6c92a23618fe1362 100644 (file)
--- a/server/sonar-web/src/main/js/helpers/measures.js
+++ b/server/sonar-web/src/main/js/helpers/measures.js
@@ -56,7 +56,7 @@ export function groupByDomain (metrics) {
  */
 
 function useFormatter (value, formatter) {
-  return value != null && formatter != null ?
+  return value != null && value !== '' && formatter != null ?
       formatter(value) : null;
 }
 

diff --git a/server/sonar-web/tests/helpers/measures-test.js b/server/sonar-web/tests/helpers/measures-test.js
index e302e2db0ab28cd0a73b98cdf13048e7551cd2e9..503d6e3214a2998f23bd1c26838d435a96674b50 100644 (file)
--- a/server/sonar-web/tests/helpers/measures-test.js
+++ b/server/sonar-web/tests/helpers/measures-test.js
@@ -146,6 +146,10 @@ describe('Measures', function () {
       expect(formatMeasure('random value', 'RANDOM_TYPE')).to.equal('random value');
     });
 
+    it('should return null if value is empty string', function () {
+      expect(formatMeasure('', 'PERCENT')).to.be.null;
+    });
+
     it('should not fail without parameters', function () {
       expect(formatMeasure()).to.be.null;
     });