diff --git a/app/controllers/documents_controller.rb b/app/controllers/documents_controller.rb index b9e2633cc..101e5443a 100644 --- a/app/controllers/documents_controller.rb +++ b/app/controllers/documents_controller.rb @@ -83,6 +83,7 @@ class DocumentsController < ApplicationController def destroy @document.destroy if request.delete? + flash[:notice] = l(:notice_successful_delete) redirect_to project_documents_path(@project) end diff --git a/app/controllers/issues_controller.rb b/app/controllers/issues_controller.rb index 0d1acc95b..b442b355b 100644 --- a/app/controllers/issues_controller.rb +++ b/app/controllers/issues_controller.rb @@ -448,7 +448,10 @@ class IssuesController < ApplicationController end end respond_to do |format| - format.html {redirect_back_or_default _project_issues_path(@project)} + format.html { + flash[:notice] = l(:notice_successful_delete) + redirect_back_or_default _project_issues_path(@project) + } format.api {render_api_ok} end end diff --git a/app/controllers/news_controller.rb b/app/controllers/news_controller.rb index 511f10ddb..a67ef88c8 100644 --- a/app/controllers/news_controller.rb +++ b/app/controllers/news_controller.rb @@ -115,7 +115,10 @@ class NewsController < ApplicationController def destroy @news.destroy respond_to do |format| - format.html { redirect_to project_news_index_path(@project) } + format.html { + flash[:notice] = l(:notice_successful_delete) + redirect_to project_news_index_path(@project) + } format.api { render_api_ok } end end diff --git a/app/controllers/wiki_controller.rb b/app/controllers/wiki_controller.rb index 527f0b821..fdb523cb6 100644 --- a/app/controllers/wiki_controller.rb +++ b/app/controllers/wiki_controller.rb @@ -284,7 +284,10 @@ class WikiController < ApplicationController end @page.destroy respond_to do |format| - format.html { redirect_to project_wiki_index_path(@project) } + format.html { + flash[:notice] = l(:notice_successful_delete) + redirect_to project_wiki_index_path(@project) + } format.api { render_api_ok } end end diff --git a/test/functional/documents_controller_test.rb b/test/functional/documents_controller_test.rb index f4d218cc0..078b95c0f 100644 --- a/test/functional/documents_controller_test.rb +++ b/test/functional/documents_controller_test.rb @@ -240,6 +240,7 @@ class DocumentsControllerTest < Redmine::ControllerTest } end assert_redirected_to '/projects/ecookbook/documents' + assert_equal 'Successful deletion.', flash[:notice] assert_nil Document.find_by_id(1) end diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb index cb8a6703b..9f36c7177 100644 --- a/test/functional/issues_controller_test.rb +++ b/test/functional/issues_controller_test.rb @@ -7491,6 +7491,7 @@ class IssuesControllerTest < Redmine::ControllerTest delete(:destroy, :params => {:id => 2}) end assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] assert_nil Issue.find_by_id(2) end @@ -7570,6 +7571,7 @@ class IssuesControllerTest < Redmine::ControllerTest end end assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] assert !(Issue.find_by_id(1) || Issue.find_by_id(3)) assert_nil TimeEntry.find_by_id([1, 2]) end @@ -7591,6 +7593,7 @@ class IssuesControllerTest < Redmine::ControllerTest end end assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] assert !(Issue.find_by_id(1) || Issue.find_by_id(3)) assert_nil TimeEntry.find(1).issue_id assert_nil TimeEntry.find(2).issue_id @@ -7612,6 +7615,7 @@ class IssuesControllerTest < Redmine::ControllerTest end end assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] assert !(Issue.find_by_id(1) || Issue.find_by_id(3)) assert_equal 2, TimeEntry.find(1).issue_id assert_equal 2, TimeEntry.find(2).issue_id @@ -7636,6 +7640,7 @@ class IssuesControllerTest < Redmine::ControllerTest } ) assert_response 302 + assert_equal 'Successful deletion.', flash[:notice] end end assert_equal 3, target.time_entries.count @@ -7712,6 +7717,7 @@ class IssuesControllerTest < Redmine::ControllerTest ) end assert_redirected_to :controller => 'issues', :action => 'index' + assert_equal 'Successful deletion.', flash[:notice] assert !(Issue.find_by_id(1) || Issue.find_by_id(2) || Issue.find_by_id(6)) end @@ -7732,6 +7738,7 @@ class IssuesControllerTest < Redmine::ControllerTest ) end assert_response 302 + assert_equal 'Successful deletion.', flash[:notice] end def test_destroy_invalid_should_respond_with_404 @@ -7752,6 +7759,7 @@ class IssuesControllerTest < Redmine::ControllerTest delete(:destroy, :params => {:id => issue.id}) end assert_response 302 + assert_equal 'Successful deletion.', flash[:notice] end def test_destroy_without_permission_on_tracker_should_be_denied diff --git a/test/functional/news_controller_test.rb b/test/functional/news_controller_test.rb index c870272ef..76517e1a1 100644 --- a/test/functional/news_controller_test.rb +++ b/test/functional/news_controller_test.rb @@ -243,6 +243,7 @@ class NewsControllerTest < Redmine::ControllerTest @request.session[:user_id] = 2 delete(:destroy, :params => {:id => 1}) assert_redirected_to '/projects/ecookbook/news' + assert_equal 'Successful deletion.', flash[:notice] assert_nil News.find_by_id(1) end end diff --git a/test/functional/wiki_controller_test.rb b/test/functional/wiki_controller_test.rb index d5dfe1247..b75bdaae8 100644 --- a/test/functional/wiki_controller_test.rb +++ b/test/functional/wiki_controller_test.rb @@ -914,6 +914,7 @@ class WikiControllerTest < Redmine::ControllerTest @request.session[:user_id] = 2 delete :destroy, :params => {:project_id => 1, :id => 'Child_2'} assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] end def test_destroy_parent_should_ask_confirmation @@ -935,6 +936,7 @@ class WikiControllerTest < Redmine::ControllerTest delete :destroy, :params => {:project_id => 1, :id => 'Another_page', :todo => 'nullify'} end assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] assert_nil WikiPage.find_by_id(2) end @@ -944,6 +946,7 @@ class WikiControllerTest < Redmine::ControllerTest delete :destroy, :params => {:project_id => 1, :id => 'Another_page', :todo => 'destroy'} end assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] assert_nil WikiPage.find_by_id(2) assert_nil WikiPage.find_by_id(5) end @@ -954,6 +957,7 @@ class WikiControllerTest < Redmine::ControllerTest delete :destroy, :params => {:project_id => 1, :id => 'Another_page', :todo => 'reassign', :reassign_to_id => 1} end assert_redirected_to :action => 'index', :project_id => 'ecookbook' + assert_equal 'Successful deletion.', flash[:notice] assert_nil WikiPage.find_by_id(2) assert_equal WikiPage.find(1), WikiPage.find_by_id(5).parent end