cafe-grader-web Commit - r634:56cf4357ce2a

cafe-grader-web

Commit `r634:56cf4357ce2a` public

parent | child

Description:

- fix pdf loading fail

Commit status:

[Not Reviewed]

References:

java

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r634:56cf4357ce2a - Wed, 25 Jan 2017 07:43:00 - 4 files changed: 11 inserted, 4 deleted

app/controllers/problems_controller.rb ¶ +7 -1

                     end
                     @problem.description = @description
                   elsif @description
                     if !@description.update_attributes(params[:description])
                       flash[:notice] = 'Error saving description'
                       render :action => 'edit' and return
                     end
                   end
                   if params[:file] and params[:file].content_type != 'application/pdf'
                       flash[:notice] = 'Error: Uploaded file is not PDF'
                       render :action => 'edit' and return
                   end
-            -     if @problem.update_attributes(params[:problem])
+            +     if @problem.update_attributes(problem_params)
                     flash[:notice] = 'Problem was successfully updated.'
                     unless params[:file] == nil or params[:file] == ''
                       flash[:notice] = 'Problem was successfully updated and a new PDF file is uploaded.'
                       out_dirname = "#{Problem.download_file_basedir}/#{@problem.id}"
                       if not FileTest.exists? out_dirname
                         Dir.mkdir out_dirname
                       end
                       out_filename = "#{out_dirname}/#{@problem.name}.pdf"
                       if FileTest.exists? out_filename
                         File.delete out_filename
                       end
                   params.keys.each do |k|
                     if k.index('prob-')==0
                       name, id, order = k.split('-')
                       problems << Problem.find(id)
                     end
                   end
                   problems
                 end
                 def get_problems_stat
                 end
+            +   private
+            +
+            +     def problem_params
+            +       params.require(:problem).permit(:name, :full_name, :full_score, :date_added, :available, :test_allowed,:output_only, :url, :description)
+            +     end
+            +
               end

app/controllers/user_admin_controller.rb ¶ +1 -1

                                           :remark => remark})
                       end
                       user.activated = true
                       user.save
                       if added_random_password
                         note << "'#{login}' (+)"
                       else
                         note << login
                       end
                     end
                   end
-            -     flash[:notice] = 'User(s) ' + note.join(', ') +
+            +     flash[:success] = 'User(s) ' + note.join(', ') +
                     ' were successfully created.  ' +
                     '( (+) - created with random passwords.)'
                   redirect_to :action => 'index'
                 end
                 def edit
                   @user = User.find(params[:id])
                 end
                 def update
                   @user = User.find(params[:id])
                   if @user.update_attributes(user_params)

app/views/problems/_form.html.erb ¶ +2 -1

               <%= error_messages_for 'problem' %>
               <!--[form:problem]-->
               <p><label for="problem_name">Name</label><br/>
-            - <%= text_field 'problem', 'name'  %></p>
+            + <%= text_field 'problem', 'name'  %> Do not directly edit the problem name, unless you know what you are doing. If you want to change the name, use the name change button in the problem management menu instead.
+            + </p>
               <p><label for="problem_full_name">Full name</label><br/>
               <%= text_field 'problem', 'full_name'  %></p>
               <p><label for="problem_full_score">Full score</label><br/>
               <%= text_field 'problem', 'full_score'  %></p>
               <p><label for="problem_date_added">Date added</label><br/>
               <%= date_select 'problem', 'date_added'  %></p>
               <%
               # TODO: these should be put in model Problem, but I can't think of

app/views/problems/show.html.erb ¶ +1 -1

               <%= markdown(@problem.description.body) %>
                   <% else %>
               <pre>
               <%= @problem.description.body %>
               </pre>
                   <% end %>
                 <% else %>
                   (not available)
                 <% end %>
               </p>
               <%= link_to 'Edit', :action => 'edit', :id => @problem %> |
-            - <%= link_to 'Back', :action => 'list' %>
+            + <%= link_to 'Back', problems_path %>

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r634:56cf4357ce2a - Wed, 25 Jan 2017 07:43:00 - 4 files changed: 11 inserted, 4 deleted

Sign in to your account

Commit r634:56cf4357ce2a public

Author

r634:56cf4357ce2a - Wed, 25 Jan 2017 07:43:00 - 4 files changed: 11 inserted, 4 deleted

Commit `r634:56cf4357ce2a` public