]> source.dussan.org Git - sonarqube.git/commitdiff
improve code quality
authorStas Vilchik <vilchiks@gmail.com>
Mon, 22 Feb 2016 15:05:40 +0000 (16:05 +0100)
committerStas Vilchik <vilchiks@gmail.com>
Mon, 22 Feb 2016 15:05:40 +0000 (16:05 +0100)
21 files changed:
server/sonar-web/src/main/js/apps/coding-rules/rule-details-view.js
server/sonar-web/src/main/js/apps/coding-rules/rule/custom-rule-creation-view.js
server/sonar-web/src/main/js/apps/coding-rules/rule/profile-activation-view.js
server/sonar-web/src/main/js/apps/coding-rules/rule/rule-profiles-view.js
server/sonar-web/src/main/js/apps/coding-rules/workspace-list-item-view.js
server/sonar-web/src/main/js/apps/issues/facets/project-facet.js
server/sonar-web/src/main/js/apps/issues/facets/rule-facet.js
server/sonar-web/src/main/js/apps/issues/facets/tag-facet.js
server/sonar-web/src/main/js/apps/measures/app.js
server/sonar-web/src/main/js/apps/permission-templates/users-view.js
server/sonar-web/src/main/js/apps/project-permissions/users-view.js
server/sonar-web/src/main/js/apps/quality-profiles/change-profile-parent-view.js
server/sonar-web/src/main/js/apps/quality-profiles/copy-profile-view.js
server/sonar-web/src/main/js/apps/quality-profiles/delete-profile-view.js
server/sonar-web/src/main/js/apps/quality-profiles/restore-built-in-profiles-view.js
server/sonar-web/src/main/js/components/SelectList/index.js
server/sonar-web/src/main/js/components/source-viewer/main.js
server/sonar-web/src/main/js/components/source-viewer/measures-overlay.js
server/sonar-web/src/main/js/components/source-viewer/popups/coverage-popup.js
server/sonar-web/src/main/js/components/source-viewer/popups/duplication-popup.js
server/sonar-web/src/main/js/widgets/issue-filter/widget.js

index ddf87c1efbe46675372c33e1eefefc0790644e48..b70cd1df324a2fe43edb09c443dd9c6c9012efcc 100644 (file)
@@ -185,8 +185,8 @@ export default Marionette.LayoutView.extend({
 
     return _.extend(Marionette.ItemView.prototype.serializeData.apply(this, arguments), {
       isEditable,
-      canWrite: this.options.app.canWrite,
       qualityProfilesVisible,
+      canWrite: this.options.app.canWrite,
       allTags: _.union(this.model.get('sysTags'), this.model.get('tags'))
     });
   }
index a76577128f913d1a34f8df5a539509bda98d3c11..cea722f54ccbc67c8f683925a91e265ea481dfeb 100644 (file)
@@ -200,10 +200,10 @@ export default ModalFormView.extend({
     });
 
     return _.extend(ModalFormView.prototype.serializeData.apply(this, arguments), {
-      change: this.model && this.model.has('key'),
       params,
-      severities: ['BLOCKER', 'CRITICAL', 'MAJOR', 'MINOR', 'INFO'],
-      statuses
+      statuses,
+      change: this.model && this.model.has('key'),
+      severities: ['BLOCKER', 'CRITICAL', 'MAJOR', 'MINOR', 'INFO']
     });
   }
 });
index e9f526e398e1dd5228e2362daf76430846b08ab0..2b76f1f1fc6f45cfb15ee3c3b5574cc8658e6943 100644 (file)
@@ -101,9 +101,9 @@ export default ModalForm.extend({
       type: 'POST',
       url: '/api/qualityprofiles/activate_rule',
       data: {
+        severity,
         profile_key: profileKey,
         rule_key: ruleKey,
-        severity,
         params: paramsHash
       },
       statusCode: {
@@ -148,10 +148,10 @@ export default ModalForm.extend({
     const contextProfile = this.options.app.state.get('query').qprofile;
 
     return _.extend(ModalForm.prototype.serializeData.apply(this, arguments), {
-      change: this.model && this.model.has('severity'),
       params,
-      qualityProfiles: availableProfiles,
       contextProfile,
+      change: this.model && this.model.has('severity'),
+      qualityProfiles: availableProfiles,
       severities: ['BLOCKER', 'CRITICAL', 'MAJOR', 'MINOR', 'INFO'],
       saveEnabled: !_.isEmpty(availableProfiles) || (this.model && this.model.get('qProfile')),
       isCustomRule: (this.model && this.model.has('templateKey')) || this.options.rule.has('templateKey')
index 2dda9a223a5e52a3480154caa3525c3c44e32b41..88c5dc208ea8e61acc9ef73d9747cca920775d9f 100644 (file)
@@ -71,8 +71,8 @@ export default Marionette.CompositeView.extend({
       if (that.options.app.state.get('query').qprofile === profile) {
         const activation = {
           severity,
-          inherit: 'NONE',
           params,
+          inherit: 'NONE',
           qProfile: profile
         };
         that.model.set({ activation });
index 86e26521d024991632f00547f3154b27ca039646..13a433a87e9161e1639f443456b9749bce2427ce 100644 (file)
@@ -68,8 +68,8 @@ export default WorkspaceListItemView.extend(RuleFilterMixin).extend({
     activationView.on('profileActivated', function (severity, params, profile) {
       const activation = {
         severity,
-        inherit: 'NONE',
         params,
+        inherit: 'NONE',
         qProfile: profile
       };
       that.model.set({ activation });
index ffbc7772b2d2feb6bd0b3dedc013a7b63c5b2709..50cc3e3531abf7845395cf3ae2cf9c24b690b5a1 100644 (file)
@@ -61,7 +61,12 @@ export default CustomValuesFacet.extend({
         quietMillis: 300,
         url: this.getUrl(),
         data (term, page) {
-          return { q: term, componentId, p: page, ps: 25 };
+          return {
+            componentId,
+            q: term,
+            p: page,
+            ps: 25
+          };
         },
         results (data) {
           return {
index 99be8cb6dbd6d36e88467fcdc0653faef76c88ea..b0dd063d6ae83bf50cc38cbd18497c6bd993bb77 100644 (file)
@@ -43,8 +43,8 @@ export default CustomValuesFacet.extend({
       },
       width: '100%',
       ajax: {
-        quietMillis: 300,
         url,
+        quietMillis: 300,
         data (term, page) {
           return { q: term, p: page };
         },
index c383ac36624ff4fefed6b18d690050f0f2b6ed69..e537955cebcc72feaa336b9b7f9a4eaf61cf25ce 100644 (file)
@@ -40,8 +40,8 @@ export default CustomValuesFacet.extend({
       },
       width: '100%',
       ajax: {
-        quietMillis: 300,
         url,
+        quietMillis: 300,
         data (term) {
           return { q: term, ps: 10 };
         },
index 6492108fa921d10aa348b9f0f8f4b660378851d3..a0979df4f13f901f507ada37e51a23f3bb3802c9 100644 (file)
@@ -31,6 +31,7 @@ import MetricFilterView from '../../components/navigator/filters/metric-filters'
 import { translate } from '../../helpers/l10n';
 
 const NavigatorApp = new Marionette.Application();
+
 const newLastAnalysisFilter = function () {
   return new BaseFilters.Filter({
     name: translate('measure_filter.criteria.last_analysis'),
@@ -42,10 +43,11 @@ const newLastAnalysisFilter = function () {
     optional: true
   });
 };
+
 const newMetricFilter = function (property) {
   return new BaseFilters.Filter({
-    name: translate('measure_filter.criteria.metric'),
     property,
+    name: translate('measure_filter.criteria.metric'),
     type: MetricFilterView,
     metrics: window.SS.metrics,
     periods: window.SS.metricPeriods,
@@ -54,6 +56,7 @@ const newMetricFilter = function (property) {
     optional: true
   });
 };
+
 const newNameFilter = function () {
   return new BaseFilters.Filter({
     name: translate('measure_filter.name_contains'),
@@ -63,6 +66,7 @@ const newNameFilter = function () {
     optional: true
   });
 };
+
 const newAlertFilter = function () {
   return new BaseFilters.Filter({
     name: translate('measure_filter.criteria.alert'),
@@ -77,6 +81,7 @@ const newAlertFilter = function () {
     }
   });
 };
+
 const init = function () {
   NavigatorApp.addRegions({ filtersRegion: '.navigator-filters' });
 
index 9e2d42ececde030efc6e63117f3f95000b725198..318dabbb294ec3c501e169a44265761d88c1f0ce 100644 (file)
@@ -30,6 +30,7 @@ export default Modal.extend({
     const searchUrl = '/api/permissions/template_users?ps=100&permission=' + this.options.permission.key +
         '&templateId=' + this.options.permissionTemplate.id;
     new window.SelectList({
+      searchUrl,
       el: this.$('#permission-templates-users'),
       width: '100%',
       readOnly: false,
@@ -38,7 +39,6 @@ export default Modal.extend({
         return item.name + '<br><span class="note">' + item.login + '</span>';
       },
       queryParam: 'q',
-      searchUrl,
       selectUrl: '/api/permissions/add_user_to_template',
       deselectUrl: '/api/permissions/remove_user_from_template',
       extra: {
index 691da91d8d478eea0ea88a1598099b41e36b502c..07fc18ad858450751e3fe861cdccb2f46203fded 100644 (file)
@@ -30,6 +30,7 @@ export default Modal.extend({
     const searchUrl = '/api/permissions/users?ps=100&permission=' + this.options.permission +
         '&projectId=' + this.options.project;
     new window.SelectList({
+      searchUrl,
       el: this.$('#project-permissions-users'),
       width: '100%',
       readOnly: false,
@@ -38,7 +39,6 @@ export default Modal.extend({
         return item.name + '<br><span class="note">' + item.login + '</span>';
       },
       queryParam: 'q',
-      searchUrl,
       selectUrl: '/api/permissions/add_user',
       deselectUrl: '/api/permissions/remove_user',
       extra: {
index 183877eba76f1367a98cfacb061b9e1df7d8fbf3..7dab90e61aefc5163f718934f14d4b628e307a95 100644 (file)
@@ -49,8 +49,8 @@ export default ModalFormView.extend({
       parentKey: parent
     };
     return $.ajax({
-      type: 'POST',
       url,
+      type: 'POST',
       data: options,
       statusCode: {
         // do not show global error
index ad5950a776471c14a483df212bc1b3f05fd30571..1ecb6c92aeaf3f4879ab444e4863fc909c503062 100644 (file)
@@ -40,8 +40,8 @@ export default ModalFormView.extend({
       toName: name
     };
     return $.ajax({
-      type: 'POST',
       url,
+      type: 'POST',
       data: options,
       statusCode: {
         // do not show global error
index 79d24d47ced86c9e8cb65b4e179a26f9a9e0de0c..bc255295038fff48ebced7fe3f8494966f638c51 100644 (file)
@@ -39,8 +39,8 @@ export default ModalFormView.extend({
     const url = '/api/qualityprofiles/delete';
     const options = { profileKey: this.model.get('key') };
     return $.ajax({
-      type: 'POST',
       url,
+      type: 'POST',
       data: options,
       statusCode: {
         // do not show global error
index 2115422bf3843e8e5091bf9df5545271cf266100..97cd9797c6d000757863e0ac2eaf7d6d4c98bdca 100644 (file)
@@ -52,8 +52,8 @@ export default ModalFormView.extend({
     const options = { language: lang };
     this.selectedLanguage = _.findWhere(this.options.languages, { key: lang }).name;
     return $.ajax({
-      type: 'POST',
       url,
+      type: 'POST',
       data: options,
       statusCode: {
         // do not show global error
index 6c668832d43d1f999c6c42c7edda833cddc808d7..b60aeb879c0ae9c7e484f9001ac4078a188c9a9d 100644 (file)
@@ -337,9 +337,9 @@ const SelectListView = Backbone.View.extend({
       data[this.settings.queryParam] = query;
       data.selected = 'all';
       this.collection.fetch({
+        data,
         url: this.settings.searchUrl,
         reset: true,
-        data,
         success () {
           that.hideFetchSpinner();
         },
index 845332483237411774befa903a8161a824a96cf9..c013b8e3c538054e31e9ee131f821ec8236f168c 100644 (file)
@@ -152,9 +152,9 @@ export default Marionette.LayoutView.extend({
     const url = '/api/components/app';
     const data = { uuid: this.model.id };
     return $.ajax({
-      type: 'GET',
       url,
       data,
+      type: 'GET',
       statusCode: {
         404 () {
           that.model.set({ exist: false });
@@ -425,8 +425,8 @@ export default Marionette.LayoutView.extend({
     };
     return $.get(url, options).done(function (data) {
       const popup = new CoveragePopupView({
-        collection: new Backbone.Collection(data.tests),
         row,
+        collection: new Backbone.Collection(data.tests),
         tests: $(e.currentTarget).data('tests'),
         triggerEl: $(e.currentTarget)
       });
@@ -474,9 +474,9 @@ export default Marionette.LayoutView.extend({
       return isOk;
     });
     const popup = new DuplicationPopupView({
+      inRemovedComponent,
       triggerEl: $(e.currentTarget),
       model: this.model,
-      inRemovedComponent,
       collection: new Backbone.Collection(blocks)
     });
     popup.render();
@@ -503,9 +503,9 @@ export default Marionette.LayoutView.extend({
     const that = this;
     const line = $(e.currentTarget).data('line-number');
     const popup = new LineActionsPopupView({
+      line,
       triggerEl: $(e.currentTarget),
       model: this.model,
-      line,
       row: $(e.currentTarget).closest('.source-line')
     });
     popup.on('onManualIssueAdded', function (issue) {
index 5cb86419f1520193768c931b3249c5242b05ea32..bfcbe8d3126c97881b90b461fbaaa4e86152e986 100644 (file)
@@ -205,9 +205,9 @@ export default ModalView.extend({
       });
       that.model.set({
         issuesFacets,
-        issuesCount: data.total,
         maxCountBySeverity,
-        maxCountByTag
+        maxCountByTag,
+        issuesCount: data.total
       });
     });
   },
index 81d3a175c9a75cc940a5ca11270dd620006c372b..94e8c5d3689248cb138ddad9a71f134c3b55a9a3 100644 (file)
@@ -61,8 +61,8 @@ export default Popup.extend({
     });
     return {
       testFiles,
-      tests: this.options.tests,
-      row
+      row,
+      tests: this.options.tests
     };
   }
 });
index 43a3a29b475426c0d1223154057674cccc8bd8b8..6aa3f7e2c7e511f21e2e4e56de1ed8513ca79986 100644 (file)
@@ -54,8 +54,8 @@ export default Popup.extend({
       return '' + a + b + c;
     });
     return {
-      component: this.model.toJSON(),
       duplications,
+      component: this.model.toJSON(),
       inRemovedComponent: this.options.inRemovedComponent
     };
   }
index ef4375060fb4241d9244df47258408981e2056ee..3382342b98840e2cb95905d4e54611d9ef66a9ec 100644 (file)
@@ -32,17 +32,21 @@ import StatusesTemplate from './templates/widget-issue-filter-statuses.hbs';
 
 
 const FACET_LIMIT = 15;
+
 const defaultComparator = function (item) {
   return -item.count;
 };
+
 const defaultFilter = function (item) {
   const items = this.query[this.property];
   return items == null ||
       (items != null && items.split(',').indexOf(item.val) !== -1);
 };
+
 const defaultLabel = function (item) {
   return item.val;
 };
+
 const defaultLink = function (item, property, query, index, items, mode) {
   const criterion = {};
   criterion[property] = item.val;
@@ -57,6 +61,7 @@ const defaultLink = function (item, property, query, index, items, mode) {
     return '/issues/search#' + getQuery(r);
   }
 };
+
 const byDistributionConf = {
   'severities': {
     template: SeveritiesTemplate,
@@ -300,9 +305,9 @@ export default Marionette.ItemView.extend({
     const facetMode = this.options.displayMode;
     const url = '/api/issues/search';
     const options = _.extend({}, this.query, {
+      facetMode,
       ps: 1,
       facets: this.options.distributionAxis,
-      facetMode,
       additionalFields: '_all'
     });
     if (this.options.componentUuid != null) {