cafe-grader-web Commit - r598:db36298d10ca

cafe-grader-web

Commit `r598:db36298d10ca` public

parent | child

Description:

- remove inplace editor from view - add link to edit announcement directly - modify submission_short display button

Commit status:

[Not Reviewed]

References:

java

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r598:db36298d10ca - Wed, 05 Oct 2016 08:26:00 - 4 files changed: 55 inserted, 22 deleted

app/controllers/report_controller.rb ¶ +46 -14

@@ -12,12 +12,27
12	admin_authorization	12	admin_authorization
13	}	13	}
14		14
15	def max_score	15	def max_score
16	end	16	end
17		17
		18	+ def current_score
		19	+ @problems = Problem.find_available_problems
		20	+ @users = User.includes(:contests).includes(:contest_stat).where(enabled: true)
		21	+ @scorearray = calculate_max_score(problems, users,0,0,{max: true})
		22	+
		23	+ #rencer accordingly
		24	+ if params[:commit] == 'download csv' then
		25	+ csv = gen_csv_from_scorearray(@scorearray,@problems)
		26	+ send_data csv, filename: 'max_score.csv'
		27	+ else
		28	+ #render template: 'user_admin/user_stat'
		29	+ render 'current_score'
		30	+ end
		31	+ end
		32	+
18	def show_max_score	33	def show_max_score
19	#process parameters	34	#process parameters
20	#problems	35	#problems
21	@problems = []	36	@problems = []
22	params[:problem_id].each do \|id\|	37	params[:problem_id].each do \|id\|
23	next unless id.strip != ""	38	next unless id.strip != ""
@@ -32,27 +47,16
32	end	47	end
33		48
34	#set up range from param	49	#set up range from param
35	since_id = params.fetch(:min_id, 0).to_i	50	since_id = params.fetch(:min_id, 0).to_i
36	until_id = params.fetch(:max_id, 0).to_i	51	until_id = params.fetch(:max_id, 0).to_i
37		52
38	- #get data	53	+ #calculate the routine
39	- @scorearray = Array.new	54	+ @scorearray = calculate_max_score(problems, users,since_id,until_id)
40	- @users.each do \|u\|
41	- ustat = Array.new
42	- ustat[0] = u
43	- @problems.each do \|p\|
44	- max_points = 0
45	- Submission.find_in_range_by_user_and_problem(u.id,p.id,since_id,until_id).each do \|sub\|
46	- max_points = sub.points if sub and sub.points and (sub.points > max_points)
47	- end
48	- ustat << [(max_points.to_f*100/p.full_score).round, (max_points>=p.full_score)]
49	- end
50	- @scorearray << ustat
51	- end
52		55
		56	+ #rencer accordingly
53	if params[:commit] == 'download csv' then	57	if params[:commit] == 'download csv' then
54	csv = gen_csv_from_scorearray(@scorearray,@problems)	58	csv = gen_csv_from_scorearray(@scorearray,@problems)
55	send_data csv, filename: 'max_score.csv'	59	send_data csv, filename: 'max_score.csv'
56	else	60	else
57	#render template: 'user_admin/user_stat'	61	#render template: 'user_admin/user_stat'
58	render 'max_score'	62	render 'max_score'
@@ -445,8 +449,36
445		449
446		450
447		451
448		452
449	end	453	end
450		454
		455	+ protected
		456	+
		457	+ def calculate_max_score(problems, users,since_id,until_id, get_last_score = false)
		458	+ scorearray = Array.new
		459	+ users.each do \|u\|
		460	+ ustat = Array.new
		461	+ ustat[0] = u
		462	+ problems.each do \|p\|
		463	+ unless get_last_score
		464	+ #get max score
		465	+ max_points = 0
		466	+ Submission.find_in_range_by_user_and_problem(u.id,p.id,since_id,until_id).each do \|sub\|
		467	+ max_points = sub.points if sub and sub.points and (sub.points > max_points)
		468	+ end
		469	+ ustat << [(max_points.to_f*100/p.full_score).round, (max_points>=p.full_score)]
		470	+ else
		471	+ #get latest score
		472	+ sub = Submission.find_last_by_user_and_problem(u.id,p.id)
		473	+ if (sub!=nil) and (sub.points!=nil) and p and p.full_score
		474	+ ustat << [(sub.points.to_f*100/p.full_score).round, (sub.points>=p.full_score)]
		475	+ else
		476	+ ustat << [0,false]
		477	+ end
		478	+ end
		479	+ scorearray << ustat
		480	+ end
		481	+ return scorearray
		482	+ end
451		483
452	end	484	end

app/views/main/_announcement.html.haml ¶ +2

               %li.list-group-item
                 %strong
                   = announcement.title
+            +   - if @current_user.admin?
+            +     = link_to 'Edit', edit_announcement_path(announcement), class: 'btn btn-xs btn-default'
                 %small= "(updated #{time_ago_in_words(announcement.updated_at)} ago on #{announcement.updated_at})"
                 %br
                   = markdown(announcement.body)
                 :javascript
                   Announcement.updateRecentId(#{announcement.id});

app/views/main/_submission_short.html.haml ¶ +4 -5

                     = submission.grader_comment
                   = "]"
                   %br
                   %strong View:
                   - if GraderConfiguration.show_grading_result
                     = link_to '[detailed result]', :action => 'result', :id => submission.id
-            -     = link_to("[#{t 'main.cmp_msg'}]", {:action => 'compiler_msg', :id => submission.id}, {:popup => true})
+            +     = link_to "#{t 'main.cmp_msg'}", {:action => 'compiler_msg', :id => submission.id}, {popup: true,class: 'btn btn-xs btn-info'}
-            -   = " | "
+            +   = link_to "#{t 'main.src_link'}",{:action => 'source', :id => submission.id}, class: 'btn btn-xs btn-info'
-            -   = link_to("[#{t 'main.src_link'}]",{:action => 'source', :id => submission.id})
+            +   = link_to "#{t 'main.submissions_link'}", problem_submissions_path(problem_id), class: 'btn btn-xs btn-info'
-            -   = " | "
-            -   = link_to "[#{t 'main.submissions_link'}]", problem_submissions_path(problem_id)

app/views/problems/index.html.haml ¶ +3 -3

                     %sup{class: 'text-primary',data: {toggle: 'tooltip'}, title: 'Let user uses test interface on this problem?' } [?]
                   - if GraderConfiguration.multicontests?
                     %th Contests
                 - for problem in @problems
                   %tr{:class => "#{(problem.available) ? "success" : "danger"}", :id => "prob-#{problem.id}", :name => "prob-#{problem.id}"}
                     - @problem=problem
-            -       %td= in_place_editor_field :problem, :name, {}, :rows=>1
+            +       %td= problem.name #in_place_editor_field :problem, :name, {}, :rows=>1
-            -       %td= in_place_editor_field :problem, :full_name, {}, :rows=>1
+            +       %td= problem.full_name #in_place_editor_field :problem, :full_name, {}, :rows=>1
-            -       %td.text-right= in_place_editor_field :problem, :full_score, {}, :rows=>1
+            +       %td.text-right= problem.full_score #in_place_editor_field :problem, :full_score, {}, :rows=>1
                     %td= problem.date_added
                     %td= toggle_button(@problem.available?, toggle_problem_url(@problem), "problem-avail-#{@problem.id}")
                     %td= toggle_button(@problem.test_allowed?, toggle_test_problem_url(@problem), "problem-test-#{@problem.id}")
                     - if GraderConfiguration.multicontests?
                       %td
                         = problem.contests.collect { |c| c.name }.join(', ')

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r598:db36298d10ca - Wed, 05 Oct 2016 08:26:00 - 4 files changed: 55 inserted, 22 deleted

Sign in to your account

Commit r598:db36298d10ca public

Author

r598:db36298d10ca - Wed, 05 Oct 2016 08:26:00 - 4 files changed: 55 inserted, 22 deleted

Commit `r598:db36298d10ca` public