Description:
fixed bug: increament of number on old records in submission
git-svn-id: http://theory.cpe.ku.ac.th/grader/web/trunk@98 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
r46:c9ffe114dd77 - - 1 file changed: 3 inserted, 2 deleted
@@ -10,7 +10,7 | |||||
|
10 | validate :must_specify_language |
|
10 | validate :must_specify_language |
|
11 | validate :must_have_valid_problem |
|
11 | validate :must_have_valid_problem |
|
12 |
|
12 | ||
|
13 | - before_save :assign_latest_number |
|
13 | + before_save :assign_latest_number_if_new_recond |
|
14 |
|
14 | ||
|
15 | def self.find_last_by_user_and_problem(user_id, problem_id) |
|
15 | def self.find_last_by_user_and_problem(user_id, problem_id) |
|
16 | last_sub = find(:first, |
|
16 | last_sub = find(:first, |
@@ -111,7 +111,8 | |||||
|
111 | end |
|
111 | end |
|
112 |
|
112 | ||
|
113 | # callbacks |
|
113 | # callbacks |
|
114 | - def assign_latest_number |
|
114 | + def assign_latest_number_if_new_recond |
|
|
115 | + return if !self.new_record? | ||
|
115 | latest = Submission.find_last_by_user_and_problem(self.user_id, self.problem_id) |
|
116 | latest = Submission.find_last_by_user_and_problem(self.user_id, self.problem_id) |
|
116 | self.number = (latest==nil) ? 1 : latest.number + 1; |
|
117 | self.number = (latest==nil) ? 1 : latest.number + 1; |
|
117 | end |
|
118 | end |
You need to be logged in to leave comments.
Login now