I am trying to create a site that will show step by step instructions. A user will view a question and select answer. The answer view is shown below:
<p id="notice"><%= notice %></p>
<p>
<strong>Post:</strong>
<%= @question.post %>
</p>
<%= link_to 'Answer', new_step_path(:question_id=>@question.id) %> |
<%= link_to 'Edit', edit_question_path(@question) %> |
<%= link_to 'Back', questions_path %>
When the user selects "answer", I redirect to Step#new which renders the Step form.
<%= form_for(@step) do |f| %>
<% if @step.errors.any? %>
<div id="error_explanation">
<h2><%= pluralize(@step.errors.count, "error") %> prohibited this step from being saved:</h2>
<ul>
<% @step.errors.full_messages.each do |msg| %>
<li><%= msg %></li>
<% end %>
</ul>
</div>
<% end %>
<div class="actions">
<%= @question.id %>
<%= f.hidden_field :question_id, :value => @question.id %>
</div>
<div class="field">
<%= f.label :post %><br>
<%= f.text_field :post %>
</div>
<div class="actions">
<%= f.submit %>
</div>
<% end %>
I pass the related question from the URL and then into a hidden field.
Given that Steps has_many :questions, :through=>:instruction, how do I insert the hidden field values into the Instructions model after the Steps controller creates the Step?
class StepsController < ApplicationController
before_action :set_step, only: [:show, :edit, :update, :destroy]
# GET /steps
# GET /steps.json
def index
@steps = Step.all
end
# GET /steps/1
# GET /steps/1.json
def show
end
# GET /steps/new
def new
@step = Step.new
@question = Question.find(params[:question_id])
end
# GET /steps/1/edit
def edit
end
# POST /steps
# POST /steps.json
def create
@step = Step.new(step_params)
respond_to do |format|
if @step.save
@instruction = Instruction.create(:question_id=>@question, :step_id=>@step, :order=>1)
format.html { redirect_to @step, notice: 'Step was successfully created.' }
format.json { render action: 'show', status: :created, location: @step }
else
format.html { render action: 'new' }
format.json { render json: @step.errors, status: :unprocessable_entity }
end
end
end
# PATCH/PUT /steps/1
# PATCH/PUT /steps/1.json
def update
respond_to do |format|
if @step.update(step_params)
format.html { redirect_to @step, notice: 'Step was successfully updated.' }
format.json { head :no_content }
else
format.html { render action: 'edit' }
format.json { render json: @step.errors, status: :unprocessable_entity }
end
end
end
# DELETE /steps/1
# DELETE /steps/1.json
def destroy
@step.destroy
respond_to do |format|
format.html { redirect_to steps_url }
format.json { head :no_content }
end
end
private
# Use callbacks to share common setup or constraints between actions.
def set_step
@step = Step.find(params[:id])
end
# Never trust parameters from the scary internet, only allow the white list through.
def step_params
params.require(:step).permit(:post)
end
end
Although I asked you for the model relations it is still not clear how the three models are related to each other (you only mentioned: Step has_many :questions, :through=>:instruction). Anyway I answer your question based on my assumptions. So be careful:
The models:
and now your steps_controller.rb:
First of all: where is the @question instantiated in your code?
That line also is very confusing from REST point of view:
Why should a StepsController#create do create an Instruction?
If you can not handle it in another way, put it into a Step model callback. You will want it also from the transactional point of view ;)
That's why your action should more look like:
therefore the Step model:
I think you get the idea.