Browse Source

rubocop

pull/40/head
Kat 4 years ago
parent
commit
9d3d5e3ab8
  1. 4
      app/controllers/case_logs_controller.rb
  2. 2
      config/routes.rb
  3. 1
      db/migrate/20211007160908_add_accessibility_fields.rb

4
app/controllers/case_logs_controller.rb

@ -27,7 +27,7 @@ class CaseLogsController < ApplicationController
questions_for_page = form.questions_for_page(previous_page) questions_for_page = form.questions_for_page(previous_page)
checked_answers = get_checked_answers(params[:case_log], questions_for_page) checked_answers = get_checked_answers(params[:case_log], questions_for_page)
answers_for_page = page_params(questions_for_page.keys).select { |k, _v| questions_for_page.keys.include?(k) } answers_for_page = page_params(questions_for_page.keys).select { |k, _v| questions_for_page.key?(k) }
if @case_log.update(checked_answers) && @case_log.update(answers_for_page) if @case_log.update(checked_answers) && @case_log.update(answers_for_page)
redirect_path = form.next_page_redirect_path(previous_page) redirect_path = form.next_page_redirect_path(previous_page)
redirect_to(send(redirect_path, @case_log)) redirect_to(send(redirect_path, @case_log))
@ -41,7 +41,7 @@ class CaseLogsController < ApplicationController
checked_questions = {} checked_questions = {}
checkbox_questions = questions_for_page.select { |_title, question| question["type"] == "checkbox" } checkbox_questions = questions_for_page.select { |_title, question| question["type"] == "checkbox" }
checkbox_questions.each do |title, question| checkbox_questions.each do |title, question|
valid_answer_options = question["answer_options"].select {|key, _value| !key.match?(/divider/) } valid_answer_options = question["answer_options"].reject { |key, _value| key.match?(/divider/) }
valid_answer_options.each do |_key, value| valid_answer_options.each do |_key, value|
checked_questions["#{title}_#{value.parameterize(separator: '_')}"] = case_log_params[title].include?(value) ? true : false checked_questions["#{title}_#{value.parameterize(separator: '_')}"] = case_log_params[title].include?(value) ? true : false
end end

2
config/routes.rb

@ -3,7 +3,7 @@ Rails.application.routes.draw do
get "about", to: "about#index" get "about", to: "about#index"
get "/", to: "test#index" get "/", to: "test#index"
post '/case_logs/:id', to: "case_logs#submit_form" post "/case_logs/:id", to: "case_logs#submit_form"
form = Form.new(2021, 2022) form = Form.new(2021, 2022)
resources :case_logs do resources :case_logs do

1
db/migrate/20211007160908_add_accessibility_fields.rb

@ -11,4 +11,3 @@ class AddAccessibilityFields < ActiveRecord::Migration[6.1]
end end
end end
end end

Loading…
Cancel
Save