ruby-on-railsactiverecordhidden-fieldafter-create

Rails: Associating an Object After Creation


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

Solution

  • 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.