CODEWITHSUNDEEP
ruby-on-railsnested-formsnested-attributes
manafire
manafire

Reputation: 6084

RecordNotFound with accepts_nested_attributes_for and belongs_to

I get

ActiveRecord::RecordNotFound: Couldn't find Client with ID=3 for Order with ID=

when trying to submit an Order form for an existing client. This happens through the form or the console by typing: 

Order.new(:client_attributes => { :id => 3 })

payment_form.html.erb:

<%= semantic_form_for @order, :url => checkout_purchase_url(:secure => true) do |f| %>

        <%= f.inputs "Personal Information" do %>

            <%= f.semantic_fields_for :client do |ff| %>
                <%= ff.input :first_name %>
                <%= ff.input :last_name %>              
                <!-- looks like semantic_fields_for auto-inserts a hidden field for client ID -->
            <% end %>

        <% end %>
<% end %>

Order.rb:

class Order < ActiveRecord::Base
  belongs_to :client
  accepts_nested_attributes_for :client, :reject_if => :check_client

  def check_client(client_attr)
    if _client = Client.find(client_attr['id'])
      self.client = _client
      return true
    else
      return false
    end    
  end
end

The reject_if idea came from here but I logged the method and it's not even being called! It doesn't matter what its name is!

Upvotes: 6

Views: 3636

Answers (4)

manafire
manafire

Reputation: 6084

Note: Feb 2020

Since I'm starting to get downvotes on this 8 years later, adding this note. While this was the original solution I went with 8 years ago, a better one has been proposed by MatayoshiMariano (5 years after my OP).

My Original Fix

Fixed the issue by overloading the client_attributes= method, as described here:

  def client_attributes=(client_attrs)    
    self.client = Client.find_or_initialize_by_id(client_attrs.delete(:id))
    self.client.attributes = client_attrs
  end

Upvotes: 7

MatayoshiMariano
MatayoshiMariano

Reputation: 2116

If you only want a new Order with an existing client, without modifying the client, you need to assign the id.

Order.new(client_id: 3)

This is another way to do this without overloading the client_attributes= method and cleanest

The new Order now has the client with ID 3

If you also want to update ant client's attributes you must add the client_attributes, for example:

Order.new(client_id: 3, client_attributes: { id: 3, last_order_at: Time.current })

See https://github.com/rails/rails/issues/7256 from 2012.

Upvotes: 4

Henry Jacob
Henry Jacob

Reputation: 101

If you have has_many relationship, this will work. Tested on Rails 6.0.2

  def clients_attributes =(attributes)
    # Get IDs for any clients that already exist.
    client_ids = attributes.values.map { |a| a[:id] }.compact

    # Now find them all and move them to this section.
    clients << Client.find(client_ids)

    # Update them with standard `accepts_nested_attributes_for` behaviour.
    super attributes
  end

Upvotes: 1

Dende
Dende

Reputation: 594

Had the same error creating a new Thing for existing model with has_many and belongs_to relations.

Fixed it by adding a hidden field for the id of the existing model, for instance User, to the form.

= form.input :user_id, as: :hidden

Then new Thing was created without the error.

Upvotes: 0

Related Questions