diff --git a/app/models/time_entry_query.rb b/app/models/time_entry_query.rb index c8f8eeaeb..c5c2099ad 100644 --- a/app/models/time_entry_query.rb +++ b/app/models/time_entry_query.rb @@ -25,7 +25,7 @@ class TimeEntryQuery < Query QueryColumn.new(:project, :sortable => "#{Project.table_name}.name", :groupable => true), QueryColumn.new(:spent_on, :sortable => ["#{TimeEntry.table_name}.spent_on", "#{TimeEntry.table_name}.created_on"], :default_order => 'desc', :groupable => true), TimestampQueryColumn.new(:created_on, :sortable => "#{TimeEntry.table_name}.created_on", :default_order => 'desc', :groupable => true), - QueryColumn.new(:tweek, :sortable => ["#{TimeEntry.table_name}.spent_on", "#{TimeEntry.table_name}.created_on"], :caption => :label_week), + QueryColumn.new(:tweek, :sortable => ["#{TimeEntry.table_name}.tyear", "#{TimeEntry.table_name}.tweek"], :caption => :label_week), QueryColumn.new(:author, :sortable => lambda {User.fields_for_order_statement}), QueryColumn.new(:user, :sortable => lambda {User.fields_for_order_statement}, :groupable => true), QueryColumn.new(:activity, :sortable => "#{TimeEntryActivity.table_name}.position", :groupable => true), diff --git a/test/functional/timelog_controller_test.rb b/test/functional/timelog_controller_test.rb index 77b2cd66e..875ee189d 100644 --- a/test/functional/timelog_controller_test.rb +++ b/test/functional/timelog_controller_test.rb @@ -1128,6 +1128,31 @@ class TimelogControllerTest < Redmine::ControllerTest assert_equal [t3, t1, t2].map(&:id).map(&:to_s), css_select('input[name="ids[]"]').map {|e| e.attr('value')} end + def test_index_should_sort_by_tweek_and_spent_on + t1 = TimeEntry.generate!(:spent_on => '2012-06-10') # tyear:2012, tweek:23 + t2 = TimeEntry.generate!(:spent_on => '2012-06-11') # tyear:2012, tweek:24 + t3 = TimeEntry.generate!(:spent_on => '2012-06-12') # tyear:2012, tweek:24 + t4 = TimeEntry.generate!(:spent_on => '2013-06-12') # tyear:2013, tweek:24 + + params = { + :project_id => 1, + :f => ['spent_on'], + :op => {'spent_on' => '><'}, + :v => {'spent_on' => ['2012-06-10', '2013-06-12']} + } + + [ + [{:sort => 'tweek,spent_on'}, [t1, t2, t3, t4]], + [{:sort => 'tweek,spent_on:desc'}, [t1, t3, t2, t4]], + [{:sort => 'tweek:desc,spent_on'}, [t4, t2, t3, t1]], + [{:sort => 'tweek:desc,spent_on:desc'}, [t4, t3, t2, t1]], + ].each do |sort_criteria, expect| + get :index, :params => params.dup.merge(sort_criteria) + assert_response :success + assert_equal expect.map(&:id).map(&:to_s), css_select('input[name="ids[]"]').map {|e| e.attr('value')} + end + end + def test_index_with_activity_filter activity = TimeEntryActivity.create!(:name => 'Activity') entry = TimeEntry.generate!(:issue_id => 1, :hours => 4.5, :activity => activity)