Commit 1c7b79b3 authored by maeda's avatar maeda
Browse files

Display notice on forum updates (#2635).

Patch by Go MAEDA.


git-svn-id: https://svn.redmine.org/redmine/trunk@17738 e93f8b46-1217-0410-a6f0-8f06a7374b81
parent 60ff70c6
......@@ -63,6 +63,7 @@ class MessagesController < ApplicationController
if @message.save
call_hook(:controller_messages_new_after_save, { :params => params, :message => @message})
render_attachment_warning_if_needed(@message)
flash[:notice] = l(:notice_successful_create)
redirect_to board_message_path(@board, @message)
end
end
......@@ -80,6 +81,7 @@ class MessagesController < ApplicationController
attachments = Attachment.attach_files(@reply, params[:attachments])
render_attachment_warning_if_needed(@reply)
end
flash[:notice] = l(:notice_successful_update)
redirect_to board_message_path(@board, @topic, :r => @reply)
end
......@@ -101,6 +103,7 @@ class MessagesController < ApplicationController
(render_403; return false) unless @message.destroyable_by?(User.current)
r = @message.to_param
@message.destroy
flash[:notice] = l(:notice_successful_delete)
if @message.parent
redirect_to board_message_path(@board, @message.parent, :r => r)
else
......
......@@ -129,6 +129,7 @@ class MessagesControllerTest < Redmine::ControllerTest
}
}
end
assert_equal I18n.t(:notice_successful_create), flash[:notice]
message = Message.find_by_subject('Test created message')
assert_not_nil message
assert_redirected_to "/boards/1/topics/#{message.to_param}"
......@@ -171,6 +172,7 @@ class MessagesControllerTest < Redmine::ControllerTest
}
}
assert_redirected_to '/boards/1/topics/1'
assert_equal I18n.t(:notice_successful_update), flash[:notice]
message = Message.find(1)
assert_equal 'New subject', message.subject
assert_equal 'New body', message.content
......@@ -189,6 +191,7 @@ class MessagesControllerTest < Redmine::ControllerTest
}
}
assert_redirected_to '/boards/1/topics/1'
assert_equal I18n.t(:notice_successful_update), flash[:notice]
message = Message.find(1)
assert_equal true, message.sticky?
assert_equal true, message.locked?
......@@ -222,6 +225,7 @@ class MessagesControllerTest < Redmine::ControllerTest
}
reply = Message.order('id DESC').first
assert_redirected_to "/boards/1/topics/1?r=#{reply.id}"
assert_equal I18n.t(:notice_successful_update), flash[:notice]
assert Message.find_by_subject('Test reply')
end
......@@ -234,6 +238,7 @@ class MessagesControllerTest < Redmine::ControllerTest
}
end
assert_redirected_to '/projects/ecookbook/boards/1'
assert_equal I18n.t(:notice_successful_delete), flash[:notice]
assert_nil Message.find_by_id(1)
end
......@@ -246,6 +251,7 @@ class MessagesControllerTest < Redmine::ControllerTest
}
end
assert_redirected_to '/boards/1/topics/1?r=2'
assert_equal I18n.t(:notice_successful_delete), flash[:notice]
assert_nil Message.find_by_id(2)
end
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment