Statuses won't save user_id

Not sure if you got this to work but here's something that fixed it for me:

Add/Edit this to your statuses_controller.rb under App > Controllers

def status_params params.require(:status).permit(:name, :content, :user_id) end

Mine was already on line 72

Hope that helps you!

Also, you need to create a status. If there is no status (meaning @status is nil), then @status.user will also be nil, so @status.user.first_name will be giving you that error.

Brandon, I got the error TRYING to post a status. If I go back and look at the statuses, it ends up showing up. The only problem then if I try to edit the status and assign it a user, it brings the error back up. It is not assigning a user_id to a status.

class StatusesController < ApplicationController
before_action :set_status, only: [:show, :edit, :update, :destroy]

  # GET /statuses
  # GET /statuses.json
  def index
    @statuses = Status.all
    respond_to do |format|
      format.html # index.html.erb
     format.json { render json: @statuses }
    end
  end

  # GET /statuses/1
  # GET /statuses/1.json
  def show
    #@status = Status.find(params[:id])

    #respond_to do |format|
     # format.html # show.html.erb
     #format.json { render json: @status }
    #end
  end

   # GET /statuses/new
  def new
    @status = Status.new
  end

  # GET /statuses/1/edit
  def edit
  end

   # POST /statuses
  # POST /statuses.json
   def create
    @status = Status.new(status_params)

     respond_to do |format|
      if @status.save
         format.html { redirect_to @status, notice: 'Status was successfully created.' }
        format.json { render action: 'show', status: :created, location: @status }
      else
         format.html { render action: 'new' }
        format.json { render json: @status.errors, status: :unprocessable_entity }
       end
    end
  end

  # PATCH/PUT /statuses/1
  # PATCH/PUT /statuses/1.json
  def update
    respond_to do |format|
      if @status.update(status_params)
        format.html { redirect_to @status, notice: 'Status was successfully updated.' }
        format.json { head :no_content }
      else
        format.html { render action: 'edit' }
        format.json { render json: @status.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /statuses/1
  # DELETE /statuses/1.json
   def destroy
     @status.destroy
     respond_to do |format|
       format.html { redirect_to statuses_url }
      format.json { head :no_content }
     end
   end

   private
     # Use callbacks to share common setup or constraints between actions.
    def set_status
      @status = Status.find(params[:id])
    end

     # Never trust parameters from the scary internet, only allow the white list through.
    def status_params
      params.require(:status).permit(:name, :content)
    end
end

Ok, so in your create and update actions, you need to add current_user before Status... meaning:

def create
    @status = current_user.statuses.new(params[:status])
end

def update
    @status = current_user.statuses.find(params[:id])
end

On the create action, that will assign the status to the user id. Then, on the update action, it will have the user_id available in the form which seems to be causing your error.

Did not work. Either I didn't put it in the right place, or it just doesn't work with my configuration.

I still say that this tutorial is unable to be finished without any new types of configurations. I've modified it in every possible way I can, and I've re-done this tutorial 9!!!!!! TIMES IN 2-3 months! and I've yet to get passed the Creating Relationships tutorial on my version of the program. I have yet to fix this part despite using RVM, adjusting the version of bootstrap, the version of simple form, and all other things. I think that being a paying customer we should have tutorials that are at least UP-TO-DATE. I joined to learn ruby primarily. I am still wanting to learn, but there is nothing I can do with just going through the videos without actually implementing the program on my own.

I'm frustrated.... and I am just throwing my hands in the air and giving up. I don't do that often, but I've gone through EVERY SINGLE portion and morsel of code and tried to make sense of it. It doesn't work. Something about it just doesn't work. Up until Creating Relationships I've done EVERYTHING right. And all of the sudden it just doesn't work....

Ok, i don't work for treehouse, but feel your pain. I was in your shoes this past summer. It helps to ping Jason Seifer In the meantime, can you send me the link to your code on Github? I'll clone it, then try to duplicate the error on my computer.

Are you using ruby 1.9.3 and rails 3.2.12?

Ok, Mark Hipshire, you have a migration (where you changed the database) that removed the column "Name" in statuses and replaced it with a user_id. It seems you missed a step in one of the videos, hence why it is saying:

undefined method `name' for #<Status:0x00000104058048>

There is no "name" column in the database anymore. So, in your _form.html.erb, you'll need to change the

<div class="field">
<%= f.label :name %><br />
<%= f.text_field :name %>
</div>

It can't say name. You could try :user_id for now. I think that's what they have you do eventually. I would find the video where you do a migration to delete "name" from the database and watch that again closely.

If something is not working, push through, and you can always download the source files (they are near the video) and compare their files with yours.

Hope that helps!

I had this same problem since I am using Ruby 2.1.1, so I had to get rid of the attr_accessible call that the tutorial calls for.

I then had to go into app / controllers / application_controller.rb and modify it to look like this:

class ApplicationController < ActionController::Base
  # Prevent CSRF attacks by raising an exception.
  # For APIs, you may want to use :null_session instead.
  protect_from_forgery with: :exception

  before_action :configure_permitted_parameters, if: :devise_controller?

  protected

  def configure_permitted_parameters
    devise_parameter_sanitizer.for(:sign_up) { |u| u.permit(:first_name, :last_name, :profile_name, :email, :password, :password_confirmation) }
  end  
end

Then doing the following line in app / controllers / statuses_controller.rb

def status_params 
    params.require(:status).permit(:name, :content, :user_id) 
end

worked, except since we have no :name anymore I modified it to:

def status_params 
    params.require(:status).permit(:content, :user_id) 
end

This allowed it to store the user_id. After this I still kept having problems getting the name to show up, but then I went into the rails console and realized that my user_id was 5, not 1 (having created and deleted users a few times). I deleted all the users and statuses just to be safe, registered a user, verified my user_id and then I created a status using that id. After this code we were trying to get to work worked:

<%= @status.user.first_name %>

Hope that helps some.

I wanna thank everyone on this thread. It fixed all my problems with Treebook. +1000x points