Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

One-to-One: Undefined method build

got a problem with one to one relations

I have some Matches and i want to have one score for a match.

my Match.rb

has_one :score, :dependent => :destroy

my score.rb

belongs_to :match

my scores_controller.rb

def new
@match = Match.find(params[:match_id])
@score = @match.score.new
end

def create
@match = Match.find(params[:match_id])
@score = @match.score.create(params[:score])
end

my routes.rb

resources :matches do
resources :scores
end

my scores/new.html.haml

= form_for([@match, @match.score.build]) do |f|
    = f.label :score1
    = f.text_field :score1
    %br
    = f.label :score2
    =f.text_field :score2
    %br
    = f.submit

my error that i get

undefined method `new' for nil:NilClass

i haven't worked with one to one relations so far since I'm pretty new to RoR, any suggestions?

EDIT

edited my code to match create_score and build_score, seems to work. but now i have some kind of strange behavior.

in my score.rb

attr_accessible :score1, :score2

but when i try to invoke in my matches/show.html.haml

= @match.score.score1

i get an unknown method call or i don't see anything at all... but if i just call

= @match.score

i get an score object returned (e.g. #)#

EDIT 2

Fix'd problem. I was calling

scores/new.haml.html

= form_for([@match, @match.create_score])

needs to be

= form_for([@match, @match.build_score])

everything works as intended.

needed to enter rails console and fetch those objects to see every :score1 :score2 was nil

like image 391
cschaeffler Avatar asked Feb 29 '12 11:02

cschaeffler


3 Answers

Use build instead of new:

def new
    @match = Match.find(params[:match_id])
    @score = @match.build_score
end

Here are the docs for this: http://guides.rubyonrails.org/association_basics.html#belongs_to-build_association

Similarly, in the create method, do it like this:

def create
    @match = Match.find(params[:match_id])
    @score = @match.create_score(params[:score])
end

Docs for this: http://guides.rubyonrails.org/association_basics.html#belongs_to-create_association

like image 140
Ben Lee Avatar answered Nov 12 '22 09:11

Ben Lee


You should be doing match.build_score. This is because when you call the score method it would attempt to fetch the association, and because it's not defined yet, it will return nil. Then you call build on nil and that's why it blows up.

has_many association methods return a sort of "proxy" object to the objects returned by calls to them, so this is why things like posts.comments.build works. The methods for belongs_to and has_one associations attempt to fetch the associations straight off, and so you need to do build_association rather than association.build.

like image 45
Ryan Bigg Avatar answered Nov 12 '22 09:11

Ryan Bigg


You can create a score by using the below example

@match.build_score
or
@match.create_score
like image 5
sangeethkumar Avatar answered Nov 12 '22 09:11

sangeethkumar