From 7c1d779b0038531d3b51fee54142c18fc8fb5f8a Mon Sep 17 00:00:00 2001
From: David Gageot <david@gageot.net>
Date: Thu, 5 Jul 2012 09:28:21 +0200
Subject: [PATCH] Revert "SONAR-3529 Display properties in same order as
 annotation"

This reverts commit f5ddbab60c57c9eb0d1dac904268f0fdfbf8f8f2.
---
 .../java/org/sonar/server/ui/ViewProxy.java   |  9 ++---
 .../dashboard/_widget_properties.html.erb     | 38 +++++++++++--------
 2 files changed, 26 insertions(+), 21 deletions(-)

diff --git a/sonar-server/src/main/java/org/sonar/server/ui/ViewProxy.java b/sonar-server/src/main/java/org/sonar/server/ui/ViewProxy.java
index 81422af11cb..5945598ca93 100644
--- a/sonar-server/src/main/java/org/sonar/server/ui/ViewProxy.java
+++ b/sonar-server/src/main/java/org/sonar/server/ui/ViewProxy.java
@@ -21,8 +21,7 @@ package org.sonar.server.ui;
 
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.ImmutableSetMultimap;
-import com.google.common.collect.LinkedListMultimap;
-import com.google.common.collect.ListMultimap;
+import com.google.common.collect.LinkedHashMultimap;
 import com.google.common.collect.Maps;
 import com.google.common.collect.SetMultimap;
 import org.apache.commons.lang.ArrayUtils;
@@ -66,7 +65,7 @@ public class ViewProxy<V extends View> implements Comparable<ViewProxy> {
   private String[] defaultForMetrics = {};
   private String description = "";
   private Map<String, WidgetProperty> widgetPropertiesByKey = Maps.newHashMap();
-  private ListMultimap<WidgetPropertySet, WidgetProperty> widgetPropertiesBySet = LinkedListMultimap.create();
+  private SetMultimap<WidgetPropertySet, WidgetProperty> widgetPropertiesBySet = LinkedHashMultimap.create();
   private String[] widgetCategories = {};
   private WidgetLayoutType widgetLayout = WidgetLayoutType.DEFAULT;
   private boolean isDefaultTab = false;
@@ -226,8 +225,8 @@ public class ViewProxy<V extends View> implements Comparable<ViewProxy> {
     return widgetPropertiesByKey.values();
   }
 
-  public ListMultimap<WidgetPropertySet, WidgetProperty> getWidgetPropertiesBySet() {
-    return widgetPropertiesBySet;
+  public SetMultimap<WidgetPropertySet, WidgetProperty> getWidgetPropertiesBySet() {
+    return ImmutableSetMultimap.copyOf(widgetPropertiesBySet);
   }
 
   public WidgetProperty getWidgetProperty(String propertyKey) {
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
index 348a6ab243d..b14797cfcc3 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_widget_properties.html.erb
@@ -13,22 +13,28 @@
       </tr>
     <% end %>
 
-    <% widget.java_definition.getWidgetPropertiesBySet().asMap().each do |set, properties| %>
-      <% if widget.java_definition.getWidgetPropertiesBySet().asMap().size > 1 or !set.key().empty? %>
-        <tr>
-          <td celspan="2"><h3><%= set.key().empty? ? 'other' : set.key() -%></h3></td>
-        </tr>
-      <% end %>
-      <% properties.each do |property_def| %>
-        <tr>
-          <td class="form-key-cell"><%= property_def.key() -%><%= " *" unless property_def.optional() -%></td>
-          <td class="form-val-cell" id="row_<%= property_def.key() -%>">
-            <%= property_value_field(property_def, widget.property_text_value(property_def.key())) -%>
-            <div class="form-val-note">
-              <%= message("widget." + widget.key + ".param." + property_def.key(), :default => property_def.description()) -%>
-            </div>
-          </td>
-        </tr>
+    <% widget.java_definition.getWidgetPropertiesBySet().asMap().sort { |(s1,p1), (s2,p2)| natural_comparison(s1.key, s2.key) }.each do |set,properties| %>
+        <% if widget.java_definition.getWidgetPropertiesBySet().asMap().size > 1 %>
+          <% if set.key().empty? %>
+             <tr>
+              <td celspan="2"><h3>other</h3></td>
+            </tr>
+          <% else %>
+            <tr>
+              <td celspan="2"><h3><%= set.key() -%></h3></td>
+            </tr>
+          <% end %>
+        <% end %>
+        <% properties.sort { |w1, w2| natural_comparison(w1.key, w2.key) }.each do |property_def| %>
+         <tr>
+            <td class="form-key-cell"><%= property_def.key() -%><%= " *" unless property_def.optional() -%></td>
+            <td class="form-val-cell" id="row_<%= property_def.key() -%>">
+                <%= property_value_field(property_def, widget.property_text_value(property_def.key())) -%>
+                <div class="form-val-note">
+                    <%= message("widget." + widget.key + ".param." + property_def.key(), :default => property_def.description()) -%>
+                </div>
+            </td>
+         </tr>
       <% end %>
     <% end %>
 
-- 
2.39.5