cafe-grader-web

Commit r232:f00dec11b375 public

parent | child
Description:
fixed timeout bug for wrong submission (thanks to @dtinth), and js bugs occurring in IE
Commit status:
[Not Reviewed]
References:
default
Diff options:
Raw Diff | Patch Diff | Download Diff | Ignore whitespace | Increase context
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Side by Side Unified
Expand All Files Collapse All Files Wide Mode Diff
Add another comment
Browse Files

r232:f00dec11b375 - - 2 files changed: 6 inserted, 2 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -279,7 +279,7
279 279 if assignment == nil
280 280 timeout = nil
281 281 else
282 - if assignment.expired?
282 + if (assignment.expired?) or (assignment.submitted)
283 283 timeout = 0
284 284 else
285 285 timeout = assignment.created_at + TEST_ASSIGNMENT_EXPIRATION_DURATION - Time.new.gmtime
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -8,6 +8,8
8 8
9 9 updateProblemMessages: function() {
10 10 CodejomTimeout.timeouts.each(function(data) {
11 + if(data==null)
12 + return;
11 13 if(data.timeout==null) {
12 14 $("problem-submission-form-" + data.problem).hide();
13 15 } else if(data.timeout==0) {
@@ -26,6 +28,8
26 28 var timeElapsed = ((new Date()).getTime() - CodejomTimeout.timeStarted)/1000;
27 29 // update timeout info
28 30 CodejomTimeout.timeouts.each(function(data) {
31 + if(data==null)
32 + return;
29 33 if(data.timeout > timeElapsed) {
30 34 data.timeout -= timeElapsed;
31 35 } else if(data.timeout > 0) {
@@ -46,7 +50,7
46 50
47 51 updateTimeoutAfterDownloadClick: function(problem) {
48 52 CodejomTimeout.timeouts
49 - .filter(function(data) { return data.problem==problem; })
53 + .filter(function(data) { return (data!=null) && (data.problem==problem); })
50 54 .each(function(data) {
51 55 if(data.timeout==0 || data.timeout==null) {
52 56 // TODO: use value from rails app.
You need to be logged in to leave comments. Login now