Creating Two Models in One Form (Page 2 of 6)

Does build work with a belongs_to and has_one relationship?  In my model a user has_one :provider and a provider belongs_to :user.

When I try to follow the tutorial I substitute the singular for the plural to reflect the has_one instead of has_many e.g.

@provider = @user.provider.build(params[:provider])

@provider = @user.providers.build(params[:provider])

Any suggestions?

I am surprised no one has mentioned this. The view can also be coded using fields_for. This has the advantage of being much DRYer/cleaner for longer forms.

# in the projects/new.rhtml
<h1>New Project</h1>
<%= error_messages_for :project %>
<%= error_messages_for :task %>
<%= form_for :project, :url => { :action => 'create'} %>
<p>
Project Name:
<%= f.text_field :name %>
</p>
<%= fields_for :tasks do |detail| do %>
<p>
 First Task:
 <%= f.text_field :name %>
</p>
<%= end %>
<p>
  <%= submit_tag 'Create' %>
</p>
<%= end %>

awesome. but in the controller i just did this.
<code>
def create
@tasks = Task.new(params[:task])
@tasks.project_id = params[:project]
@tasks.save
end
</code>

I think this makes more sense to me, being a noob . remember if you are using drop down menues
<code><%= collection_select("project", "id" , Project.find(:all), "id", "title") %></code>

then in your controller u can just add

<code>
@tasks.project_id = params[:project][:id]
</code>

i donno if its useful or not, just that usually when you are choosing categories, its common to use drop down menus especially if its really long.

Thanks to this tutorial, I have conquored a problem I've been having for a week or so. I am faced with another problem. I am working with 3 models and one controller. I have figured out how to display in my list method one of the values associated with my main model. however I can't get the values from the 2nd model displayed. In the list i can get the category displayed but i can't get anything from the menuItemSize table displayed but the name of the model. here is my code in snippets..Thank you..

MODELS
=========================

class MenuItem < ActiveRecord::Base
    belongs_to :category
    has_many :menu_item_sizes
    validates_associated :menu_item_sizes
    validates_presence_of :description, :name
end
class MenuItemSize < ActiveRecord::Base
    belongs_to :menu_item
    
    validates_presence_of :price, :size
    validates_numericality_of :price, :only_integer => true
end
class Category < ActiveRecord::Base
    has_many :menu_items
    validates_presence_of :category
end

 def create
    @menu_item = MenuItem.new(params[:menu_item])
    @menu_item_size = @menu_item.menu_item_sizes.build(params[:menu_item_size])
    
    if @menu_item.save
     flash[:notice] = 'a new menu item was successfully created.'
      redirect_to :action => 'list'
    else
      render :action => 'new'
    end
  end
def list
    @menu_item_pages, @menu_items = paginate :menu_items, :per_page => 10
    
  end

<h1>Listing menu_items</h1>
<table border='1' >
  <tr>
    <th>menu Item</th>
    <th>Description</th>
    <th>Category</th>
    <th>size</th>
    <th>price</th>
  </tr>
  
<% for menu_item in @menu_items %>
  <tr>
    <td><%=h(menu_item.name)%></td>
    <td><%=h(menu_item.description)%></td>
    <td><%=h(menu_item.category.category)%></td>
    <td><%=h(menu_item.menu_item_sizes.price)%></td>
   
    
    <td><%= link_to 'Show', :action => 'show', :id => menu_item %></td>
    <td><%= link_to 'Edit', :action => 'edit', :id => menu_item %></td>
    <td><%= link_to 'Destroy', { :action => 'destroy', :id => menu_item }, :confirm => 'Are you sure?', :method => :post %></td>
  </tr>
<% end %>
</table>
<%= link_to 'Previous page', { :page => @menu_item_pages.current.previous } if @menu_item_pages.current.previous %>
<%= link_to 'Next page', { :page => @menu_item_pages.current.next } if @menu_item_pages.current.next %> 
<br />
<%= link_to 'New menu_item', :action => 'new' %>

The problem is you are calling "prices" on an array of menu item sizes. A given menu item can have many sizes, so Rails doesn't know what to do when you call price on many:

<td><%=h(menu_item.menu_item_sizes.price)%></td>

<td><%=h(menu_item.menu_item_sizes.first.price)%></td>

<% for menu_item_size in menu_item.menu_item_sizes %>
  <%= h(menu_item_size.price) %>
<% end %>

How is recipe related to category? And are you certain Recipe is passing validation? Lastly, what version of Rails are you using?

That's documented in the has_many method.

Hi,

I am confused about the lack of transactions here. The example in Pragmatic Rails uses transactions for the save part. Is that redundant or just another way of doing this?

Thanks.

Thank-you for this tutorial.

I'm currently editing two models in one form where the models have a has_one/belongs_to relationship (Account has_one AgentDetail. There is validation on both models, however validation errors are not appearing for AgentDetail in the form. Validation is working though, since an account is saved without an associated AgentDetail when AgentDetail fails validation.

My code:

#Accounts Controller
#########
def create
 @account = Account.new(params[:account])
 @agent_detail = @account.build_agent_detail(params[:agent_detail])     
 if @account.save
   redirect_to some_path
 else
   render :action => 'new'      
 end
end

class AgentDetail < ActiveRecord::Base
  belongs_to :account
  validates_presence_of   :agency_name
end
class Account < ActiveRecord::Base
  has_one  :agent_detail
  validates_presence_of     :login, :email
end

<%= error_messages_for :account, :agent_detail %>
<% form_for :account, :url => accounts_path do |f| -%>
  <legend>Account Details</legend>
  <p><label for="login">Login</label><br/>
  <%= f.text_field :login %></p>
  .... 
  
  <% fields_for :agent_detail do |d|  %>    
  ....
  <legend>Agent Details</legend>
  <p><label for="agency_name">Agency name</label><br/>
  <%= d.text_field :agency_name %></p>
  ....

Many thanks