Description:
problems sorted by update time
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Author
Jittat Fakcharoenphol
-
r258:1821e2e3aeb0 - - 4 files changed: 25 inserted, 13 deleted
| @@ -0,0 +1,9 | |||||
|
|
|
1 | + class AddTimestampToProblems < ActiveRecord::Migration | ||
|
|
|
2 | + def self.up | ||
|
|
|
3 | + add_column :problems, :updated_at, :timestamp | ||
|
|
|
4 | + end | ||
|
|
|
5 | + | ||
|
|
|
6 | + def self.down | ||
|
|
|
7 | + remove_column :problems, :updated_at | ||
|
|
|
8 | + end | ||
|
|
|
9 | + end | ||
| @@ -34,13 +34,15 | |||||
|
|
34 | test_num = 1 + rand(test_pair_count) |
|
34 | test_num = 1 + rand(test_pair_count) |
|
|
35 | end |
|
35 | end |
|
|
36 | test_pairs.find_by_number test_num |
|
36 | test_pairs.find_by_number test_num |
|
|
37 | end |
|
37 | end |
|
|
38 |
|
38 | ||
|
|
39 | def self.find_available_problems |
|
39 | def self.find_available_problems |
|
|
40 | - find(:all, :conditions => {:available => true}, :order => "date_added DESC") |
|
40 | + find(:all, |
|
|
|
41 | + :conditions => {:available => true}, | ||
|
|
|
42 | + :order => "updated_at DESC") | ||
|
|
41 | end |
|
43 | end |
|
|
42 |
|
44 | ||
|
|
43 | # TODO: may try to optimize this using cache |
|
45 | # TODO: may try to optimize this using cache |
|
|
44 | def self.available_problem_count |
|
46 | def self.available_problem_count |
|
|
45 | return Problem.find_available_problems.length |
|
47 | return Problem.find_available_problems.length |
|
|
46 | end |
|
48 | end |
| @@ -34,9 +34,9 | |||||
|
|
34 | = "Announcement.refreshUrl = '#{url_for :controller => 'main', :action => 'announcements'}';" |
|
34 | = "Announcement.refreshUrl = '#{url_for :controller => 'main', :action => 'announcements'}';" |
|
|
35 | Announcement.registerRefreshEventTimer(); |
|
35 | Announcement.registerRefreshEventTimer(); |
|
|
36 | = render :partial => 'submission_timeouts' |
|
36 | = render :partial => 'submission_timeouts' |
|
|
37 | CodejomTimeout.updateProblemMessages(); |
|
37 | CodejomTimeout.updateProblemMessages(); |
|
|
38 | CodejomTimeout.registerRefreshEvent(); |
|
38 | CodejomTimeout.registerRefreshEvent(); |
|
|
39 |
|
39 | ||
|
|
40 | - = periodically_call_remote(:url => { :action => 'problems' }, :update => 'problem-list') |
|
40 | + = periodically_call_remote(:url => { :action => 'problems' }, :update => 'problem-list', :frequency => '3') |
|
|
41 |
|
41 | ||
|
|
42 |
|
42 | ||
| @@ -6,13 +6,13 | |||||
|
|
6 | # to create the application database on another system, you should be using db:schema:load, not running |
|
6 | # to create the application database on another system, you should be using db:schema:load, not running |
|
|
7 | # all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
7 | # all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
|
8 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
8 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
|
9 | # |
|
9 | # |
|
|
10 | # It's strongly recommended to check this file into your version control system. |
|
10 | # It's strongly recommended to check this file into your version control system. |
|
|
11 |
|
11 | ||
|
|
12 |
- ActiveRecord::Schema.define(:version => 201002 |
|
12 | + ActiveRecord::Schema.define(:version => 20100210012432) do |
|
|
13 |
|
13 | ||
|
|
14 | create_table "announcements", :force => true do |t| |
|
14 | create_table "announcements", :force => true do |t| |
|
|
15 | t.string "author" |
|
15 | t.string "author" |
|
|
16 | t.text "body" |
|
16 | t.text "body" |
|
|
17 | t.boolean "published" |
|
17 | t.boolean "published" |
|
|
18 | t.datetime "created_at" |
|
18 | t.datetime "created_at" |
| @@ -81,22 +81,23 | |||||
|
|
81 | t.boolean "replied" |
|
81 | t.boolean "replied" |
|
|
82 | t.datetime "created_at" |
|
82 | t.datetime "created_at" |
|
|
83 | t.datetime "updated_at" |
|
83 | t.datetime "updated_at" |
|
|
84 | end |
|
84 | end |
|
|
85 |
|
85 | ||
|
|
86 | create_table "problems", :force => true do |t| |
|
86 | create_table "problems", :force => true do |t| |
|
|
87 | - t.string "name", :limit => 30 |
|
87 | + t.string "name", :limit => 30 |
|
|
88 | - t.string "full_name" |
|
88 | + t.string "full_name" |
|
|
89 | - t.integer "full_score" |
|
89 | + t.integer "full_score" |
|
|
90 | - t.date "date_added" |
|
90 | + t.date "date_added" |
|
|
91 | - t.boolean "available" |
|
91 | + t.boolean "available" |
|
|
92 | - t.string "url" |
|
92 | + t.string "url" |
|
|
93 | - t.integer "description_id" |
|
93 | + t.integer "description_id" |
|
|
94 | - t.boolean "test_allowed" |
|
94 | + t.boolean "test_allowed" |
|
|
95 | - t.boolean "output_only" |
|
95 | + t.boolean "output_only" |
|
|
96 | - t.integer "level", :default => 0 |
|
96 | + t.integer "level", :default => 0 |
|
|
|
97 | + t.datetime "updated_at" | ||
|
|
97 | end |
|
98 | end |
|
|
98 |
|
99 | ||
|
|
99 | create_table "rights", :force => true do |t| |
|
100 | create_table "rights", :force => true do |t| |
|
|
100 | t.string "name" |
|
101 | t.string "name" |
|
|
101 | t.string "controller" |
|
102 | t.string "controller" |
|
|
102 | t.string "action" |
|
103 | t.string "action" |
You need to be logged in to leave comments.
Login now