diff options
| author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-06-20 19:24:58 +0000 |
|---|---|---|
| committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-06-20 19:24:58 +0000 |
| commit | 7ed2c481d898ea20db0cec3e9474bf7e43abda21 (patch) | |
| tree | 0e274f8f3a28a9d8e4cb48030bf076bd9a8a3089 | |
| parent | 26e75568cf490a9802825eea2a40fa32f21d4458 (diff) | |
| download | redmine-7ed2c481d898ea20db0cec3e9474bf7e43abda21.tar.gz redmine-7ed2c481d898ea20db0cec3e9474bf7e43abda21.zip | |
Reversed order of priorities on the issue summary page (#11205).
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@9879 e93f8b46-1217-0410-a6f0-8f06a7374b81
| -rw-r--r-- | app/controllers/reports_controller.rb | 4 | ||||
| -rw-r--r-- | test/functional/reports_controller_test.rb | 9 |
2 files changed, 10 insertions, 3 deletions
diff --git a/app/controllers/reports_controller.rb b/app/controllers/reports_controller.rb index 65f01978d..c9e9a39fa 100644 --- a/app/controllers/reports_controller.rb +++ b/app/controllers/reports_controller.rb @@ -22,7 +22,7 @@ class ReportsController < ApplicationController def issue_report @trackers = @project.trackers @versions = @project.shared_versions.sort - @priorities = IssuePriority.all + @priorities = IssuePriority.all.reverse @categories = @project.issue_categories @assignees = (Setting.issue_group_assignment? ? @project.principals : @project.users).sort @authors = @project.users.sort @@ -53,7 +53,7 @@ class ReportsController < ApplicationController @report_title = l(:field_version) when "priority" @field = "priority_id" - @rows = IssuePriority.all + @rows = IssuePriority.all.reverse @data = Issue.by_priority(@project) @report_title = l(:field_priority) when "category" diff --git a/test/functional/reports_controller_test.rb b/test/functional/reports_controller_test.rb index b9954d34d..c44a8a229 100644 --- a/test/functional/reports_controller_test.rb +++ b/test/functional/reports_controller_test.rb @@ -38,9 +38,11 @@ class ReportsControllerTest < ActionController::TestCase assert_template 'issue_report' [:issues_by_tracker, :issues_by_version, :issues_by_category, :issues_by_assigned_to, - :issues_by_author, :issues_by_subproject].each do |ivar| + :issues_by_author, :issues_by_subproject, :issues_by_priority].each do |ivar| assert_not_nil assigns(ivar) end + + assert_equal IssuePriority.all.reverse, assigns(:priorities) end def test_get_issue_report_details @@ -56,6 +58,11 @@ class ReportsControllerTest < ActionController::TestCase end end + def test_get_issue_report_details_by_priority + get :issue_report_details, :id => 1, :detail => 'priority' + assert_equal IssuePriority.all.reverse, assigns(:rows) + end + def test_get_issue_report_details_with_an_invalid_detail get :issue_report_details, :id => 1, :detail => 'invalid' |