cafe-grader-web

Commit r606:7b10ea3e4720 public

parent | child
Description:
- clean up link to problem stat and user stat
Commit status:
[Not Reviewed]
References:
java
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

r606:7b10ea3e4720 - - 6 files changed: 15 inserted, 13 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -109,7 +109,7
109 109 redirect_to :action => 'forget'
110 110 end
111 111
112 - def profile
112 + def stat
113 113 @user = User.find(params[:id])
114 114 @submission = Submission.includes(:problem).where(user_id: params[:id])
115 115
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -53,9 +53,9
53 53 %tbody
54 54 - @submission.each do |sub|
55 55 %tr.inactive
56 - %td= link_to sub.id, controller: 'graders' ,action: 'submission', id: sub.id
57 - %td= sub.try(:user).try(:full_name)
58 - %td= sub.try(:problem).try(:full_name)
56 + %td= link_to sub.id, submission_path(sub.id)
57 + %td= link_to sub.try(:user).try(:full_name), stat_user_path(sub.user.id)
58 + %td= link_to sub.try(:problem).try(:full_name), stat_problem_path(sub.problem.id)
59 59 %td= "#{time_ago_in_words(sub.submitted_at)} ago"
60 60 %td= sub.graded_at ? "#{time_ago_in_words(sub.graded_at)} ago" : " "
61 61 %td= sub.grader_comment
@@ -71,9 +71,9
71 71 %tbody
72 72 - @backlog_submission.each do |sub|
73 73 %tr.inactive
74 - %td= link_to sub.id, controller: 'graders' ,action: 'submission', id: sub.id
75 - %td= sub.try(:user).try(:full_name)
76 - %td= sub.try(:problem).try(:full_name)
74 + %td= link_to sub.id, submission_path(sub.id)
75 + %td= link_to sub.try(:user).try(:full_name), stat_user_path(sub.user.id)
76 + %td= link_to sub.try(:problem).try(:full_name), stat_problem_path(sub.problem.id)
77 77 %td= "#{time_ago_in_words(sub.submitted_at)} ago"
78 78 %td= sub.graded_at ? "#{time_ago_in_words(sub.graded_at)} ago" : " "
79 79 %td= sub.grader_comment
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -41,8 +41,8
41 41 - next unless sub.user
42 42 - row_odd,curr = !row_odd, sub.user if curr != sub.user
43 43 %tr{class: row_odd ? "info-odd" : "info-even"}
44 - %td= link_to sub.id, controller: 'graders', action: 'submission', id: sub.id
45 - %td= link_to sub.user.login, controller: :users, action: :profile, id: sub.user.id
44 + %td= link_to sub.id, submission_path(sub)
45 + %td= link_to sub.user.login, stat_user_path(sub.user)
46 46 %td= sub.user.full_name
47 47 %td= time_ago_in_words(sub.submitted_at) + " ago"
48 48 %td= sub.points
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -29,7 +29,7
29 29 %strong User
30 30 %td
31 31 - if @submission.user
32 - = link_to "(#{@submission.user.login})", controller: "users", action: "profile", id: @submission.user
32 + = link_to "#{@submission.user.login}", stat_user_path(@submission.user)
33 33 = @submission.user.full_name
34 34 - else
35 35 = "(n/a)"
@@ -38,7 +38,7
38 38 %strong Task
39 39 %td
40 40 - if @submission.problem!=nil
41 - = link_to "(#{@submission.problem.name})", controller: "problems", action: "stat", id: @submission.problem
41 + = link_to "[#{@submission.problem.name}]", stat_problem_path(@submission.problem)
42 42 = @submission.problem.full_name
43 43 - else
44 44 = "(n/a)"
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -52,8 +52,8
52 52 - @submission.each do |s|
53 53 - next unless s.problem
54 54 %tr
55 - %td= link_to "#{s.id}", controller: "graders", action: "submission", id: s.id
56 - %td= link_to s.problem.name, controller: "problems", action: "stat", id: s.problem
55 + %td= link_to s.id, submission_path(s)
56 + %td= link_to s.problem.name, stat_problem_path(s.problem)
57 57 %td= s.problem.full_name
58 58 %td= s.language.pretty_name
59 59 %td #{s.submitted_at.strftime('%Y-%m-%d %H:%M')} (#{time_ago_in_words(s.submitted_at)} ago)
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -17,6 +17,7
17 17 member do
18 18 get 'toggle'
19 19 get 'toggle_test'
20 + get 'stat'
20 21 end
21 22 collection do
22 23 get 'turn_all_off'
@@ -31,6 +32,7
31 32 resources :users do
32 33 member do
33 34 get 'toggle_activate', 'toggle_enable'
35 + get 'stat'
34 36 end
35 37 end
36 38
You need to be logged in to leave comments. Login now