Adds a version filter on time entries (#13558).

git-svn-id: http://svn.redmine.org/redmine/trunk@15646 e93f8b46-1217-0410-a6f0-8f06a7374b81

2 files changed, 26 insertions, 2 deletions

diff --git a/app/models/query.rb b/app/models/query.rb
index 1038637f2..5e8e0cd90 100644
--- a/app/models/query.rb
+++ b/app/models/query.rb
@@ -691,9 +691,9 @@ class Query < ActiveRecord::Base
       if field =~ /cf_(\d+)$/
         # custom field
         filters_clauses << sql_for_custom_field(field, operator, v, $1)
-      elsif respond_to?("sql_for_#{field}_field")
+      elsif respond_to?(method = "sql_for_#{field.gsub('.','_')}_field")
         # specific statement
-        filters_clauses << send("sql_for_#{field}_field", field, operator, v)
+        filters_clauses << send(method, field, operator, v)
       else
         # regular field
         filters_clauses << '(' + sql_for_field(field, operator, v, queried_table_name, field) + ')'
diff --git a/app/models/time_entry_query.rb b/app/models/time_entry_query.rb
index bb042ff16..dde34264a 100644
--- a/app/models/time_entry_query.rb
+++ b/app/models/time_entry_query.rb
@@ -41,6 +41,7 @@ class TimeEntryQuery < Query
     add_available_filter "spent_on", :type => :date_past
 
     principals = []
+    versions = []
     if project
       principals += project.principals.visible.sort
       unless project.leaf?
@@ -52,6 +53,7 @@ class TimeEntryQuery < Query
           principals += Principal.member_of(subprojects).visible
         end
       end
+      versions = project.shared_versions.to_a
     else
       if all_projects.any?
         # members of visible projects
@@ -69,6 +71,10 @@ class TimeEntryQuery < Query
     end
 
     add_available_filter("issue_id", :type => :tree, :label => :label_issue)
+    add_available_filter("issue.fixed_version_id",
+      :type => :list,
+      :name => l("label_attribute_of_issue", :name => l(:field_fixed_version)),
+      :values => Version.sort_by_status(versions).collect{|s| ["#{s.project.name} - #{s.name}", s.id.to_s, l("version_status_#{s.status}")] })
 
     principals.uniq!
     principals.sort!
@@ -136,6 +142,24 @@ class TimeEntryQuery < Query
     end
   end
 
+  def sql_for_issue_fixed_version_id_field(field, operator, value)
+    issue_ids = Issue.where(:fixed_version_id => value.first.to_i).pluck(:id)
+    case operator
+    when "="
+      if issue_ids.any?
+        "#{TimeEntry.table_name}.issue_id IN (#{issue_ids.join(',')})"
+      else
+        "1=0"
+      end
+    when "!"
+      if issue_ids.any?
+        "#{TimeEntry.table_name}.issue_id NOT IN (#{issue_ids.join(',')})"
+      else
+        "1=1"
+      end
+    end
+  end
+
   def sql_for_activity_id_field(field, operator, value)
     condition_on_id = sql_for_field(field, operator, value, Enumeration.table_name, 'id')
     condition_on_parent_id = sql_for_field(field, operator, value, Enumeration.table_name, 'parent_id')