cafe-grader-web

Commit r357:686426417ab0 public

parent | child
Description:
Implement list_all action for message console
Commit status:
[Not Reviewed]
References:
default
Diff options:
Raw Diff | Patch Diff | Download Diff | Ignore whitespace | Increase context
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Side by Side Unified
Expand All Files Collapse All Files Wide Mode Diff
Add another comment
Browse Files

r357:686426417ab0 - - 2 files changed: 8 inserted, 2 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -3,13 +3,13
3 before_filter :authenticate
3 before_filter :authenticate
4
4
5 verify :method => :post, :only => ['create'],
5 verify :method => :post, :only => ['create'],
6 :redirect_to => { :action => 'list' }
6 :redirect_to => { :action => 'list' }
7
7
8 before_filter :admin_authorization, :only => ['console','show',
8 before_filter :admin_authorization, :only => ['console','show',
9 - 'reply','hide']
9 + 'reply','hide','list_all']
10
10
11 def list
11 def list
12 @user = User.find(session[:user_id])
12 @user = User.find(session[:user_id])
13 @messages = Message.find_all_sent_by_user(@user)
13 @messages = Message.find_all_sent_by_user(@user)
14 end
14 end
15
15
@@ -19,12 +19,17
19 end
19 end
20
20
21 def show
21 def show
22 @message = Message.find(params[:id])
22 @message = Message.find(params[:id])
23 end
23 end
24
24
25 + def list_all
26 + @user = User.find(session[:user_id])
27 + @messages = Message.where(receiver_id: nil).order(:created_at)
28 + end
29 +
25 def create
30 def create
26 user = User.find(session[:user_id])
31 user = User.find(session[:user_id])
27 @message = Message.new(params[:message])
32 @message = Message.new(params[:message])
28 @message.sender = user
33 @message.sender = user
29 if !@message.save
34 if !@message.save
30 render :action => 'list' and return
35 render :action => 'list' and return
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -5,7 +5,8
5 %td (user deleted)
5 %td (user deleted)
6 %td= "#{short_message.created_at}"
6 %td= "#{short_message.created_at}"
7 %td=h truncate(short_message.body)
7 %td=h truncate(short_message.body)
8 %td
8 %td
9 - if short_message.sender
9 - if short_message.sender
10 = link_to "[reply]", :action => 'show', :id => short_message.id
10 = link_to "[reply]", :action => 'show', :id => short_message.id
11 - = link_to "[hide]", :action => 'hide', :id => short_message.id
11 + - if params[:action] != 'list_all'
12 + = link_to "[hide]", :action => 'hide', :id => short_message.id
You need to be logged in to leave comments. Login now