Browse Source

Include inactive activities in time entries filter (#20117).

git-svn-id: http://svn.redmine.org/redmine/trunk@14626 e93f8b46-1217-0410-a6f0-8f06a7374b81
tags/3.2.0
Jean-Philippe Lang 8 years ago
parent
commit
4464467990
2 changed files with 11 additions and 1 deletions
  1. 1
    1
      app/models/time_entry_query.rb
  2. 10
    0
      test/unit/time_entry_query_test.rb

+ 1
- 1
app/models/time_entry_query.rb View File

@@ -76,7 +76,7 @@ class TimeEntryQuery < Query
:type => :list_optional, :values => users_values
) unless users_values.empty?

activities = (project ? project.activities : TimeEntryActivity.shared.active)
activities = (project ? project.activities : TimeEntryActivity.shared)
add_available_filter("activity_id",
:type => :list, :values => activities.map {|a| [a.name, a.id.to_s]}
) unless activities.empty?

+ 10
- 0
test/unit/time_entry_query_test.rb View File

@@ -27,6 +27,16 @@ class TimeEntryQueryTest < ActiveSupport::TestCase
:groups_users,
:enabled_modules

def test_cross_project_activity_filter_should_propose_non_active_activities
activity = TimeEntryActivity.create!(:name => 'Disabled', :active => false)
assert !activity.active?

query = TimeEntryQuery.new(:name => '_')
assert options = query.available_filters['activity_id']
assert values = options[:values]
assert_include ["Disabled", activity.id.to_s], values
end

def test_activity_filter_should_consider_system_and_project_activities
TimeEntry.delete_all
system = TimeEntryActivity.create!(:name => 'Foo')

Loading…
Cancel
Save