cafe-grader-web Commit - r700:757ebc0ccd63

cafe-grader-web

Commit `r700:757ebc0ccd63` public

parent | child

Description:

add tags

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

r700:757ebc0ccd63 - Sat, 10 Mar 2018 07:07:14 - 7 files changed: 56 inserted, 29 deleted

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

                     end
                     flash[:success] = "The following problems are added to the group #{group.name}: " + ok.join(', ') if ok.count > 0
                     flash[:alert] = "The following problems are already in the group #{group.name}: " + failed.join(', ') if failed.count > 0
+            +     elsif params.has_key? 'add_tags'
+            +       get_problems_from_params.each do |p|
+            +         p.tag_ids += params[:tag_ids]
+            +       end
                   end
                   redirect_to :action => 'manage'
                 private
                   def problem_params
-            -       params.require(:problem).permit(:name, :full_name, :full_score, :date_added, :available, :test_allowed,:output_only, :url, :description)
+            +       params.require(:problem).permit(:name, :full_name, :full_score, :date_added, :available, :test_allowed,:output_only, :url, :description, tag_ids:[])
                   end
               end

app/controllers/tags_controller.rb ¶ +2

                 # DELETE /tags/1
                 def destroy
+            +     #remove any association
+            +     ProblemTag.where(tag_id: @tag.id).destroy_all
                   @tag.destroy
                   redirect_to tags_url, notice: 'Tag was successfully destroyed.'
                 end

app/views/layouts/_header.html.haml ¶ +1

                             %ul.dropdown-menu
                               = add_menu( 'Announcements', 'announcements', 'index')
                               = add_menu( 'Problems', 'problems', 'index')
+            +                 = add_menu( 'Tags', 'tags', 'index')
                               = add_menu( 'Users', 'user_admin', 'index')
                               = add_menu( 'User Groups', 'groups', 'index')
                               = add_menu( 'Graders', 'graders', 'list')

app/views/problems/_form.html.haml ¶ +3

                 %label{:for => "problem_full_score"} Full score
                 = text_field 'problem', 'full_score', class: 'form-control'
               .form-group
+            +   %label{:for => "problem_full_score"} Tags
+            +   = collection_select(:problem, :tag_ids, Tag.all, :id, :name,  {}, {multiple: true, class: 'form-control select2'})
+            + .form-group
                 %label{:for => "problem_date_added"} Date added
                 = date_select 'problem', 'date_added', class: 'form-control'
               - # TODO: these should be put in model Problem, but I can't think of

app/views/problems/index.html.haml ¶ +5

                   %th Name
                   %th Full name
                   %th.text-right Full score
+            +     %th Tags
                   %th
                     Submit
                     %sup{class: 'text-primary',data: {toggle: 'tooltip'}, title: 'Admin can always submit to any problem' } [?]
                       = problem.full_name #in_place_editor_field :problem, :full_name, {}, :rows=>1
                       = link_to_description_if_any "[#{t 'main.problem_desc'}] <span class='glyphicon glyphicon-file'></span>".html_safe, problem
                     %td.text-right= problem.full_score #in_place_editor_field :problem, :full_score, {}, :rows=>1
+            +       %td
+            +         - problem.tags.each do |t|
+            +           - #%button.btn.btn-default.btn-xs= t.name
+            +           %span.label.label-default= t.name
                     %td= link_to "Submit", direct_edit_problem_submissions_path(problem,@current_user.id), class: 'btn btn-xs btn-primary'
                     %td= problem.date_added
                     %td= toggle_button(@problem.available?, toggle_problem_path(@problem), "problem-avail-#{@problem.id}")

app/views/problems/manage.html.haml ¶ +39 -28

@@ -49,7 +49,7
49	(You can shift-click to select a range of problems)	49	(You can shift-click to select a range of problems)
50	%ul.form-inline	50	%ul.form-inline
51	%li	51	%li
52	- Change date added to	52	+ Change "Date added" to
53	.input-group.date	53	.input-group.date
54	= text_field_tag :date_added, class: 'form-control'	54	= text_field_tag :date_added, class: 'form-control'
55	%span.input-group-addon	55	%span.input-group-addon
@@ -58,44 +58,53
58		58
59	= submit_tag 'Change', :name => 'change_date_added', class: 'btn btn-primary btn-sm'	59	= submit_tag 'Change', :name => 'change_date_added', class: 'btn btn-primary btn-sm'
60	%li	60	%li
61	- Set available to	61	+ Set "Available" to
62	= submit_tag 'True', :name => 'enable_problem', class: 'btn btn-primary btn-sm'	62	= submit_tag 'True', :name => 'enable_problem', class: 'btn btn-primary btn-sm'
63	= submit_tag 'False', :name => 'disable_problem', class: 'btn btn-primary btn-sm'	63	= submit_tag 'False', :name => 'disable_problem', class: 'btn btn-primary btn-sm'
64		64
65	- if GraderConfiguration.multicontests?	65	- if GraderConfiguration.multicontests?
66	%li	66	%li
67	- Add to	67	+ Add selected problems to contest
68	= select("contest","id",Contest.all.collect {\|c\| [c.title, c.id]})	68	= select("contest","id",Contest.all.collect {\|c\| [c.title, c.id]})
69	= submit_tag 'Add', :name => 'add_to_contest', class: 'btn btn-primary btn-sm'	69	= submit_tag 'Add', :name => 'add_to_contest', class: 'btn btn-primary btn-sm'
70	%li	70	%li
71	- Add problems to group	71	+ Add selected problems to user group
72	= select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_name]), id: 'group_name',class: 'select2'	72	= select_tag "group_id", options_from_collection_for_select( Group.all, 'id','name',params[:group_name]), id: 'group_name',class: 'select2'
73	- = submit_tag 'Add', name: 'add_group', class: 'btn btn-~~default~~'	73	+ = submit_tag 'Add', name: 'add_group', class: 'btn btn-primary'
74	-	74	+ %li
		75	+ Add the following tags to the selected problems
		76	+ = select_tag "tag_ids", options_from_collection_for_select( Tag.all, 'id','name'), id: 'tags_name',class: 'select2', multiple: true, data: {placeholder: 'Select tags by clicking', width: "200px"}
		77	+ = submit_tag 'Add', name: 'add_tags', class: 'btn btn-primary'
75		78
76	- %table.table.table-hover	79	+ %table.table.table-hover.datatable
77	- %tr{style: "text-align: left;"}	80	+ %thead
78	- %th= check_box_tag 'select_all'	81	+ %tr{style: "text-align: left;"}
79	- %th Name	82	+ %th= check_box_tag 'select_all'
80	- %th ~~Full n~~ame	83	+ %th Name
81	- %th ~~Availabl~~e	84	+ %th Full name
82	- %th ~~Date added~~	85	+ %th Tags
83	- - if GraderConfiguration.multicontests?	86	+ %th Available
84	- %th ~~Contests~~	87	+ %th Date added
		88	+ - if GraderConfiguration.multicontests?
		89	+ %th Contests
85		90
86	- - num = 0	91	+ %tbody
87	- - for problem in @problems	92	+ - num = 0
88	- - num += 1	93	+ - for problem in @problems
89	- %tr{:id => "row-prob-#{problem.id}", :name=> "prob-#{problem.id}"}	94	+ - num += 1
90	- %td= check_box_tag "prob-#{problem.id}-#{num}"	95	+ %tr{:id => "row-prob-#{problem.id}", :name=> "prob-#{problem.id}"}
91	- %td= problem.name	96	+ %td= check_box_tag "prob-#{problem.id}-#{num}"
92	- %td= problem.~~full_~~name	97	+ %td= problem.name
93	- %td= problem.~~availabl~~e	98	+ %td= problem.full_name
94	- %td= problem.date_added
95	- - if GraderConfiguration.multicontests?
96	%td	99	%td
97	- - problem.~~contest~~s.each do \|~~contes~~t\|	100	+ - problem.tags.each do \|t\|
98	- = "(#{contest.name} [#{link_to 'x', :action => 'remove_contest', :id => problem.id, :contest_id => contest.id }])"	101	+ %span.label.label-default= t.name
		102	+ %td= problem.available
		103	+ %td= problem.date_added
		104	+ - if GraderConfiguration.multicontests?
		105	+ %td
		106	+ - problem.contests.each do \|contest\|
		107	+ = "(#{contest.name} [#{link_to 'x', :action => 'remove_contest', :id => problem.id, :contest_id => contest.id }])"
99		108
100	:javascript	109	:javascript
101	$('.input-group.date').datetimepicker({	110	$('.input-group.date').datetimepicker({
@@ -104,4 +113,6
104	widgetPositioning: {horizontal: 'auto', vertical: 'bottom'},	113	widgetPositioning: {horizontal: 'auto', vertical: 'bottom'},
105		114
106	});	115	});
107	-	116	+ $('.datatable').DataTable({
		117	+ paging: false
		118	+ });

app/views/sites/edit.html.haml ¶ +1

               :javascript
                 $('.input-group.date').datetimepicker({
                   format: 'DD/MMM/YYYY HH:mm',
+            +     showTodayButton: true,
                 });

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

r700:757ebc0ccd63 - Sat, 10 Mar 2018 07:07:14 - 7 files changed: 56 inserted, 29 deleted

Sign in to your account

Commit r700:757ebc0ccd63 public

Author

r700:757ebc0ccd63 - Sat, 10 Mar 2018 07:07:14 - 7 files changed: 56 inserted, 29 deleted

Commit `r700:757ebc0ccd63` public