SONAR-5007 Fix display of facets, translations, paging, filters

diff --git a/sonar-server/src/main/coffee/coding-rules/app.coffee b/sonar-server/src/main/coffee/coding-rules/app.coffee
index 100614eb00d83a4b1fdb3ee4e2b221142d0a5940..ff01752e8fe79b9d9cac292a0c83c6b587a49d64 100644 (file)
--- a/sonar-server/src/main/coffee/coding-rules/app.coffee
+++ b/sonar-server/src/main/coffee/coding-rules/app.coffee
@@ -116,7 +116,7 @@ requirejs [
 
   App.fetchList = (firstPage, fromFacets) ->
     query = @getQuery()
-    fetchQuery = _.extend { p: @pageIndex, facets: !fromFacets }, query
+    fetchQuery = _.extend { p: @pageIndex, ps: 25, facets: !fromFacets }, query
 
     if @codingRulesFacetsView
       _.extend fetchQuery, @codingRulesFacetsView.getQuery()
@@ -177,10 +177,7 @@ requirejs [
 
   App.facetLabel = (property, value) ->
     return value unless App.facetPropertyToLabels[property]
-    if App.facetPropertyToLabels[property][value]
-      return App.facetPropertyToLabels[property][value]
-    else
-      return _.findWhere(App.facetPropertyToLabels[property], key: value).name
+    App.facetPropertyToLabels[property](value)
 
 
   App.fetchFirstPage = (fromFacets = false) ->
@@ -300,8 +297,8 @@ requirejs [
       multiple: false
       qualityProfileFilter: @qualityProfileFilter
       choices:
-        'true': t 'coding_rules.filters.activation.active'
-        'false': t 'coding_rules.filters.activation.inactive'
+        true: t 'coding_rules.filters.activation.active'
+        false: t 'coding_rules.filters.activation.inactive'
     @filters.add @activationFilter
 
     @filters.add new BaseFilters.Filter
@@ -369,8 +366,10 @@ requirejs [
     App.characteristics = r.characteristics
 
     App.facetPropertyToLabels =
-      'languages': App.languages
-      'repositories': App.repositories
+      'languages': (value) -> App.languages[value]
+      'repositories': (value) ->
+        repo = _.findWhere(App.repositories, key: value)
+        repo.name + ' - ' + App.languages[repo.language]
 
   # Message bundles
   l10nXHR = window.requestMessages()

diff --git a/sonar-server/src/main/coffee/coding-rules/views/coding-rules-detail-view.coffee b/sonar-server/src/main/coffee/coding-rules/views/coding-rules-detail-view.coffee
index 3eae915bf2cf7c6a4f2b39db64c8014d886230de..ec05d9868cd1c6b2649c606c2f1e4e2bbf6e5d0c 100644 (file)
--- a/sonar-server/src/main/coffee/coding-rules/views/coding-rules-detail-view.coffee
+++ b/sonar-server/src/main/coffee/coding-rules/views/coding-rules-detail-view.coffee
@@ -63,7 +63,7 @@ define [
 
       qualityProfile = @options.app.getQualityProfile()
       if qualityProfile
-        @contextProfile = qualityProfiles.findWhere key: qualityProfile
+        @contextProfile = qualityProfiles.findWhere qProfile: qualityProfile
         unless @contextProfile
           @contextProfile = new Backbone.Model
             key: qualityProfile, name: @options.app.qualityProfileFilter.view.renderValue()
@@ -71,6 +71,7 @@ define [
           app: @options.app
           model: @contextProfile
           rule: @model
+          qualityProfiles: qualityProfiles
         @listenTo @contextProfile, 'destroy', @hideContext
 
       @model.set 'lang', @options.app.languages[@model.get 'lang']
@@ -81,6 +82,7 @@ define [
       @model.set 'createdAt', new Date(@model.get 'createdAt')
 
     onRender: ->
+      @$el.find('.open-modal').modal();
       @qualityProfilesRegion.show @qualityProfilesView
 
       if @options.app.getQualityProfile()

diff --git a/sonar-server/src/main/coffee/coding-rules/views/coding-rules-quality-profile-activation-view.coffee b/sonar-server/src/main/coffee/coding-rules/views/coding-rules-quality-profile-activation-view.coffee
index 49df39f88054b7aa0d728b3c0d790cbe30cd6ac4..b61623b8be847fd842c7f506e80d3e7cebf5477c 100644 (file)
--- a/sonar-server/src/main/coffee/coding-rules/views/coding-rules-quality-profile-activation-view.coffee
+++ b/sonar-server/src/main/coffee/coding-rules/views/coding-rules-quality-profile-activation-view.coffee
@@ -32,6 +32,8 @@ define [
 
       if @model
         profileKey = @model.get('qProfile')
+        unless profileKey
+          profileKey = @model.get('key')
       severity = @ui.qualityProfileSeverity.val()
 
       @$('.modal-foot').html '<i class="spinner"></i>'

diff --git a/sonar-server/src/main/hbs/coding-rules/coding-rules-facets.hbs b/sonar-server/src/main/hbs/coding-rules/coding-rules-facets.hbs
index c3b2a33a4a8a79f6bf46c8cb3d221447eba549ed..d5dc493de9979f4db2323b31b941e5a4179c5b37 100644 (file)
--- a/sonar-server/src/main/hbs/coding-rules/coding-rules-facets.hbs
+++ b/sonar-server/src/main/hbs/coding-rules/coding-rules-facets.hbs
@@ -4,10 +4,12 @@
       <div class="navigator-facets-list-item-name">{{t property_message}}</div>
       <div class="navigator-facets-list-item-options">
         {{#each values}}
+          {{#if count}}
           <a class="navigator-facets-list-item-option" data-key="{{val}}" data-property="{{../property}}">
             <span class="navigator-facets-list-item-option-name">{{text}}</span>
             <span class="navigator-facets-list-item-option-stat">{{count}}</span>
           </a>
+          {{/if}}
         {{/each}}
       </div>
     </div>

diff --git a/sonar-server/src/main/hbs/coding-rules/coding-rules-header.hbs b/sonar-server/src/main/hbs/coding-rules/coding-rules-header.hbs
index 37f1e63824498bc6dcb9b329f764348fae4a3f50..598ded5a8a6541cc38aeee38c8f4b37ac8cc83d5 100644 (file)
--- a/sonar-server/src/main/hbs/coding-rules/coding-rules-header.hbs
+++ b/sonar-server/src/main/hbs/coding-rules/coding-rules-header.hbs
@@ -2,4 +2,4 @@
 
 <div class="navigator-header-actions button-group">
   <button id="coding-rules-new-search">{{t 'coding_rules.new_search'}}</button>
-</div>
\ No newline at end of file
+</div>

diff --git a/sonar-server/src/main/hbs/coding-rules/coding-rules-profile-filter-detail.hbs b/sonar-server/src/main/hbs/coding-rules/coding-rules-profile-filter-detail.hbs
index 9a9cbe330b595b7d471ee5a523cf99fccb218628..7bc3562578ddf38714c106455a9ca82b27d926f2 100644 (file)
--- a/sonar-server/src/main/hbs/coding-rules/coding-rules-profile-filter-detail.hbs
+++ b/sonar-server/src/main/hbs/coding-rules/coding-rules-profile-filter-detail.hbs
@@ -4,7 +4,7 @@
     <span>
       {{text}}
       <br>
-      <span class="subtitle">{{lang}}</span>
+      <span class="subtitle">{{language}}</span>
     </span>
   </label>
 </li>