Rails: Associating an Object After Creation

78 views Asked by At

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
1

There are 1 answers

3
Christian Rolle On BEST ANSWER

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:

class Step < ActiveRecord::Base
  belongs_to :instruction
  has_many :questions, 
    through: :instruction
end

class Instruction < ActiveRecord::Base
  has_many :steps
  has_many :questions
end

class Question < ActiveRecord::Base
  belongs_to :instruction
end

and now your steps_controller.rb:
First of all: where is the @question instantiated in your code?

@instruction = Instruction.create(:question_id=>@question, :step_id=>@step, :order=>1)

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:

def create
  @step = Step.new(step_params)
  respond_to do |format|
    if @step.save
      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

therefore the Step model:

class Step < ActiveRecord::Base
  belongs_to :instruction
  has_many :questions, 
    through: :instruction
  attr_accessor :question_id
  before_create :create_related_instruction

  private
  def create_related_instruction
    self.create_instruction question_id: question_id, order: 1
  end
end

I think you get the idea.