sort custom field issue filter by type and position (#12018)

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10555 e93f8b46-1217-0410-a6f0-8f06a7374b81

3 files changed, 29 insertions, 2 deletions

diff --git a/app/helpers/queries_helper.rb b/app/helpers/queries_helper.rb
index c85e014e0..6ea879a35 100644
--- a/app/helpers/queries_helper.rb
+++ b/app/helpers/queries_helper.rb
@@ -24,7 +24,28 @@ module QueriesHelper
 
   def filters_options(query)
     options = [[]]
-    options += query.available_filters.sort {|a,b| a[1][:order] <=> b[1][:order]}.map do |field, field_options|
+    sorted_options = query.available_filters.sort do |a, b|
+      ord = 0
+      if !(a[1][:order] == 20 && b[1][:order] == 20) 
+        ord = a[1][:order] <=> b[1][:order]
+      else
+        cn = (CustomField::CUSTOM_FIELDS_NAMES.index(a[1][:field].class.name) <=>
+                CustomField::CUSTOM_FIELDS_NAMES.index(b[1][:field].class.name))
+        if cn != 0
+          ord = cn
+        else
+          f = (a[1][:field] <=> b[1][:field])
+          if f != 0
+            ord = f
+          else
+            # assigned_to or author 
+            ord = (a[0] <=> b[0])
+          end
+        end
+      end
+      ord
+    end
+    options += sorted_options.map do |field, field_options|
       [field_options[:name], field]
     end
   end
diff --git a/app/models/custom_field.rb b/app/models/custom_field.rb
index a934cc877..86227a4cf 100644
--- a/app/models/custom_field.rb
+++ b/app/models/custom_field.rb
@@ -51,6 +51,8 @@ class CustomField < ActiveRecord::Base
      :label => DocumentCategory::OptionName}
   ]
 
+  CUSTOM_FIELDS_NAMES = CUSTOM_FIELDS_TABS.collect{|v| v[:name]}
+
   def set_searchable
     # make sure these fields are not searchable
     self.searchable = false if %w(int float date bool).include?(field_format)
diff --git a/app/models/query.rb b/app/models/query.rb
index 12a46c73f..dfb00825d 100644
--- a/app/models/query.rb
+++ b/app/models/query.rb
@@ -993,7 +993,11 @@ class Query < ActiveRecord::Base
         filter_id = "#{assoc}.#{filter_id}"
         filter_name = l("label_attribute_of_#{assoc}", :name => filter_name)
       end
-      @available_filters[filter_id] = options.merge({ :name => filter_name, :format => field.field_format })
+      @available_filters[filter_id] = options.merge({
+               :name => filter_name,
+               :format => field.field_format,
+               :field => field
+             })
     end
   end