Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Undefined method 'push_relation' #62

Open
tacticiankerala opened this issue Jun 26, 2013 · 5 comments
Open

Undefined method 'push_relation' #62

tacticiankerala opened this issue Jun 26, 2013 · 5 comments

Comments

@tacticiankerala
Copy link

The method push_has_many inside model.rb is calling a method called 'push_relation' but which is not defined at all.

def push_has_many_attr(col, *instances)
    ...
      push_relation(_col, *_collection)
    ...
end

I faced this issue while trying to populate a restaurant model which has many food items. I will try to create a failing spec for the same.

@sxross
Copy link
Owner

sxross commented Jun 26, 2013

Can you include your Gemfile as a comment?

@tacticiankerala
Copy link
Author

My Gemfile,

gem "ProMotion", "~> 0.5.0"
gem "bubble-wrap", "~> 1.3.0"
gem "cocoapods"
gem "motion-cocoapods"
gem "motion-testflight"
gem 'motion-dryclean'
gem 'teacup'
gem 'sugarcube'
gem 'sweettea'
gem "motion_model", :git => "https://github.com/tacticiankerala/MotionModel.git"
group :spec do
 gem "motion-facon"
 gem "webstub", "~> 0.3.0" 
end

PS: I forked motion_model yesterday to my git repository and used it because of the issue( #60 )

@sxross
Copy link
Owner

sxross commented Jun 27, 2013

@aceofspades -- this changed in commit ffe43c2. Look at model.rb:709. Can you remember what the logic was behind this? Thanks!

@DougPuchalski
Copy link
Contributor

Unfortunately looks like there were several stacked changes in there so it might be challenging to work backward. Maybe it's best to fix this one the spec is in place.

On Jun 26, 2013, at 5:15 PM, "s.ross" [email protected] wrote:

@aceofspades -- this changed in commit ffe43c2. Can you remember what the logic was behind this? Thanks!


Reply to this email directly or view it on GitHub.

@DougPuchalski
Copy link
Contributor

@tacticiankerala Let us know when you're able to spec this up, thanks for the help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants