cafe-grader-web

Commit r364:9bd14bb6e95d public

parent | child
Description:
add option to update PDF file of a problem in /problem/edit/:id (grafted from 15b2554bf365f8639b5e94db4f1d9a277007808a)
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

r364:9bd14bb6e95d - - 3 files changed: 25 inserted, 1 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -68,50 +68,72
68 68 end
69 69 end
70 70
71 71 def edit
72 72 @problem = Problem.find(params[:id])
73 73 @description = @problem.description
74 74 end
75 75
76 76 def update
77 77 @problem = Problem.find(params[:id])
78 78 @description = @problem.description
79 79 if @description == nil and params[:description][:body]!=''
80 80 @description = Description.new(params[:description])
81 81 if !@description.save
82 82 flash[:notice] = 'Error saving description'
83 83 render :action => 'edit' and return
84 84 end
85 85 @problem.description = @description
86 86 elsif @description!=nil
87 87 if !@description.update_attributes(params[:description])
88 88 flash[:notice] = 'Error saving description'
89 89 render :action => 'edit' and return
90 90 end
91 91 end
92 + if params[:file] and params[:file].content_type != 'application/pdf'
93 + flash[:notice] = 'Error: Uploaded file is not PDF'
94 + render :action => 'edit' and return
95 + end
92 96 if @problem.update_attributes(params[:problem])
93 97 flash[:notice] = 'Problem was successfully updated.'
98 + unless params[:file] == nil or params[:file] == ''
99 + flash[:notice] = 'Problem was successfully updated and a new PDF file is uploaded.'
100 + out_dirname = "#{Problem.download_file_basedir}/#{@problem.id}"
101 + if not FileTest.exists? out_dirname
102 + Dir.mkdir out_dirname
103 + end
104 +
105 + out_filename = "#{out_dirname}/#{@problem.name}.pdf"
106 + if FileTest.exists? out_filename
107 + File.delete out_filename
108 + end
109 +
110 + File.open(out_filename,"wb") do |file|
111 + file.write(params[:file].read)
112 + end
113 + @problem.description_filename = "#{@problem.name}.pdf"
114 + @problem.save
115 + end
94 116 redirect_to :action => 'show', :id => @problem
95 117 else
96 118 render :action => 'edit'
97 119 end
98 120 end
99 121
100 122 def destroy
101 123 Problem.find(params[:id]).destroy
102 124 redirect_to :action => 'list'
103 125 end
104 126
105 127 def toggle
106 128 @problem = Problem.find(params[:id])
107 129 @problem.available = !(@problem.available)
108 130 @problem.save
109 131 end
110 132
111 133 def turn_all_off
112 134 Problem.find(:all,
113 135 :conditions => "available = 1").each do |problem|
114 136 problem.available = false
115 137 problem.save
116 138 end
117 139 redirect_to :action => 'list'
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -27,26 +27,28
27 27 <%= check_box :problem, :available %>
28 28
29 29 <label for="problem_test_allowed">Test allowed?</label>
30 30 <%= check_box :problem, :test_allowed %>
31 31
32 32 <label for="problem_output_only">Output only?</label>
33 33 <%= check_box :problem, :output_only %>
34 34 </p>
35 35
36 36 <%= error_messages_for 'description' %>
37 37
38 38 <p><label for="description_body">Description</label><br/>
39 39 <%= text_area :description, :body, :rows => 10, :cols => 80 %></p>
40 40
41 41 <p><label for="description_markdowned">Markdowned?</label>
42 42 <%= select "description",
43 43 "markdowned",
44 44 [['True',true],['False',false]],
45 45 {:selected => (@description) ? @description.markdowned : false }
46 46 %></p>
47 47
48 48 <p><label for="problem_url">URL</label><br/>
49 49 <%= text_field 'problem', 'url' %></p>
50 50
51 + <p>Task PDF <%= file_field_tag 'file' %></p>
52 +
51 53
52 54 <!--[eoform:problem]-->
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -1,9 +1,9
1 1 <h1>Editing problem</h1>
2 2
3 - <%= form_tag :action => 'update', :id => @problem do %>
3 + <%= form_tag({action: 'update', id: @problem},multipart: true) do %>
4 4 <%= render :partial => 'form' %>
5 5 <%= submit_tag 'Edit' %>
6 6 <% end %>
7 7
8 8 <%= link_to 'Show', :action => 'show', :id => @problem %> |
9 9 <%= link_to 'Back', :action => 'list' %>
You need to be logged in to leave comments. Login now