Add an option to disable courses' code reviews

Author: nygrenh

app/controllers/courses_controller.rb

@@ -97,6 +97,12 @@ def toggle_hidden
     redirect_to(organization_course_path(@organization, @course), notice: "Course is now #{@course.hidden? ? 'hidden' : 'visible'}.")
   end
 
+  def toggle_code_review_requests
+    authorize! :teach, @organization
+    @course.update!(code_review_requests_enabled: !@course.code_review_requests_enabled)
+    redirect_to(organization_course_path(@organization, @course), notice: "Code review requests are now #{@course.code_review_requests_enabled? ? 'enabled' : 'disabled'}.")
+  end
+
   def manage_deadlines
     authorize! :manage_deadlines, @course
     add_course_breadcrumb

app/models/exercise.rb

@@ -401,6 +401,10 @@ def runtime_params_array
     ActiveSupport::JSON.decode(runtime_params)
   end
 
+  def code_review_requests_enabled?
+    course.code_review_requests_enabled? && self[:code_review_requests_enabled]
+  end
+
   def self.default_options
     {
       'deadline' => nil,

app/views/courses/show.html.erb

@@ -127,6 +127,13 @@
             <% end %>
           <% end %>
 
+            <li>
+              <% if @course.code_review_requests_enabled? %>
+                <%= link_to 'Disable code review requests', toggle_code_review_requests_organization_course_path(@organization, @course), method: :post, class: "btn btn-warning btn-sm" %>
+              <% else %>
+                <%= link_to 'Enable code review requests', toggle_code_review_requests_organization_course_path(@organization, @course), method: :post, class: "btn btn-warning btn-sm" %>
+            <% end %>
+            </li>
           <% # Course timing, deadlines, unlocks %>
           <% if can? :manage_deadlines, @course %>
             <li>

config/routes.rb

@@ -162,6 +162,7 @@
         post 'enable'
         post 'disable'
         post 'toggle_hidden'
+        post 'toggle_code_review_requests'
         get 'courses', to: 'courses#show_json', format: 'json', as: 'one_course_json'
         get 'students', to: 'courses#student_emails'
         resources :emails, only: [:index]

db/migrate/20181003103718_add_code_review_requests_enabled_to_course.rb

@@ -0,0 +1,5 @@
+class AddCodeReviewRequestsEnabledToCourse < ActiveRecord::Migration
+  def change
+    add_column :courses, :code_review_requests_enabled, :boolean, null: false, default: true
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20180912080101) do
+ActiveRecord::Schema.define(version: 20181003103718) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -116,6 +116,7 @@
     t.boolean  "hide_submissions",                         default: false, null: false
     t.boolean  "model_solution_visible_before_completion", default: false, null: false
     t.float    "soft_deadline_point_multiplier",           default: 0.75,  null: false
+    t.boolean  "code_review_requests_enabled",             default: true,  null: false
   end
 
   add_index "courses", ["organization_id"], name: "index_courses_on_organization_id", using: :btree