cafe-grader-web

Commit r656:970cebee99d6 public

parent | child
Description:
fix destroy for user and problem (grafted from 56e542ecc72e8d85b776f34c64fe90d3ea4ae9e9)
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

r656:970cebee99d6 - - 2 files changed: 5 inserted, 6 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -8,17 +8,16
8
8
9 def index
9 def index
10 @problems = Problem.find(:all, :order => 'date_added DESC')
10 @problems = Problem.find(:all, :order => 'date_added DESC')
11 end
11 end
12
12
13 # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
13 # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
14 - verify :method => :post, :only => [ :destroy,
14 + verify :method => :post, :only => [ :create, :quick_create,
15 - :create, :quick_create,
16 :do_manage,
15 :do_manage,
17 :do_import,
16 :do_import,
18 - ],
17 + ],
19 :redirect_to => { :action => :index }
18 :redirect_to => { :action => :index }
20
19
21 def show
20 def show
22 @problem = Problem.find(params[:id])
21 @problem = Problem.find(params[:id])
23 end
22 end
24
23
@@ -112,14 +111,14
112 else
111 else
113 render :action => 'edit'
112 render :action => 'edit'
114 end
113 end
115 end
114 end
116
115
117 def destroy
116 def destroy
118 - Problem.find(params[:id]).destroy
117 + p = Problem.find(params[:id]).destroy
119 - redirect_to action: :index
118 + redirect_to action: :index
120 end
119 end
121
120
122 def toggle
121 def toggle
123 @problem = Problem.find(params[:id])
122 @problem = Problem.find(params[:id])
124 @problem.update_attributes(available: !(@problem.available) )
123 @problem.update_attributes(available: !(@problem.available) )
125 respond_to do |format|
124 respond_to do |format|
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -4,13 +4,13
4
4
5 include MailHelperMethods
5 include MailHelperMethods
6
6
7 before_filter :admin_authorization
7 before_filter :admin_authorization
8
8
9 # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
9 # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
10 - verify :method => :post, :only => [ :destroy,
10 + verify :method => :post, :only => [
11 :create, :create_from_list,
11 :create, :create_from_list,
12 :update,
12 :update,
13 :manage_contest,
13 :manage_contest,
14 :bulk_mail
14 :bulk_mail
15 ],
15 ],
16 :redirect_to => { :action => :list }
16 :redirect_to => { :action => :list }
You need to be logged in to leave comments. Login now