SONAR-6265 apply feedback

diff --git a/server/sonar-web/src/main/hbs/coding-rules/coding-rules-rule-filter-form.hbs b/server/sonar-web/src/main/hbs/coding-rules/coding-rules-rule-filter-form.hbs
index 1a414836868a94d92e6b725af99c9580df8fb52e..35f896c026440579733fe131e2384d139d9e26e7 100644 (file)
--- a/server/sonar-web/src/main/hbs/coding-rules/coding-rules-rule-filter-form.hbs
+++ b/server/sonar-web/src/main/hbs/coding-rules/coding-rules-rule-filter-form.hbs
@@ -5,6 +5,26 @@
     {{langName}}
   </a>
 
+  {{#if severity}}
+    <a href="#" class="issue-action-option" data-property="severities" data-value="{{severity}}">
+      {{severityHelper severity}}
+    </a>
+  {{/if}}
+
+  {{#if debtChar}}
+    <hr>
+
+    <a href="#" class="issue-action-option" data-property="debt_characteristics" data-value="{{debtChar}}">
+      {{debtCharName}}
+    </a>
+
+    {{#if debtSubChar}}
+      <a href="#" class="issue-action-option" data-property="debt_characteristics" data-value="{{debtSubChar}}">
+        {{debtSubCharName}}
+      </a>
+    {{/if}}
+  {{/if}}
+
   {{#notEmpty tags}}
     <hr>
     {{#each tags}}

diff --git a/server/sonar-web/src/main/hbs/issue/issue.hbs b/server/sonar-web/src/main/hbs/issue/issue.hbs
index 0e63dcbdeed92d6dd5901e59e2a42b16616e3b5c..dc47a49ddb96b329bbf9b3d0f5b5ec104d45461c 100644 (file)
--- a/server/sonar-web/src/main/hbs/issue/issue.hbs
+++ b/server/sonar-web/src/main/hbs/issue/issue.hbs
@@ -35,10 +35,10 @@
           <div class="issue-meta">
             {{#inArray actions "set_severity"}}
               <a class="issue-action issue-action-with-options js-issue-set-severity">
-                <span class="issue-meta-label">{{severity severity}}</span>&nbsp;<i class="icon-dropdown"></i>
+                <span class="issue-meta-label">{{severityHelper severity}}</span>&nbsp;<i class="icon-dropdown"></i>
               </a>
             {{else}}
-              {{severity severity}}
+              {{severityHelper severity}}
             {{/inArray}}
           </div>
 

diff --git a/server/sonar-web/src/main/hbs/source-viewer/measures/_source-viewer-measures-issues.hbs b/server/sonar-web/src/main/hbs/source-viewer/measures/_source-viewer-measures-issues.hbs
index 88a406859da38893ac09a79c66f7a5d4d6aa60ab..29f14e826df806efd51f76b29cfcf06c30e92c05 100644 (file)
--- a/server/sonar-web/src/main/hbs/source-viewer/measures/_source-viewer-measures-issues.hbs
+++ b/server/sonar-web/src/main/hbs/source-viewer/measures/_source-viewer-measures-issues.hbs
@@ -20,7 +20,7 @@
       <div class="measures-list">
         {{#each issuesFacets.severities}}
           <div class="measure measure-one-line measure-with-bar">
-            <span class="measure-name">{{severity val}}</span>
+            <span class="measure-name">{{severityHelper val}}</span>
             <span class="measure-value">{{count}}</span>
             <span class="measure-bar">
               <span class="measure-bar-inner measure-bar-inner-{{val}}"

diff --git a/server/sonar-web/src/main/js/coding-rules/controller.js b/server/sonar-web/src/main/js/coding-rules/controller.js
index d8aa0bb3c62c534487b50e4533e1d559c991a24f..8d37ad5a043ef1b2c22c5d68d30fb5f930ce1664 100644 (file)
--- a/server/sonar-web/src/main/js/coding-rules/controller.js
+++ b/server/sonar-web/src/main/js/coding-rules/controller.js
@@ -27,7 +27,8 @@ define([
 
   return Controller.extend({
     pageSize: 200,
-    ruleFields: ['name', 'lang', 'langName', 'sysTags', 'tags', 'status'],
+    ruleFields: ['name', 'lang', 'langName', 'sysTags', 'tags', 'status', 'severity',
+                 'debtChar', 'debtCharName', 'debtSubChar', 'debtSubCharName'],
 
 
     _searchParameters: function () {

diff --git a/server/sonar-web/src/main/js/common/handlebars-extensions.js b/server/sonar-web/src/main/js/common/handlebars-extensions.js
index 83826c5de35cd33921d89cc5cc320636ab0031f8..2ad4f37af9976e48a367212c93fb95bc2cf88ee2 100644 (file)
--- a/server/sonar-web/src/main/js/common/handlebars-extensions.js
+++ b/server/sonar-web/src/main/js/common/handlebars-extensions.js
@@ -53,7 +53,7 @@
     );
   });
 
-  Handlebars.registerHelper('severity', function (severity) {
+  Handlebars.registerHelper('severityHelper', function (severity) {
     return new Handlebars.SafeString(
         '<i class="icon-severity-' + severity.toLowerCase() + '"></i>&nbsp;' + t('severity', severity)
     );

diff --git a/server/sonar-web/src/test/js/coding-rules-page-spec.js b/server/sonar-web/src/test/js/coding-rules-page-spec.js
index aedc936868647b03abe1a99713a01ae24709b495..37e78197596f54369dfc5e633cd7366becf9b17d 100644 (file)
--- a/server/sonar-web/src/test/js/coding-rules-page-spec.js
+++ b/server/sonar-web/src/test/js/coding-rules-page-spec.js
@@ -93,7 +93,6 @@ casper.test.begin(testName('Filter Similar Rules'), 3, function (test) {
       .start(lib.buildUrl('coding-rules'), function () {
         lib.setDefaultViewport();
 
-
         lib.mockRequestFromFile('/api/rules/app', 'app.json');
         lib.mockRequestFromFile('/api/rules/search', 'search-sql-tag.json', { data: { tags: 'sql' } });
         lib.mockRequestFromFile('/api/rules/search', 'search.json');
@@ -120,7 +119,6 @@ casper.test.begin(testName('Filter Similar Rules'), 3, function (test) {
         test.assertExists('.bubble-popup [data-property="languages"][data-value="java"]');
 
         casper.click('.bubble-popup [data-property="tags"][data-value="sql"]');
-        casper.wait(1000, function () { lib.capture(); });
         casper.waitForSelectorTextChange('#coding-rules-total');
       })