Fix quality flaws

diff --git a/sonar-server/src/main/js/issues/app.js b/sonar-server/src/main/js/issues/app.js
index f32c73c1e06f4188d257c48385478421d608599e..28c63e3c2b2d6e2a84555ded68c8214f01f53c01 100644 (file)
--- a/sonar-server/src/main/js/issues/app.js
+++ b/sonar-server/src/main/js/issues/app.js
@@ -416,7 +416,7 @@ requirejs(
             sort: sort.value,
             sortText: jQuery('[data-sort=' + sort.value + ']:first').text(),
             asc: asc.value === 'true'
-          }
+          };
         }
       };
 

diff --git a/sonar-server/src/main/js/issues/extra.js b/sonar-server/src/main/js/issues/extra.js
index 212b7d90aa81d61ed5eb68634712624e747e6eaf..c03cb85fde24732320cdb207b666a3dae1891909 100644 (file)
--- a/sonar-server/src/main/js/issues/extra.js
+++ b/sonar-server/src/main/js/issues/extra.js
@@ -378,6 +378,38 @@ define(
 
 
 
+      var IssuesDetailsFavoriteFilterView = FavoriteFiltersModule.DetailsFavoriteFilterView.extend({
+        template: Handlebars.compile(jQuery('#issues-details-favorite-filter-template').html() || ''),
+
+
+        applyFavorite: function (e) {
+          var id = $j(e.target).data('id'),
+              filter = new FavoriteFilter({ id: id }),
+              app = this.options.filterView.options.app;
+
+          filter.fetch({
+            success: function () {
+              app.state.set('search', false);
+              app.favoriteFilter.clear({ silent: true });
+              app.favoriteFilter.set(filter.toJSON());
+            }
+          });
+
+          this.options.filterView.hideDetails();
+        },
+
+
+        serializeData: function () {
+          return _.extend({}, this.model.toJSON(), {
+            items: _.sortBy(this.model.get('choices'), function(item) {
+              return item.name.toLowerCase();
+            })
+          });
+        }
+      });
+
+
+
       var IssuesFavoriteFilterView = FavoriteFiltersModule.FavoriteFilterView.extend({
 
         initialize: function () {
@@ -545,37 +577,6 @@ define(
 
 
 
-      var IssuesDetailsFavoriteFilterView = FavoriteFiltersModule.DetailsFavoriteFilterView.extend({
-        template: Handlebars.compile(jQuery('#issues-details-favorite-filter-template').html() || ''),
-
-
-        applyFavorite: function (e) {
-          var id = $j(e.target).data('id'),
-              filter = new FavoriteFilter({ id: id }),
-              app = this.options.filterView.options.app;
-
-          filter.fetch({
-            success: function () {
-              app.state.set('search', false);
-              app.favoriteFilter.clear({ silent: true });
-              app.favoriteFilter.set(filter.toJSON());
-            }
-          });
-
-          this.options.filterView.hideDetails();
-        },
-
-
-        serializeData: function () {
-          return _.extend({}, this.model.toJSON(), {
-            items: _.sortBy(this.model.get('choices'), function(item) {
-              return item.name.toLowerCase();
-            })
-          });
-        }
-      });
-
-
       var IssuesRouter = Backbone.Router.extend({
 
         routes: {

diff --git a/sonar-server/src/main/js/widgets/pie-chart.js b/sonar-server/src/main/js/widgets/pie-chart.js
index f28f61c6d280208ce488515a2c24fe103e0f0763..fd4838d4a45601c57b64870925e5139395f14cc6 100644 (file)
--- a/sonar-server/src/main/js/widgets/pie-chart.js
+++ b/sonar-server/src/main/js/widgets/pie-chart.js
@@ -71,7 +71,7 @@ window.SonarWidgets = window.SonarWidgets == null ? {} : window.SonarWidgets;
 
 
     var validData = this.components().reduce(function(p, c) {
-      return p && !!c.measures[widget.metricsPriority()[0]]
+      return p && !!c.measures[widget.metricsPriority()[0]];
     }, true);
 
     if (!validData) {

diff --git a/sonar-server/src/main/js/widgets/stack-area.js b/sonar-server/src/main/js/widgets/stack-area.js
index f3d16429a266798b90665db6708b3a52fc6c50e6..e5174e0246203be7ac6f5261bdef6ba729e94147 100644 (file)
--- a/sonar-server/src/main/js/widgets/stack-area.js
+++ b/sonar-server/src/main/js/widgets/stack-area.js
@@ -105,7 +105,9 @@ window.SonarWidgets = window.SonarWidgets == null ? {} : window.SonarWidgets;
         })])
         .nice();
 
-    this.color = function(i) { return widget.colors()[i % cl][0]; };
+    this.color = function(i) {
+      return widget.colors()[i % cl][0];
+    };
 
 
     // Configure the axis