Description:
import problem replaced old one, fixed small problems
problem validates name to have no space, test interface can deal with broken dependecies on problems.
git-svn-id: http://theory.cpe.ku.ac.th/grader/web/trunk@436 6386c4cd-e34a-4fa8-8920-d93eb39b512e
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r205:9bf8e38c2911 - - 5 files changed: 15 inserted, 3 deleted
@@ -156,6 +156,14 | |||
|
156 | 156 | render :action => 'import' and return |
|
157 | 157 | end |
|
158 | 158 | |
|
159 | + old_problem = Problem.find_by_name(@problem.name) | |
|
160 | + if old_problem!=nil | |
|
161 | + old_problem.full_name = @problem.full_name | |
|
162 | + @problem = old_problem | |
|
163 | + | |
|
164 | + flash[:notice] = "The test data has been replaced for problem #{@problem.name}" | |
|
165 | + end | |
|
166 | + | |
|
159 | 167 | @problem.save |
|
160 | 168 | @log = import_log |
|
161 | 169 | end |
@@ -101,7 +101,7 | |||
|
101 | 101 | end |
|
102 | 102 | @test_requests = [] |
|
103 | 103 | @user.test_requests.each do |ts| |
|
104 | - if ts.problem.available | |
|
104 | + if ts.problem and ts.problem.available | |
|
105 | 105 | @test_requests << ts |
|
106 | 106 | end |
|
107 | 107 | end |
@@ -3,8 +3,9 | |||
|
3 | 3 | belongs_to :description |
|
4 | 4 | |
|
5 | 5 | validates_presence_of :name |
|
6 | + validates_format_of :name, :with => /^\w+$/ | |
|
6 | 7 | validates_presence_of :full_name |
|
7 | - | |
|
8 | + | |
|
8 | 9 | def self.find_available_problems |
|
9 | 10 | find(:all, :conditions => {:available => true}, :order => "date_added DESC") |
|
10 | 11 | end |
You need to be logged in to leave comments.
Login now