diff --git a/app/controllers/trackers_controller.rb b/app/controllers/trackers_controller.rb index 1b7eddcb93ab7dd6c6aa1d8e33461e957e934e6f..8a44ad825db128a10a4c58376e53705b391d1b62 100644 --- a/app/controllers/trackers_controller.rb +++ b/app/controllers/trackers_controller.rb @@ -20,16 +20,11 @@ class TrackersController < ApplicationController before_filter :require_admin - def index - list - render :action => 'list' unless request.xhr? - end - - verify :method => :post, :only => :destroy, :redirect_to => { :action => :list } + verify :method => :post, :only => :destroy, :redirect_to => { :action => :index } - def list + def index @tracker_pages, @trackers = paginate :trackers, :per_page => 10, :order => 'position' - render :action => "list", :layout => false if request.xhr? + render :action => "index", :layout => false if request.xhr? end def new @@ -40,7 +35,7 @@ class TrackersController < ApplicationController @tracker.workflows.copy(copy_from) end flash[:notice] = l(:notice_successful_create) - redirect_to :action => 'list' + redirect_to :action => 'index' return end @trackers = Tracker.find :all, :order => 'position' @@ -51,7 +46,7 @@ class TrackersController < ApplicationController @tracker = Tracker.find(params[:id]) if request.post? and @tracker.update_attributes(params[:tracker]) flash[:notice] = l(:notice_successful_update) - redirect_to :action => 'list' + redirect_to :action => 'index' return end @projects = Project.find(:all) @@ -64,6 +59,6 @@ class TrackersController < ApplicationController else @tracker.destroy end - redirect_to :action => 'list' + redirect_to :action => 'index' end end diff --git a/app/views/trackers/list.rhtml b/app/views/trackers/index.html.erb similarity index 100% rename from app/views/trackers/list.rhtml rename to app/views/trackers/index.html.erb diff --git a/test/functional/trackers_controller_test.rb b/test/functional/trackers_controller_test.rb index 8d5787cdaf35f9b5e790262ce43428f586fede89..90d398318a9a59d7be84f7991f5c5c56ce44d23c 100644 --- a/test/functional/trackers_controller_test.rb +++ b/test/functional/trackers_controller_test.rb @@ -35,7 +35,7 @@ class TrackersControllerTest < ActionController::TestCase def test_index get :index assert_response :success - assert_template 'list' + assert_template 'index' end def test_get_new @@ -46,7 +46,7 @@ class TrackersControllerTest < ActionController::TestCase def test_post_new post :new, :tracker => { :name => 'New tracker', :project_ids => ['1', '', ''], :custom_field_ids => ['1', '6', ''] } - assert_redirected_to '/trackers/list' + assert_redirected_to :action => 'index' tracker = Tracker.find_by_name('New tracker') assert_equal [1], tracker.project_ids.sort assert_equal [1, 6], tracker.custom_field_ids @@ -55,7 +55,7 @@ class TrackersControllerTest < ActionController::TestCase def test_post_new_with_workflow_copy post :new, :tracker => { :name => 'New tracker' }, :copy_workflow_from => 1 - assert_redirected_to '/trackers/list' + assert_redirected_to :action => 'index' tracker = Tracker.find_by_name('New tracker') assert_equal 0, tracker.projects.count assert_equal Tracker.find(1).workflows.count, tracker.workflows.count @@ -84,14 +84,14 @@ class TrackersControllerTest < ActionController::TestCase def test_post_edit post :edit, :id => 1, :tracker => { :name => 'Renamed', :project_ids => ['1', '2', ''] } - assert_redirected_to '/trackers/list' + assert_redirected_to :action => 'index' assert_equal [1, 2], Tracker.find(1).project_ids.sort end def test_post_edit_without_projects post :edit, :id => 1, :tracker => { :name => 'Renamed', :project_ids => [''] } - assert_redirected_to '/trackers/list' + assert_redirected_to :action => 'index' assert Tracker.find(1).project_ids.empty? end @@ -106,7 +106,7 @@ class TrackersControllerTest < ActionController::TestCase assert_difference 'Tracker.count', -1 do post :destroy, :id => tracker.id end - assert_redirected_to '/trackers/list' + assert_redirected_to :action => 'index' assert_nil flash[:error] end @@ -114,7 +114,7 @@ class TrackersControllerTest < ActionController::TestCase assert_no_difference 'Tracker.count' do post :destroy, :id => 1 end - assert_redirected_to '/trackers/list' + assert_redirected_to :action => 'index' assert_not_nil flash[:error] end end