cafe-grader-web Commit - r644:6431e2ef4265

cafe-grader-web

parent | child

Description:

reorder submission and remove duplicate code for submission

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

r644:6431e2ef4265 - Fri, 10 Mar 2017 01:43:21 - 7 files changed: 8 inserted, 55 deleted

app/controllers/main_controller.rb ¶ -17

@@ -124,23 +124,6
124	end	124	end
125	end	125	end
126		126
127	- def submission
128	- @user = User.find(session[:user_id])
129	- @problems = @user.available_problems
130	- if params[:id]==nil
131	- @problem = nil
132	- @submissions = nil
133	- else
134	- @problem = Problem.find_by_id(params[:id])
135	- if (@problem == nil) or (not @problem.available)
136	- redirect_to :action => 'list'
137	- flash[:notice] = 'Error: submissions for that problem are not viewable.'
138	- return
139	- end
140	- @submissions = Submission.find_all_by_user_problem(@user.id, @problem.id)
141	- end
142	- end
143	-
144	def result	127	def result
145	if !GraderConfiguration.show_grading_result	128	if !GraderConfiguration.show_grading_result
146	redirect_to :action => 'list' and return	129	redirect_to :action => 'list' and return

app/controllers/submissions_controller.rb ¶ +1 -1

                       flash[:notice] = 'Error: submissions for that problem are not viewable.'
                       return
                     end
-            -       @submissions = Submission.find_all_by_user_problem(@user.id, @problem.id)
+            +       @submissions = Submission.find_all_by_user_problem(@user.id, @problem.id).order(id: :desc)
                   end
                 end

app/helpers/application_helper.rb ¶ +1 -1

                   result = content_tag(:ul,left_menu.html_safe,class: 'nav navbar-nav') + content_tag(:ul,right_menu.html_safe,class: 'nav navbar-nav navbar-right')
                 end
-            -   def add_menu(title, controller, action,html_option = {})
+            +   def add_menu(title, controller, action, html_option = {})
                   link_option = {controller: controller, action: action}
                   html_option[:class] = (html_option[:class] || '') + " active" if current_page?(link_option)
                   content_tag(:li, link_to(title,link_option),html_option)

app/views/application/_submission.html.haml ¶ +2 -2

               %tr
                 %td{:align => "center"}
-            -     = submission_counter+1
+            +     = submission.number
-            -   %td{:align => "center"}
+            +   %td.text-right
                   = link_to "##{submission.id}", submission_path(submission.id)
                 %td
                   = l submission.submitted_at, format: :long

app/views/layouts/_header.html.haml ¶ +3 -2

                         MAIN
                     .collapse.navbar-collapse#navbar-collapse
                       %ul.nav.navbar-nav
+            +           / submission
                         - if (@current_user!=nil) and (GraderConfiguration.show_tasks_to?(@current_user))
-            -             //= add_menu("#{I18n.t 'menu.tasks'}", 'tasks', 'list')
                           %li.dropdown
                             %a.dropdown-toggle{href: '#', data: {toggle:'dropdown'},  aria: {haspopup:"true", expanded:"false"}, role: "button"}
                               = "#{I18n.t 'menu.submissions'}"
                               %span.caret
                             %ul.dropdown-menu
-            -                 = add_menu("View", 'main', 'submission')
+            +                 = add_menu("View", 'submissions', 'index')
                               = add_menu("Self Test", 'test', 'index')
+            +           / hall of fame
                         - if GraderConfiguration['right.user_hall_of_fame']
                           = add_menu("#{I18n.t 'menu.hall_of_fame'}", 'report', 'problem_hof')
                         / display MODE button (with countdown in contest mode)

app/views/submissions/index.html.haml ¶ +1 -1

                   %table.table
                     %thead
                       %th No.
-            -         %th #
+            +         %th.text-right #
                       %th At
                       %th Source
                       %th Result

app/views/main/submission.html.haml removed ¶ -31

deleted file

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

	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