From dd3d8d85d3a96dc2830399c42cebc23202e78632 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9gis=20Hanol?= <regis@hanol.fr>
Date: Wed, 17 Aug 2016 23:28:29 +0200
Subject: [PATCH] FIX: proper logic for 'noTitle' param of the number helper

---
 app/assets/javascripts/discourse/helpers/application.js.es6 | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/assets/javascripts/discourse/helpers/application.js.es6 b/app/assets/javascripts/discourse/helpers/application.js.es6
index 4c414db6b46..2bedeb4b0bd 100644
--- a/app/assets/javascripts/discourse/helpers/application.js.es6
+++ b/app/assets/javascripts/discourse/helpers/application.js.es6
@@ -20,11 +20,13 @@ registerUnbound('number', (orig, params) => {
   if (params['class']) {
     classNames += ' ' + params['class'];
   }
+
   let result = "<span class='" + classNames + "'";
+  let addTitle = params.noTitle ? false : true;
 
   // Round off the thousands to one decimal place
   const n = number(orig);
-  if (n.toString() !== title.toString() && !params.noTitle) {
+  if (n.toString() !== title.toString() && addTitle) {
     result += " title='" + Handlebars.Utils.escapeExpression(title) + "'";
   }
   result += ">" + n + "</span>";