Description:
add option to update PDF file of a problem in /problem/edit/:id
(grafted from 15b2554bf365f8639b5e94db4f1d9a277007808a)
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
-
r364:9bd14bb6e95d - - 3 files changed: 25 inserted, 1 deleted
| @@ -86,14 +86,36 | |||
|
|
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 | |
| @@ -45,8 +45,10 | |||
|
|
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]--> |
| @@ -1,9 +1,9 | |||
|
|
1 | 1 | <h1>Editing problem</h1> |
|
|
2 | 2 | |
|
|
3 |
- <%= form_tag |
|
|
|
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