cafe-grader-web Commit - r271:69482b83f9d6

cafe-grader-web

Commit `r271:69482b83f9d6` public

parent | child

Description:

imports task description as pdf

Commit status:

[Not Reviewed]

References:

default

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r271:69482b83f9d6 - Fri, 19 Feb 2010 02:17:28 - 8 files changed: 62 inserted, 9 deleted

data/tasks/.gitignore created 644 ¶ +1

	@@ -0,0 +1,1
				1	+ [a-zA-Z]*

db/migrate/20100219014840_add_description_filename_to_problems.rb created 644 ¶ +9

@@ -0,0 +1,9
	1	+ class AddDescriptionFilenameToProblems < ActiveRecord::Migration
	2	+ def self.up
	3	+ add_column :problems, :description_filename, :string
	4	+ end
	5	+
	6	+ def self.down
	7	+ remove_column :problems, :description_filename
	8	+ end
	9	+ end

app/controllers/tasks_controller.rb ¶ +17 -7

                 end
                 def view
-            -     base_filename = File.basename("#{params[:file]}.#{params[:ext]}")
+            +     base_name = params[:file]
-            -     filename = "#{RAILS_ROOT}/data/tasks/#{base_filename}"
+            +     if !check_user_viewability(base_name)
-            -     #filename = "/home/ioi/web_grader/data/tasks/#{base_filename}"
+            +       redirect_to :action => 'index' and return
-            -     #filename = "/home/ioi/web_grader/public/images/rails.png"
+            +     end
-            -     if !FileTest.exists?(filename)
+            +     base_filename = File.basename("#{base_name}.#{params[:ext]}")
+            +     filename = "#{Problem.download_file_basedir}/#{base_filename}"
+            +
+            +     if !check_user_viewability(base_name) or !FileTest.exists?(filename)
                     redirect_to :action => 'index' and return
                   end
                 protected
                 def check_viewability
-            -     user = User.find(session[:user_id])
+            +     @user = User.find(session[:user_id])
-            -     if user==nil or !Configuration.show_tasks_to?(user)
+            +     if @user==nil or !Configuration.show_tasks_to?(@user)
                     redirect_to :controller => 'main', :action => 'list'
                     return false
                   end
                 end
+            +   def check_user_viewability(filename)
+            +     # individual file access control shall be added here
+            +     return false if not @user
+            +     return Configuration.show_tasks_to?(@user)
                 end
+            +
+            + end

app/helpers/main_helper.rb ¶ +15

@@ -1,3 +1,18
1	module MainHelper	1	module MainHelper
2		2
		3	+ def link_to_description_if_any(name, problem, options={})
		4	+ if !problem.url.blank?
		5	+ return link_to name, problem.url, options
		6	+ elsif !problem.description_filename.blank?
		7	+ basename, ext = problem.description_filename.split('.')
		8	+ options[:controller] = 'tasks'
		9	+ options[:action] = 'view'
		10	+ options[:file] = basename
		11	+ options[:ext] = ext
		12	+ return link_to name, options
		13	+ else
		14	+ return ''
3	end	15	end
		16	+ end
		17	+
		18	+ end

app/models/problem.rb ¶ +4

                   return problem, importer.log_msg
                 end
+            +   def self.download_file_basedir
+            +     return "#{RAILS_ROOT}/data/tasks"
+            +   end
+            +
                 protected
                 def self.to_i_or_default(st, default)

app/views/main/_problem.html.erb ¶ +1 -1

                 </td>
                 <td>
                   <%= "#{problem.full_name} (#{problem.name})" %>
-            -     <%= link_to "[#{t 'main.problem_desc'}]", problem.url, :popup => true if (problem.url!=nil) and (problem.url!='') %>
+            +     <%= link_to_description_if_any "[#{t 'main.problem_desc'}]", problem %>
                 </td>
                 <td align="center">
                   <%= @prob_submissions[problem_counter][:count] %>

db/schema.rb ¶ +2 -1

               #
               # It's strongly recommended to check this file into your version control system.
-            - ActiveRecord::Schema.define(:version => 20100216162940) do
+            + ActiveRecord::Schema.define(:version => 20100219014840) do
                 create_table "announcements", :force => true do |t|
                   t.string   "author"
                   t.boolean  "output_only"
                   t.integer  "level",                              :default => 0
                   t.datetime "updated_at"
+            +     t.string   "description_filename"
                 end
                 create_table "rights", :force => true do |t|

lib/testdata_importer.rb ¶ +13

                   end
                   @log_msg << import_problem_description(dirname)
+            +     @log_msg << import_problem_pdf(dirname)
                   return true
                 end
                   end
                 end
+            +   def import_problem_pdf(dirname)
+            +     pdf_files = Dir["#{dirname}/*.pdf"]
+            +     if pdf_files.length != 0
+            +       filename = pdf_files[0]
+            +       out_filename = "#{Problem.download_file_basedir}/#{@problem.name}.pdf"
+            +       File.rename(filename, out_filename)
+            +       @problem.description_filename = "#{@problem.name}.pdf"
+            +       @problem.save
+            +       return "\nProblem pdf imported from #{filename}."
                   end
+            +   end
+            +
+            + end

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

r271:69482b83f9d6 - Fri, 19 Feb 2010 02:17:28 - 8 files changed: 62 inserted, 9 deleted

Sign in to your account

Commit r271:69482b83f9d6 public

Author

r271:69482b83f9d6 - Fri, 19 Feb 2010 02:17:28 - 8 files changed: 62 inserted, 9 deleted

Commit `r271:69482b83f9d6` public