Patch #2635 ยป 2635-show-notice-on-forum-updates.diff
app/controllers/messages_controller.rb (working copy) | ||
---|---|---|
63 | 63 |
if @message.save |
64 | 64 |
call_hook(:controller_messages_new_after_save, { :params => params, :message => @message}) |
65 | 65 |
render_attachment_warning_if_needed(@message) |
66 |
flash[:notice] = l(:notice_successful_create) |
|
66 | 67 |
redirect_to board_message_path(@board, @message) |
67 | 68 |
end |
68 | 69 |
end |
... | ... | |
80 | 81 |
attachments = Attachment.attach_files(@reply, params[:attachments]) |
81 | 82 |
render_attachment_warning_if_needed(@reply) |
82 | 83 |
end |
84 |
flash[:notice] = l(:notice_successful_update) |
|
83 | 85 |
redirect_to board_message_path(@board, @topic, :r => @reply) |
84 | 86 |
end |
85 | 87 | |
... | ... | |
101 | 103 |
(render_403; return false) unless @message.destroyable_by?(User.current) |
102 | 104 |
r = @message.to_param |
103 | 105 |
@message.destroy |
106 |
flash[:notice] = l(:notice_successful_delete) |
|
104 | 107 |
if @message.parent |
105 | 108 |
redirect_to board_message_path(@board, @message.parent, :r => r) |
106 | 109 |
else |
test/functional/messages_controller_test.rb (working copy) | ||
---|---|---|
129 | 129 |
} |
130 | 130 |
} |
131 | 131 |
end |
132 |
assert_equal I18n.t(:notice_successful_create), flash[:notice] |
|
132 | 133 |
message = Message.find_by_subject('Test created message') |
133 | 134 |
assert_not_nil message |
134 | 135 |
assert_redirected_to "/boards/1/topics/#{message.to_param}" |
... | ... | |
171 | 172 |
} |
172 | 173 |
} |
173 | 174 |
assert_redirected_to '/boards/1/topics/1' |
175 |
assert_equal I18n.t(:notice_successful_update), flash[:notice] |
|
174 | 176 |
message = Message.find(1) |
175 | 177 |
assert_equal 'New subject', message.subject |
176 | 178 |
assert_equal 'New body', message.content |
... | ... | |
189 | 191 |
} |
190 | 192 |
} |
191 | 193 |
assert_redirected_to '/boards/1/topics/1' |
194 |
assert_equal I18n.t(:notice_successful_update), flash[:notice] |
|
192 | 195 |
message = Message.find(1) |
193 | 196 |
assert_equal true, message.sticky? |
194 | 197 |
assert_equal true, message.locked? |
... | ... | |
222 | 225 |
} |
223 | 226 |
reply = Message.order('id DESC').first |
224 | 227 |
assert_redirected_to "/boards/1/topics/1?r=#{reply.id}" |
228 |
assert_equal I18n.t(:notice_successful_update), flash[:notice] |
|
225 | 229 |
assert Message.find_by_subject('Test reply') |
226 | 230 |
end |
227 | 231 | |
... | ... | |
234 | 238 |
} |
235 | 239 |
end |
236 | 240 |
assert_redirected_to '/projects/ecookbook/boards/1' |
241 |
assert_equal I18n.t(:notice_successful_delete), flash[:notice] |
|
237 | 242 |
assert_nil Message.find_by_id(1) |
238 | 243 |
end |
239 | 244 | |
... | ... | |
246 | 251 |
} |
247 | 252 |
end |
248 | 253 |
assert_redirected_to '/boards/1/topics/1?r=2' |
254 |
assert_equal I18n.t(:notice_successful_delete), flash[:notice] |
|
249 | 255 |
assert_nil Message.find_by_id(2) |
250 | 256 |
end |
251 | 257 |