]> source.dussan.org Git - sonarqube.git/commitdiff
improve code quality
authorStas Vilchik <vilchiks@gmail.com>
Tue, 22 Dec 2015 13:57:08 +0000 (14:57 +0100)
committerStas Vilchik <vilchiks@gmail.com>
Tue, 22 Dec 2015 13:57:08 +0000 (14:57 +0100)
server/sonar-web/src/main/js/apps/update-center/plugin-changelog-view.js
server/sonar-web/src/main/js/apps/users/tokens-view.js
server/sonar-web/src/main/js/helpers/handlebars/default.js
server/sonar-web/src/main/js/helpers/handlebars/recursive.js
server/sonar-web/src/main/js/main/app.js
server/sonar-web/src/main/js/widgets/old/pie-chart.js

index fe9e1bbbaa6b312b15749e7abbfb5c5fef1278d7..eb78098ed3a01504cb915d83f2a762887d427b59 100644 (file)
@@ -11,9 +11,8 @@ export default Popup.extend({
     this.$('[data-toggle="tooltip"]').tooltip({ container: 'body', placement: 'bottom' });
   },
 
-  onClose: function () {
-    // TODO why onClose? why not onDestroy?
-    Popup.prototype.onClose.apply(this, arguments);
+  onDestroy: function () {
+    Popup.prototype.onDestroy.apply(this, arguments);
     this.$('[data-toggle="tooltip"]').tooltip('destroy');
   },
 
index 4dc0b77bf0b85d9103365337c1c418d60d23ca31..505a752c1f0508be4d814f8081538b31c6a384a5 100644 (file)
@@ -42,8 +42,8 @@ export default Modal.extend({
           this.newToken = response.token;
           this.requestTokens();
         })
-        .catch(e => {
-          e.response.json().then(response => {
+        .catch(error => {
+          error.response.json().then(response => {
             this.errors = response.errors;
             this.render();
           });
index 56ea431ae7754a5218efbb15f2cf85f7d0768c44..0bfef26251531f673bdf38e7ac9d377597f51edc 100644 (file)
@@ -3,4 +3,4 @@ module.exports = function () {
   return args.reduce(function (prev, current) {
     return prev != null ? prev : current;
   }, null);
-}
+};
index c123d31416aac60039d9aeb0649a093091a913a3..0d02840139ad44228d532d4bfa90bc7f0bcecead 100644 (file)
@@ -12,4 +12,4 @@ module.exports = function (children, options) {
   });
 
   return out;
-}
+};
index ab58da869aba3ba1e5ceb2e92e6ca7e84bd6d632..a68bf42b4b0caa5604b7ec156dd883342066f50a 100644 (file)
@@ -66,5 +66,7 @@ window.require = (module) => {
       return Marionette;
     case 'moment':
       return moment;
+    default:
+      return null;
   }
 };
index 929eb6a08baea183cbe7d502177048877f75c664..81f6cd7900950f36e09152c71b05f19ae715e949 100644 (file)
@@ -288,8 +288,8 @@ window.SonarWidgets = window.SonarWidgets == null ? {} : window.SonarWidgets;
         .on('mouseenter', function(d, i) {
           return enterHandler(d3.select(this), d.data, i, true);
         })
-        .on('mouseleave', function(d, i) {
-          return leaveHandler(d3.select(this), d.data, i, true);
+        .on('mouseleave', function() {
+          return leaveHandler(d3.select(this));
         })
         .on('click', function(d) {
           return clickHandler(d.data);
@@ -300,7 +300,7 @@ window.SonarWidgets = window.SonarWidgets == null ? {} : window.SonarWidgets;
           return enterHandler(d3.select(widget.sectors[0][i]), d, i, false);
         })
         .on('mouseleave', function(d, i) {
-          return leaveHandler(d3.select(widget.sectors[0][i]), d, i, false);
+          return leaveHandler(d3.select(widget.sectors[0][i]));
         })
         .on('click', function(d) {
           return clickHandler(d);