Description:
added level to problems, randoms problem from each level
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r255:b7ca2d765156 - - 9 files changed: 87 inserted, 7 deleted
@@ -0,0 +1,9 | |||
|
1 | + class AddLevelToProblems < ActiveRecord::Migration | |
|
2 | + def self.up | |
|
3 | + add_column :problems, :level, :integer, :default => 0 | |
|
4 | + end | |
|
5 | + | |
|
6 | + def self.down | |
|
7 | + remove_column :problems, :level, :integer | |
|
8 | + end | |
|
9 | + end |
@@ -1,13 +1,37 | |||
|
1 | 1 | class CodejomController < ApplicationController |
|
2 | 2 | |
|
3 | 3 | before_filter :admin_authorization |
|
4 | - before_filter :authenticate | |
|
5 | 4 | |
|
6 | 5 | def index |
|
7 | 6 | @user = User.find(session[:user_id]) |
|
8 | 7 | @problems = Problem.find(:all) |
|
9 | - @available_problems = @problems.find_all {|p| not p.available } | |
|
8 | + @levels = @problems.collect {|p| p.level}.uniq.sort | |
|
9 | + @available_problems = {} | |
|
10 | + @levels.each do |level| | |
|
11 | + @available_problems[level] = [] | |
|
12 | + end | |
|
13 | + @problems.find_all {|p| not p.available }.each do |problem| | |
|
14 | + @available_problems[problem.level] << problem | |
|
15 | + end | |
|
10 | 16 | @activated_problems = @problems.find_all {|p| p.available } |
|
11 | 17 | end |
|
12 | 18 | |
|
19 | + def random_problem | |
|
20 | + level = params[:id].to_i | |
|
21 | + | |
|
22 | + problems = Problem.unavailable.level(level).all | |
|
23 | + puts problems | |
|
24 | + if problems.length!=0 | |
|
25 | + if problems.length != 1 | |
|
26 | + problem = problems[rand(problems.length)] | |
|
27 | + else | |
|
28 | + problem = problems[0] | |
|
29 | + end | |
|
30 | + problem.available = true | |
|
31 | + problem.save | |
|
32 | + end | |
|
33 | + | |
|
34 | + redirect_to :action => 'index' | |
|
35 | + end | |
|
36 | + | |
|
13 | 37 | end |
@@ -1,20 +1,23 | |||
|
1 | 1 | class ProblemsController < ApplicationController |
|
2 | 2 | |
|
3 | 3 | before_filter :authenticate, :authorization |
|
4 | 4 | |
|
5 | 5 | in_place_edit_for :problem, :name |
|
6 | 6 | in_place_edit_for :problem, :full_name |
|
7 | 7 | in_place_edit_for :problem, :full_score |
|
8 | 8 | |
|
9 | + # for codejom | |
|
10 | + in_place_edit_for :problem, :level | |
|
11 | + | |
|
9 | 12 | def index |
|
10 | 13 | list |
|
11 | 14 | render :action => 'list' |
|
12 | 15 | end |
|
13 | 16 | |
|
14 | 17 | # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html) |
|
15 | 18 | verify :method => :post, :only => [ :destroy, |
|
16 | 19 | :create, :quick_create, |
|
17 | 20 | :do_manage, |
|
18 | 21 | :do_import, |
|
19 | 22 | :update ], |
|
20 | 23 | :redirect_to => { :action => :list } |
@@ -1,17 +1,24 | |||
|
1 | 1 | class Problem < ActiveRecord::Base |
|
2 | 2 | |
|
3 | 3 | belongs_to :description |
|
4 | 4 | has_many :test_pairs, :dependent => :delete_all |
|
5 | 5 | |
|
6 | + named_scope :level, lambda { |level| | |
|
7 | + { :conditions => { :level => level }} | |
|
8 | + } | |
|
9 | + | |
|
10 | + named_scope :unavailable, { :conditions => { :available => false }} | |
|
11 | + | |
|
12 | + | |
|
6 | 13 | validates_presence_of :name |
|
7 | 14 | validates_format_of :name, :with => /^\w+$/ |
|
8 | 15 | validates_presence_of :full_name |
|
9 | 16 | |
|
10 | 17 | DEFAULT_TIME_LIMIT = 1 |
|
11 | 18 | DEFAULT_MEMORY_LIMIT = 32 |
|
12 | 19 | |
|
13 | 20 | def test_pair_count |
|
14 | 21 | @test_pair_count ||= test_pairs.size |
|
15 | 22 | end |
|
16 | 23 | |
|
17 | 24 | def uses_random_test_pair? |
@@ -1,11 +1,23 | |||
|
1 | 1 | %h1 Code Jom Control Panel |
|
2 | 2 | |
|
3 | - %h2= "Available problems (#{@available_problems.length})" | |
|
4 | - %ul | |
|
5 | - - @available_problems.each do |problem| | |
|
6 | - %li= problem.name | |
|
3 | + %h2 Available problems | |
|
4 | + %table{:class => "codejom-problems"} | |
|
5 | + %tr | |
|
6 | + - @levels.each do |level| | |
|
7 | + %th= "Level #{level} (#{@available_problems[level].length})" | |
|
8 | + %tr | |
|
9 | + - @levels.each do |level| | |
|
10 | + %td | |
|
11 | + - @available_problems[level].each do |problem| | |
|
12 | + = problem.name | |
|
13 | + %br/ | |
|
14 | + %tr | |
|
15 | + - @levels.each do |level| | |
|
16 | + %td{:class => 'random-button'} | |
|
17 | + - form_tag :action => 'random_problem', :id => level do | |
|
18 | + = submit_tag 'Random' | |
|
7 | 19 | |
|
8 | 20 | %h2= "Activated problems (#{@activated_problems.length})" |
|
9 | 21 | - @activated_problems.each do |problem| |
|
10 | 22 | = problem.name |
|
11 | 23 |
@@ -19,35 +19,37 | |||
|
19 | 19 | <label for="problem_name">Name</label> |
|
20 | 20 | <%= text_field 'problem', 'name' %> | |
|
21 | 21 | <label for="problem_full_name">Full name</label> |
|
22 | 22 | <%= text_field 'problem', 'full_name' %> |
|
23 | 23 | <%= submit_tag "Create" %> |
|
24 | 24 | <% end %> |
|
25 | 25 | </div> |
|
26 | 26 | |
|
27 | 27 | <table> |
|
28 | 28 | <tr> |
|
29 | 29 | <th>Name</th> |
|
30 | 30 | <th>Full name</th> |
|
31 | + <th>Level</th> | |
|
31 | 32 | <th>Full score</th> |
|
32 | 33 | <th>Date added</th> |
|
33 | 34 | <th>Avail?</th> |
|
34 | 35 | <th>Test?</th> |
|
35 | 36 | </tr> |
|
36 | 37 | |
|
37 | 38 | <% for problem in @problems %> |
|
38 | 39 | <tr id="prob-<%= problem.id %>" name="prob-<%= problem.id %>" class="<%= (problem.available) ? "available" : "not-available" %>"> |
|
39 | 40 | <% @problem=problem %> |
|
40 | 41 | <td><%= in_place_editor_field :problem, :name, {}, :rows=>1 %></td> |
|
41 | 42 | <td><%= in_place_editor_field :problem, :full_name, {}, :rows=>1 %></td> |
|
43 | + <td><%= in_place_editor_field :problem, :level, {}, :rows=>1 %></td> | |
|
42 | 44 | <td><%= in_place_editor_field :problem, :full_score, {}, :rows=>1 %></td> |
|
43 | 45 | <td><%= problem.date_added %></td> |
|
44 | 46 | <td id="prob-<%= problem.id %>-avail"><%= problem.available %></td> |
|
45 | 47 | <td><%= problem.test_allowed %></td> |
|
46 | 48 | |
|
47 | 49 | <td><%= link_to_remote '[Toggle]', :url => {:action => 'toggle', :id => problem.id } %></td> |
|
48 | 50 | <td><%= link_to '[Stat]', :action => 'stat', :id => problem.id %></td> |
|
49 | 51 | <td><%= link_to '[Show]', :action => 'show', :id => problem %></td> |
|
50 | 52 | <td><%= link_to '[Edit]', :action => 'edit', :id => problem %></td> |
|
51 | 53 | <td><%= link_to '[Destroy]', { :action => 'destroy', :id => problem }, :confirm => 'Are you sure?', :method => :post %></td> |
|
52 | 54 | </tr> |
|
53 | 55 | <% end %> |
@@ -1,24 +1,24 | |||
|
1 | 1 | # This file is auto-generated from the current state of the database. Instead of editing this file, |
|
2 | 2 | # please use the migrations feature of Active Record to incrementally modify your database, and |
|
3 | 3 | # then regenerate this schema definition. |
|
4 | 4 | # |
|
5 | 5 | # Note that this schema.rb definition is the authoritative source for your database schema. If you need |
|
6 | 6 | # to create the application database on another system, you should be using db:schema:load, not running |
|
7 | 7 | # all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
8 | 8 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
9 | 9 | # |
|
10 | 10 | # It's strongly recommended to check this file into your version control system. |
|
11 | 11 | |
|
12 |
- ActiveRecord::Schema.define(:version => 20100 |
|
|
12 | + ActiveRecord::Schema.define(:version => 20100209145331) do | |
|
13 | 13 | |
|
14 | 14 | create_table "announcements", :force => true do |t| |
|
15 | 15 | t.string "author" |
|
16 | 16 | t.text "body" |
|
17 | 17 | t.boolean "published" |
|
18 | 18 | t.datetime "created_at" |
|
19 | 19 | t.datetime "updated_at" |
|
20 | 20 | t.boolean "frontpage", :default => false |
|
21 | 21 | t.boolean "contest_only", :default => false |
|
22 | 22 | t.string "title" |
|
23 | 23 | end |
|
24 | 24 | |
@@ -84,24 +84,25 | |||
|
84 | 84 | end |
|
85 | 85 | |
|
86 | 86 | create_table "problems", :force => true do |t| |
|
87 | 87 | t.string "name", :limit => 30 |
|
88 | 88 | t.string "full_name" |
|
89 | 89 | t.integer "full_score" |
|
90 | 90 | t.date "date_added" |
|
91 | 91 | t.boolean "available" |
|
92 | 92 | t.string "url" |
|
93 | 93 | t.integer "description_id" |
|
94 | 94 | t.boolean "test_allowed" |
|
95 | 95 | t.boolean "output_only" |
|
96 | + t.integer "level", :default => 0 | |
|
96 | 97 | end |
|
97 | 98 | |
|
98 | 99 | create_table "rights", :force => true do |t| |
|
99 | 100 | t.string "name" |
|
100 | 101 | t.string "controller" |
|
101 | 102 | t.string "action" |
|
102 | 103 | end |
|
103 | 104 | |
|
104 | 105 | create_table "rights_roles", :id => false, :force => true do |t| |
|
105 | 106 | t.integer "right_id" |
|
106 | 107 | t.integer "role_id" |
|
107 | 108 | end |
@@ -268,12 +268,22 | |||
|
268 | 268 | .notice-bar { |
|
269 | 269 | margin-top: 3px; |
|
270 | 270 | margin-bottom: 3px; |
|
271 | 271 | text-align: center; } |
|
272 | 272 | .notice-bar span.notice { |
|
273 | 273 | color: white; |
|
274 | 274 | font-weight: bold; |
|
275 | 275 | background: #000070; |
|
276 | 276 | padding: 3px 20px 3px 20px; |
|
277 | 277 | -moz-border-radius: 2px; |
|
278 | 278 | -webkit-border-radius: 5px; |
|
279 | 279 | border-radius: 5px; } |
|
280 | + | |
|
281 | + table.codejom-problems th { | |
|
282 | + background: #000070; | |
|
283 | + color: white; } | |
|
284 | + table.codejom-problems td { | |
|
285 | + width: 200px; | |
|
286 | + vertical-align: top; | |
|
287 | + text-align: center; } | |
|
288 | + table.codejom-problems td.random-button { | |
|
289 | + background: lightgrey; } |
@@ -315,12 +315,24 | |||
|
315 | 315 | margin-top: 3px |
|
316 | 316 | margin-bottom: 3px |
|
317 | 317 | text-align: center |
|
318 | 318 | |
|
319 | 319 | span.notice |
|
320 | 320 | color: white |
|
321 | 321 | font-weight: bold |
|
322 | 322 | background: #000070 |
|
323 | 323 | padding: 3px 20px 3px 20px |
|
324 | 324 | -moz-border-radius: 2px |
|
325 | 325 | -webkit-border-radius: 5px |
|
326 | 326 | border-radius: 5px |
|
327 | + | |
|
328 | + table.codejom-problems | |
|
329 | + th | |
|
330 | + background: #000070 | |
|
331 | + color: white | |
|
332 | + td | |
|
333 | + width: 200px | |
|
334 | + vertical-align: top | |
|
335 | + text-align: center | |
|
336 | + | |
|
337 | + &.random-button | |
|
338 | + background: lightgrey No newline at end of file |
You need to be logged in to leave comments.
Login now