Description:
Prevented empty messages from being sent and fixed a typo
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
Waitaya Krongapiradee
-
r377:08f39063736a - - 1 file changed: 7 inserted, 6 deleted
| @@ -22,52 +22,53 | |||
|
|
22 | 22 | @message = Message.find(params[:id]) |
|
|
23 | 23 | end |
|
|
24 | 24 | |
|
|
25 | 25 | def list_all |
|
|
26 | 26 | @user = User.find(session[:user_id]) |
|
|
27 | 27 | @messages = Message.where(receiver_id: nil).order(:created_at) |
|
|
28 | 28 | end |
|
|
29 | 29 | |
|
|
30 | 30 | def create |
|
|
31 | 31 | user = User.find(session[:user_id]) |
|
|
32 | 32 | @message = Message.new(params[:message]) |
|
|
33 | 33 | @message.sender = user |
|
|
34 | - if !@message.save | |
|
|
35 | - render :action => 'list' and return | |
|
|
34 | + if @message.body == '' or !@message.save | |
|
|
35 | + flash[:notice] = 'An error occurred' | |
|
|
36 | 36 | else |
|
|
37 | 37 | flash[:notice] = 'New message posted' |
|
|
38 | - redirect_to :action => 'list' | |
|
|
39 | 38 | end |
|
|
39 | + redirect_to :action => 'list' | |
|
|
40 | 40 | end |
|
|
41 | 41 | |
|
|
42 | 42 | def reply |
|
|
43 | 43 | user = User.find(session[:user_id]) |
|
|
44 | 44 | @message = Message.new(params[:r_message]) |
|
|
45 | 45 | @message.sender = user |
|
|
46 | - if !@message.save | |
|
|
47 | - render :action => 'show' and return | |
|
|
46 | + if @message.body == '' or !@message.save | |
|
|
47 | + flash[:notice] = 'An error occurred' | |
|
|
48 | + redirect_to :action => 'show', :id => @message.replying_message_id | |
|
|
48 | 49 | else |
|
|
49 | 50 | flash[:notice] = 'Message replied' |
|
|
50 | 51 | rep_msg = @message.replying_message |
|
|
51 | 52 | rep_msg.replied = true |
|
|
52 | 53 | rep_msg.save |
|
|
53 | 54 | redirect_to :action => 'console' |
|
|
54 | 55 | end |
|
|
55 | 56 | end |
|
|
56 | 57 | |
|
|
57 | 58 | def hide |
|
|
58 | 59 | message = Message.find(params[:id]) |
|
|
59 | 60 | message.replied = true |
|
|
60 | 61 | message.save |
|
|
61 |
- flash[:notice] = 'Message hid |
|
|
|
62 | + flash[:notice] = 'Message hidden (just marked replied)' | |
|
|
62 | 63 | redirect_to :action => 'console' |
|
|
63 | 64 | end |
|
|
64 | 65 | |
|
|
65 | 66 | protected |
|
|
66 | 67 | def build_replying_message_hierarchy(user) |
|
|
67 | 68 | @all_messages = {} |
|
|
68 | 69 | |
|
|
69 | 70 | |
|
|
70 | 71 | # manually build replies hierarchy (to improve efficiency) |
|
|
71 | 72 | [@messages, @replied_messages].each do |collection| |
|
|
72 | 73 | collection.each do |m| |
|
|
73 | 74 | @all_messages[m.id] = {:msg => m, :replies => []} |
You need to be logged in to leave comments.
Login now