From ba289e362e961ccfe83dda37cc9becd5b7368091 Mon Sep 17 00:00:00 2001
From: Stas Vilchik <vilchiks@gmail.com>
Date: Wed, 11 Jun 2014 15:52:20 +0600
Subject: [PATCH] SONAR-5209 Duplications feedback

---
 .../component-viewer/duplication-popup.coffee    |  4 ++--
 .../main/coffee/component-viewer/source.coffee   |  6 +++++-
 .../src/main/hbs/component-viewer/source.hbs     | 16 ++++++++--------
 3 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/sonar-server/src/main/coffee/component-viewer/duplication-popup.coffee b/sonar-server/src/main/coffee/component-viewer/duplication-popup.coffee
index 1ee535a16dd..f8cd287028b 100644
--- a/sonar-server/src/main/coffee/component-viewer/duplication-popup.coffee
+++ b/sonar-server/src/main/coffee/component-viewer/duplication-popup.coffee
@@ -37,8 +37,8 @@ define [
 
     serializeData: ->
       files = @options.main.source.get('duplicationFiles')
-      blocks = _.groupBy @collection.toJSON(), '_ref'
-      duplications = _.map blocks, (blocks, fileRef) ->
+      groupedBlocks = _.groupBy @collection.toJSON(), '_ref'
+      duplications = _.map groupedBlocks, (blocks, fileRef) ->
         blocks: blocks
         file: files[fileRef]
       duplications: duplications
diff --git a/sonar-server/src/main/coffee/component-viewer/source.coffee b/sonar-server/src/main/coffee/component-viewer/source.coffee
index 2e1b4d8a677..ffac6cb717d 100644
--- a/sonar-server/src/main/coffee/component-viewer/source.coffee
+++ b/sonar-server/src/main/coffee/component-viewer/source.coffee
@@ -136,10 +136,14 @@ define [
       e.stopPropagation()
       $('body').click()
       index = $(e.currentTarget).data 'index'
+      line = $(e.currentTarget).closest('[data-line-number]').data 'line-number'
+      blocks = @model.get('duplications')[index - 1].blocks
+      blocks = _.filter blocks, (b) ->
+        (b._ref != '1') || (b._ref == '1' && b.from > line) || (b._ref == '1' && b.from + b.size <= line)
       popup = new DuplicationPopupView
         triggerEl: $(e.currentTarget)
         main: @options.main
-        collection: new Backbone.Collection @model.get('duplications')[index - 1].blocks
+        collection: new Backbone.Collection blocks
       popup.render()
 
 
diff --git a/sonar-server/src/main/hbs/component-viewer/source.hbs b/sonar-server/src/main/hbs/component-viewer/source.hbs
index b4f9c67709f..938afe0a446 100644
--- a/sonar-server/src/main/hbs/component-viewer/source.hbs
+++ b/sonar-server/src/main/hbs/component-viewer/source.hbs
@@ -19,14 +19,6 @@
       <tr class="row"
           data-line-number="{{lineNumber}}">
 
-        {{#if ../../settings.duplications}}
-          <td class="stat duplications">
-            {{#each duplications}}
-              <span class="duplication {{#if this}}duplication-exists{{/if}}" data-index="{{this}}"></span>
-            {{/each}}
-          </td>
-        {{/if}}
-
         {{#if ../../settings.scm}}
           <td class="stat {{#if scm}}scm{{/if}}">
             {{#if scm}}
@@ -38,6 +30,14 @@
           </td>
         {{/if}}
 
+        {{#if ../../settings.duplications}}
+          <td class="stat duplications">
+            {{#each duplications}}
+              <span class="duplication {{#if this}}duplication-exists{{/if}}" data-index="{{this}}"></span>
+            {{/each}}
+          </td>
+        {{/if}}
+
         {{#if ../../settings.coverage}}
           <td class="stat {{#if coverage}}coverage-{{#if coverage.covered}}green{{else}}red{{/if}}{{/if}}">
             {{#if coverage}}
-- 
2.39.5