Ruby on Rails: Properly creating a model instance that has relation to multiple models

Question

Let's say I have a standalone User model and Service model.

I also have a Order model that holds the orders created by an user for a service. I'm wondering how I could properly create an order entry in rails.

Here is how I'll create an order entry if it refers to only one other model, say user.

@order = current_user.orders.build(params[:order])
@order.save

Now how do I do that if order refers to multiple models (user and service)?

Assume that Order model has user_id and service_id attributes and all model objects are properly tagged with belongs_to and has_many relationships.

fl00r · Accepted Answer · 2011-04-24 17:20:37Z

8

@order = Order.new(params[:order])
@order.user = current_user
@order.service = @service
@order.save

Where @service is some your fetched Service

edited Apr 24, 2011 at 17:20

answered Apr 22, 2011 at 21:06

fl00r

83.7k33 gold badges222 silver badges239 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Srini K Over a year ago

Doesn't answer the question at all.

Srini K · Accepted Answer · 2011-04-25 15:10:06Z

2

This is how I would to solve this problem, preferably the first way.

current_user.orders.create!(params[:order].merge({ :service => @service }))

or

@service.orders.create!(params[:order].merge({ :user => current_user }))

edited Apr 25, 2011 at 15:10

answered Apr 24, 2011 at 4:15

Srini K

3,39511 gold badges39 silver badges48 bronze badges

Comments

Vlad Khomich · Accepted Answer · 2011-04-22 21:27:59Z

0

I believe there's no 'magic' method for this case. But the logic is that you create an order when both user and service alredy exist, so service already holds user_id attribute and you should probably call build on service.orders:

@order = service.orders.build(params[:order].merge({ :user_id => service.user_id }))

edited Apr 22, 2011 at 21:27

answered Apr 22, 2011 at 21:15

Vlad Khomich

5,9001 gold badge30 silver badges40 bronze badges

1 Comment

Srini K Over a year ago

While having user_id in Service model is not an option for me, it does make sense to do something similar to your answer. I prefer to merge service_id with order params and let the user build the order.

Collectives™ on Stack Overflow

Ruby on Rails: Properly creating a model instance that has relation to multiple models

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related