Added all list and boolean custom data fields to the Time Report. #4077

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

Added all list and boolean custom data fields to the Time Report. #4077
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2951 e93f8b46-1217-0410-a6f0-8f06a7374b81
2e0cbd28 · Eric Davis · 37d401ac · 2e0cbd28 · 2e0cbd28
Commit 2e0cbd28 authored Oct 21, 2009 by Eric Davis
Hide whitespace changes
Inline Side-by-side

Showing with 12 additions and 3 deletions

timelog_controller.rb app/controllers/timelog_controller.rb +8 -1

timelog_controller_test.rb test/functional/timelog_controller_test.rb +4 -2

No files found.
--- a/app/controllers/timelog_controller.rb
+++ b/app/controllers/timelog_controller.rb
@@ -67,7 +67,14 @@ class TimelogController < ApplicationController
                                             :format => cf.field_format,
                                             :label => cf.name}
    end
-    
+
+    # Add list and boolean time entry activity custom fields
+    TimeEntryActivityCustomField.find(:all).select {|cf| %w(list bool).include? cf.field_format }.each do |cf|
+      @available_criterias["cf_#{cf.id}"] = {:sql => "(SELECT c.value FROM #{CustomValue.table_name} c WHERE c.custom_field_id = #{cf.id} AND c.customized_type = 'Enumeration' AND c.customized_id = #{TimeEntry.table_name}.activity_id)",
+                                             :format => cf.field_format,
+                                             :label => cf.name}
+    end
+
    @criterias = params[:criterias] || []
    @criterias = @criterias.select{|criteria| @available_criterias.has_key? criteria}
    @criterias.uniq!

--- a/test/functional/timelog_controller_test.rb
+++ b/test/functional/timelog_controller_test.rb
@@ -238,12 +238,12 @@ class TimelogControllerTest < ActionController::TestCase
  end
  
  def test_report_custom_field_criteria
-    get :report, :project_id => 1, :criterias => ['project', 'cf_1']
+    get :report, :project_id => 1, :criterias => ['project', 'cf_1', 'cf_7']
    assert_response :success
    assert_template 'report'
    assert_not_nil assigns(:total_hours)
    assert_not_nil assigns(:criterias)
-    assert_equal 2, assigns(:criterias).size
+    assert_equal 3, assigns(:criterias).size
    assert_equal "162.90", "%.2f" % assigns(:total_hours)
    # Custom field column
    assert_tag :tag => 'th', :content => 'Database'
@@ -252,6 +252,8 @@ class TimelogControllerTest < ActionController::TestCase
                             :sibling => { :tag => 'td', :attributes => { :class => 'hours' },
                                                         :child => { :tag => 'span', :attributes => { :class => 'hours hours-int' },
                                                                                     :content => '1' }}
+    # Second custom field column
+    assert_tag :tag => 'th', :content => 'Billable'
  end
  
  def test_report_one_criteria_no_result