cafe-grader-web Commit - r695:55afb135c5f9

cafe-grader-web

Commit `r695:55afb135c5f9` public

parent | child

Description:

merge

Commit status:

[Not Reviewed]

References:

merge java

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r695:55afb135c5f9 - Thu, 02 Nov 2017 11:33:31 - 12 files changed: 131 inserted, 68 deleted

Gemfile ¶ +4 -3

               gem 'jquery-tablesorter'
               gem 'jquery-countdown-rails'
               #syntax highlighter
               gem 'rouge'
-            - #add bootstrap
+            + #bootstrap add-ons
               gem 'bootstrap-sass', '~> 3.2.0'
               gem 'bootstrap-switch-rails'
               gem 'bootstrap-toggle-rails'
               gem 'autoprefixer-rails'
-            - #bootstrap sortable
               gem 'momentjs-rails'
               gem 'rails_bootstrap_sortable'
+            + gem 'bootstrap-datepicker-rails'
+            + gem 'bootstrap3-datetimepicker-rails'
+            + gem 'jquery-datatables-rails'
               #----------- user interface -----------------
               #select 2
               gem 'select2-rails'
               #ace editor
               gem 'ace-rails-ap'

Gemfile.lock ¶ +13 -1

                   arel (6.0.4)
                   autoprefixer-rails (6.6.0)
                     execjs
                   best_in_place (3.0.3)
                     actionpack (>= 3.2)
                     railties (>= 3.2)
+            +     bootstrap-datepicker-rails (1.7.1.1)
+            +       railties (>= 3.0)
                   bootstrap-sass (3.2.0.2)
                     sass (~> 3.2)
                   bootstrap-switch-rails (3.3.3)
                   bootstrap-toggle-rails (2.2.1.0)
+            +     bootstrap3-datetimepicker-rails (4.17.47)
+            +       momentjs-rails (>= 2.8.1)
                   builder (3.2.2)
                   coffee-rails (4.2.1)
                     coffee-script (>= 2.2.0)
                     railties (>= 4.0.0, < 5.2.x)
                   coffee-script (2.4.1)
                     coffee-script-source
                     haml (~> 4.0.0)
                     nokogiri (~> 1.6.0)
                     ruby_parser (~> 3.5)
                   i18n (0.7.0)
                   in_place_editing (1.2.0)
                   jquery-countdown-rails (2.0.2)
+            +     jquery-datatables-rails (3.4.0)
+            +       actionpack (>= 3.1)
+            +       jquery-rails
+            +       railties (>= 3.1)
+            +       sass-rails
                   jquery-rails (4.2.1)
                     rails-dom-testing (>= 1, < 3)
                     railties (>= 4.2.0)
                     thor (>= 0.14, < 2.0)
                   jquery-tablesorter (1.23.3)
                     railties (>= 3.2, < 6)
               DEPENDENCIES
                 ace-rails-ap
                 activerecord-session_store
                 autoprefixer-rails
                 best_in_place (~> 3.0.1)
+            +   bootstrap-datepicker-rails
                 bootstrap-sass (~> 3.2.0)
                 bootstrap-switch-rails
                 bootstrap-toggle-rails
+            +   bootstrap3-datetimepicker-rails
                 coffee-rails
                 dynamic_form
                 fuzzy-string-match
                 haml
                 haml-rails
                 in_place_editing
                 jquery-countdown-rails
+            +   jquery-datatables-rails
                 jquery-rails
                 jquery-tablesorter
                 jquery-timepicker-addon-rails
                 jquery-ui-rails
                 mail
                 minitest-reporters
                 uglifier
                 verification!
                 will_paginate (~> 3.0.7)
                 yaml_db
               BUNDLED WITH
-            -    1.13.6
+            +    1.15.4

app/assets/javascripts/application.js ¶ +6

               //
               // WARNING: THE FIRST BLANK LINE MARKS THE END OF WHAT'S TO BE PROCESSED, ANY BLANK LINE SHOULD
               // GO AFTER THE REQUIRES BELOW.
               //
               //= require jquery
               //= require jquery_ujs
+            + //= require dataTables/jquery.dataTables
+            + //= require dataTables/bootstrap/3/jquery.dataTables.bootstrap
               //= require jquery-ui
               //= require bootstrap-sprockets
               //= require moment
+            + //= require moment/th
               //= require bootstrap-sortable
+            + //= require bootstrap-datetimepicker
               //= require select2
               //= require ace-rails-ap
               //= require ace/mode-c_cpp
               //= require ace/mode-python
               //= require ace/mode-ruby
               //= require ace/mode-pascal
               //= require jquery.countdown
               //-------------- addition from local_jquery -----------
               //= require jquery-tablesorter
               //= require best_in_place
               //= require best_in_place.jquery-ui
               //= require brython
+            + //= require bootstrap-datepicker
+            + //= require bootstrap-datetimepicker
               // since this is after blank line, it is not downloaded
               //x= require prototype
               //x= require prototype_ujs
               //x= require effects
               //x= require dragdrop

app/assets/stylesheets/application.css.scss ¶ +3

               @import "select2";
               @import "select2-bootstrap";
               //@import bootstrap3-switch
               @import "bootstrap-toggle";
               @import "bootstrap-sortable";
+            + @import "bootstrap-datepicker3";
+            + @import "bootstrap-datetimepicker";
+            + @import "dataTables/bootstrap/3/jquery.dataTables.bootstrap";
               //bootstrap navbar color (from)
               $bgDefault: #19197b;
               $bgHighlight: #06064b;
               $colDefault: #8e8eb4;
               $colHighlight: #ffffff;

app/controllers/login_controller.rb ¶ +26 -20

@@ -4,38 +4,44
4	# show login screen	4	# show login screen
5	reset_session	5	reset_session
6	redirect_to :controller => 'main', :action => 'login'	6	redirect_to :controller => 'main', :action => 'login'
7	end	7	end
8		8
9	def login	9	def login
10	- if (!GraderConfiguration['right.bypass_agreement']) and (!params[:accept_agree])	10	+ user = User.authenticate(params[:login], params[:password])
		11	+ unless user
		12	+ flash[:notice] = 'Wrong password'
		13	+ redirect_to :controller => 'main', :action => 'login'
		14	+ return
		15	+ end
		16	+
		17	+ if (!GraderConfiguration['right.bypass_agreement']) and (!params[:accept_agree]) and !user.admin?
11	flash[:notice] = 'You must accept the agreement before logging in'	18	flash[:notice] = 'You must accept the agreement before logging in'
12	redirect_to :controller => 'main', :action => 'login'	19	redirect_to :controller => 'main', :action => 'login'
13	- elsif user = User.authenticate(params[:login], params[:password])	20	+ return
14	- session[:user_id] = user.id	21	+ end
15	- session[:admin] = user.admin?	22	+
		23	+ #process logging in
		24	+ session[:user_id] = user.id
		25	+ session[:admin] = user.admin?
16		26
17	- # clear forced logout flag for multicontests contest change	27	+ # clear forced logout flag for multicontests contest change
18	- if GraderConfiguration.multicontests?	28	+ if GraderConfiguration.multicontests?
19	- contest_stat = user.contest_stat	29	+ contest_stat = user.contest_stat
20	- if contest_stat.respond_to? :forced_logout	30	+ if contest_stat.respond_to? :forced_logout
21	- if contest_stat.forced_logout	31	+ if contest_stat.forced_logout
22	- contest_stat.forced_logout = false	32	+ contest_stat.forced_logout = false
23	- contest_stat.save	33	+ contest_stat.save
24	- end
25	end	34	end
26	end	35	end
27	-	36	+ end
28	- #save login information
29	- Login.create(user_id: user.id, ip_address: request.remote_ip)
30		37
31	- redirect_to :controller => 'main', :action => 'list'	38	+ #save login information
32	- else	39	+ Login.create(user_id: user.id, ip_address: request.remote_ip)
33	- flash[:notice] = 'Wrong password'	40	+
34	- redirect_to :controller => 'main', :action => 'l~~ogin~~'	41	+ redirect_to :controller => 'main', :action => 'list'
35	- end
36	end	42	end
37		43
38	def site_login	44	def site_login
39	begin	45	begin
40	site = Site.find(params[:login][:site_id])	46	site = Site.find(params[:login][:site_id])
41	rescue ActiveRecord::RecordNotFound	47	rescue ActiveRecord::RecordNotFound

app/controllers/problems_controller.rb ¶ +1 -4

                     return false
                   end
                 end
                 def change_date_added
                   problems = get_problems_from_params
-            -     year = params[:date_added][:year].to_i
+            +     date = Date.parse(params[:date_added])
-            -     month = params[:date_added][:month].to_i
-            -     day = params[:date_added][:day].to_i
-            -     date = Date.new(year,month,day)
                   problems.each do |p|
                     p.date_added = date
                     p.save
                   end
                 end

app/controllers/user_admin_controller.rb ¶ +1

                     @users = User.all
                     @paginated = false
                   else
                     @users = User.paginate :page => params[:page]
                     @paginated = true
                   end
+            +     @users = User.all
                   @hidden_columns = ['hashed_password', 'salt', 'created_at', 'updated_at']
                   @contests = Contest.enabled
                 end
                 def active
                   sessions = ActiveRecord::SessionStore::Session.where("updated_at >= ?", 60.minutes.ago)

app/views/problems/manage.html.haml ¶ +19 -6

                     Action
                   .panel-body
                     .submit-box
                       What do you want to do to the selected problem?
                       %br/
                       (You can shift-click to select a range of problems)
-            -         %ul
+            +         %ul.form-inline
                         %li
                           Change date added to
-            -             = select_date Date.current, :prefix => 'date_added'
+            +             .input-group.date
+            +               = text_field_tag :date_added, class: 'form-control'
+            +               %span.input-group-addon
+            +                 %span.glyphicon.glyphicon-calendar
+            +             -# = select_date Date.current, :prefix => 'date_added'
                           &nbsp;&nbsp;&nbsp;
-            -             = submit_tag 'Change', :name => 'change_date_added', class: 'btn btn-default'
+            +             = submit_tag 'Change', :name => 'change_date_added', class: 'btn btn-primary btn-sm'
                         %li
                           Set available to
-            -             = submit_tag 'True', :name => 'enable_problem', class: 'btn btn-default'
+            +             = submit_tag 'True', :name => 'enable_problem', class: 'btn btn-primary btn-sm'
-            -             = submit_tag 'False', :name => 'disable_problem', class: 'btn btn-default'
+            +             = submit_tag 'False', :name => 'disable_problem', class: 'btn btn-primary btn-sm'
                         - if GraderConfiguration.multicontests?
                           %li
                             Add to
                             = select("contest","id",Contest.all.collect {|c| [c.title, c.id]})
-            -               = submit_tag 'Add', :name => 'add_to_contest', class: 'btn btn-default'
+            +               = submit_tag 'Add', :name => 'add_to_contest', class: 'btn btn-primary btn-sm'
                         %li
                           Add problems to group
                           = select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_name]), id: 'group_name',class: 'select2'
                           = submit_tag 'Add', name: 'add_group', class: 'btn btn-default'
                       %td= problem.available
                       %td= problem.date_added
                       - if GraderConfiguration.multicontests?
                         %td
                           - problem.contests.each do |contest|
                             = "(#{contest.name} [#{link_to 'x', :action => 'remove_contest', :id => problem.id, :contest_id => contest.id }])"
+            +
+            + :javascript
+            +   $('.input-group.date').datetimepicker({
+            +     format: 'DD/MMM/YYYY',
+            +     showTodayButton: true,
+            +     widgetPositioning: {horizontal: 'auto', vertical: 'bottom'},
+            +
+            +   });
+            +

app/views/problems/stat.html.haml ¶ +8 -4

               %h2 Submissions Count
               = render partial: 'application/bar_graph', locals: { histogram: @histogram }
               %h2 Submissions
               - if @submissions and @submissions.count > 0
-            -   %table.info#main_table
+            +   %table#main_table.table.table-condensed.table-striped
                   %thead
-            -       %tr.info-head
+            +       %tr
                       %th ID
                       %th Login
                       %th Name
                       %th Submitted_at
                       %th Points
                       %th comment
                       %th IP
                   %tbody
                     - row_odd,curr = true,''
                     - @submissions.each do |sub|
                       - next unless sub.user
                       - row_odd,curr = !row_odd, sub.user if curr != sub.user
-            -         %tr{class: row_odd ? "info-odd" : "info-even"}
+            +         %tr
                         %td= link_to sub.id, submission_path(sub)
                         %td= link_to sub.user.login, stat_user_path(sub.user)
                         %td= sub.user.full_name
-            -           %td= time_ago_in_words(sub.submitted_at) + " ago"
+            +           %td{data: {order: sub.submitted_at}}= time_ago_in_words(sub.submitted_at) + " ago"
                         %td= sub.points
                         %td.fix-width= sub.grader_comment
                         %td= sub.ip_address
               - else
                 No submission
+            + :javascript
+            +   $("#main_table").DataTable({
+            +     paging: false
+            +   });

app/views/sites/edit.html.haml ¶ +29 -18

@@ -1,24 +1,35
1	%h1 Editing site	1	%h1 Editing site
2	= error_messages_for :site	2	= error_messages_for :site
3	= form_for(@site) do \|f\|	3	= form_for(@site) do \|f\|
4	- %p	4	+ .row
5	- %b Name	5	+ .col-md-4
6	- %br/	6	+ .form-group.field
7	- = f.text_field :name	7	+ = f.label :name, "Name"
8	- %p	8	+ = f.text_field :name, class: 'form-control'
9	- %b Password	9	+ .form-group.field
10	- %br/	10	+ = f.label :password, "Password"
11	- = f.text_field :password	11	+ = f.text_field :password, class: 'form-control'
12	- %p	12	+ .form-group.field
13	- %b Started	13	+ = f.label :started, "Started"
14	- %br/	14	+ = f.check_box :started, class: 'form-control'
15	- = f.check_box :started	15	+ .form-group.field
16	- %p	16	+ = f.label :start_time, "Start time"
17	- %b Start time	17	+ -# = f.datetime_select :start_time, :include_blank => true
18	- %br/	18	+ .input-group.date
19	- = f.datetime_select :start_time, :include_blank => true	19	+ = f.text_field :start_time, class:'form-control' , value: (@site.start_time ? @site.start_time.strftime('%d/%b/%Y %H:%M') : '')
20	- %p	20	+ %span.input-group-addon
21	- = f.submit "Update"	21	+ %span.glyphicon.glyphicon-calendar
		22	+ .actions
		23	+ = f.submit "Update", class: 'btn btn-primary'
		24	+ .col-md-8
		25	+
22	= link_to 'Show', @site	26	= link_to 'Show', @site
23	\|	27	\|
24	= link_to 'Back', sites_path	28	= link_to 'Back', sites_path
		29	+
		30	+
		31	+ :javascript
		32	+ $('.input-group.date').datetimepicker({
		33	+ format: 'DD/MMM/YYYY HH:mm',
		34	+ });
		35	+

app/views/user_admin/index.html.haml ¶ +16 -11

-            - %h1 Listing users
+            + %h1 Users
               .panel.panel-primary
                 .panel-title.panel-heading
                   Quick Add
                 .panel-body
                   = form_tag( {method: 'post'}, {class: 'form-inline'})  do
                     = submit_tag 'Submit', class: 'btn btn-default'
               %p
                 = link_to '+ New user', { :action => 'new' }, { class: 'btn btn-success '}
                 = link_to '+ New list of users', { :action => 'new_list' }, { class: 'btn btn-success '}
+            +   = link_to 'Bulk Manage', bulk_manage_user_admin_path , { class: 'btn btn-default btn-info'}
                 = link_to 'View administrators',{ :action => 'admin'}, { class: 'btn btn-default '}
-            -   = link_to 'Bulk Manage', bulk_manage_user_admin_path , { class: 'btn btn-default '}
                 = link_to 'Random passwords',{ :action => 'random_all_passwords'}, { class: 'btn btn-default '}
                 = link_to 'View active users',{ :action => 'active'}, { class: 'btn btn-default '}
                 = link_to 'Mass mailing',{ :action => 'mass_mailing'}, { class: 'btn btn-default '}
               - if GraderConfiguration.multicontests?
                 %br/
                 = link_to '[Manage bulk users in contests]', :action => 'contest_management'
                 View users in:
                 - @contests.each do |contest|
                   = link_to "[#{contest.name}]", :action => 'contests', :id => contest.id
                 = link_to "[no contest]", :action => 'contests', :id => 'none'
-            - Total #{@user_count} users |
+            + -# Total #{@user_count} users |
-            - - if !@paginated
+            + -# - if !@paginated
-            -   Display all users.
+            + -#   Display all users.
-            -   \#{link_to '[show in pages]', :action => 'index', :page => '1'}
+            + -#   \#{link_to '[show in pages]', :action => 'index', :page => '1'}
-            - - else
+            + -# - else
-            -   Display in pages.
+            + -#   Display in pages.
-            -   \#{link_to '[display all]', :action => 'index', :page => 'all'} |
+            + -#   \#{link_to '[display all]', :action => 'index', :page => 'all'} |
-            -   \#{will_paginate @users, :container => false}
+            + -#   \#{will_paginate @users, :container => false}
-            - %table.table.table-hover.table-condense
+            + %table.table.table-hover.table-condense.datatable
                 %thead
                   %th Login
                   %th Full name
                   %th email
                   %th Remark
                   %th
                     %td= link_to 'Show', {:action => 'show', :id => user}, class: 'btn btn-default btn-xs btn-block'
                     %td= link_to 'Edit', {:action => 'edit', :id => user}, class: 'btn btn-default btn-xs btn-block'
                     %td= link_to 'Destroy', user_admin_destroy_path(user), data: {confirm: 'Are you sure?'}, method: :delete, class: 'btn btn-danger btn-xs btn-block'
               %br/
               = link_to '+ New user', { :action => 'new' }, { class: 'btn btn-success '}
               = link_to '+ New list of users', { :action => 'new_list' }, { class: 'btn btn-success '}
+            +
+            + :javascript
+            +   $('.datatable').DataTable({
+            +     'pageLength': 50
+            +   });

app/views/users/stat.html.haml ¶ +5 -1

               %h2 Submission History
               =render partial: 'application/bar_graph', locals: {histogram: @histogram, param: {bar_width: 7}}
-            - %table.tablesorter-cafe#submission_table
+            + %table#submission_table.table.table-striped
                 %thead
                   %tr
                     %th ID
                     %th Problem code
                     %th Problem full name
                     %th Language
                       %td= ( s.try(:points) ? (s.points*100/s.problem.full_score)  : '' )
                       - if session[:admin]
                         %td= s.ip_address
+            + :javascript
+            +   $("#submission_table").DataTable({
+            +     paging: false
+            +   });

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

r695:55afb135c5f9 - Thu, 02 Nov 2017 11:33:31 - 12 files changed: 131 inserted, 68 deleted

Sign in to your account

Commit r695:55afb135c5f9 public

Author

r695:55afb135c5f9 - Thu, 02 Nov 2017 11:33:31 - 12 files changed: 131 inserted, 68 deleted

Commit `r695:55afb135c5f9` public