ansaurus

Question

Please help rails problem with stringify_keys error

Answer 1

A:

Taking a look at the params in your trace, I think the form may be incorrect in some way, but it's hard to say how without a little more information. The params you're getting from the form right now are (reformated for legibility):

{ "commit" => "Send message",
  "_method" => "put",
  "machine_enquiry" => { 
    "messages_attributes" => {
      "message" => "2",
      "title" => "1",
      "message_type_id" => "1",
      "contact_detail_ids" => [ "1", "11" ]
    }
  },
  "id" => "2",
  "datetime" => ""
}

But they should look something like:

{ "commit" => "Send message",
  "_method" => "put",
  "machine_enquiry" => { 
    "messages_attributes" => {
      0 => {
        "message" => "2", # this may still be wrong
        "title" => "1",
        "message_type_id" => "1",
        "contact_detail_ids" => [ "1", "11" ]
      }
    }
  },
  "id" => "2",
  "datetime" => ""
}

Note that the hash that was directly inside of messages_attributes is now a level deeper. The nested attributes section of the form should contain a hash with a unique key for each message that is being created or edited. The source of the error was that the form format parser was seeing message, title, message_id, and contact_detail_ids as the unique keys for four separate messages, but the values associated with those keys weren't hashes, as it expected.

The line I've called out with a comment in the corrected example I'm a little unsure of. Without seeing the form in its entirety or the Message class, it's hard to know if that's an attribute of the message, or maybe it's supposed to be the message id. The message id, by the way, should be included only if you're editing an existing message.

There's an exhaustive explanation of the Rails forms micro-format available. Search for has_many to find the relevant section.

The source of this mis-formatted params hash is almost certainly your form_for. If you include the entirety of it, someone may be able to help you diagnose why the form is being created incorrectly.

Updated

After looking at your full form, the problem is this line:

<% me_form.fields_for :messages_attributes  do |f| %>

It should read:

<% me_form.fields_for :messages  do |f| %>

The fields_for helper expects the association name in order to act correctly. Another problem I saw while looking at the form is the radio buttons that you use for contacts will need to have the name changed as well to something like machine_enquiry[messages_attributes][0][contact_detail_ids][]. You'll probably need to find a way to generate those radio buttons using the form helper so they can get the correct message id.

Emily 2009-10-12 22:20:41

This should work, but I was having the same situation before 2 days and ur suggested line didn't convert <% me_form.fields_for :messages do |f| %> to machine_enquiry[messages_attributes][0][contact_detail_ids][] but to machine_enquiry[messages][contact_detail_ids][], there is something wrong somewhere

khelll 2009-10-13 03:59:41

Answer 2

A:

thank you for your help so far here is my full form :

  <p>
    <h1>Create message</h1>
  </p>

  <% form_for @machine_enquiry, machine_enquiry_path(@machine_enquiry) do|me_form| %>

    <div class="form_full_holder span-18 last">
      <%= me_form.error_messages %>
    </div>

    <% me_form.fields_for :messages_attributes  do |f| %>


      <%= f.hidden_field :in, :value=>1 unless is_admin%>

      <div class="form_full_holder span-18 last">
        <div class="form_full_label span-5">
        Title <%= "| type | reminder" if is_admin %>
       </div>
         <div class="form_full_label span-13 last">
          <%= f.text_field :title -%>
        <%= f.collection_select(:message_type_id, MessageType.all, :id, :thetype, {}, {:class=>"select_normal"}) if is_admin -%>
        <% if is_admin %>
         <input type="text" name="datetime" id="datetime" value="" class="btn_normal">
        <% end %>
       </div>
      </div>

      <div class="form_full_holder span-18 last">
       <div class="infiniteCarousel">
       <div class="wrapper">
       <ul>
         <% @contact_details.each do |contact| %>
          <li>
           <%= radio_button_tag "machine_enquiry[messages_attributes[contact_detail_ids][]", contact.id, false, :class=>"checkbox_normal" %> <%= " Contact : " + contact.name %>
           <%= "<br>Email : " + contact.email %>

          </li>
         <% end %>
       </ul> 
       </div>
       <a class="arrow back"></a>
       <a class="arrow forward"></a>
      </div>
     </div>

     <div class="form_full_holder span-18 last">
        <%= f.text_area :message %>
      </div>

      <div class="form_full_holder span-18 last">
       <div class="infiniteCarousel">
       <div class="wrapper">
       <ul>
        <% @companies.each do |company| %>

           <% company.contact_details.each do |contact| %>
            <li>
             <%= check_box_tag "machine_enquiry[messages_attributes[contact_detail_ids][]", contact.id, false, :class=>"checkbox_normal" %> 

             <% if is_admin %>
              <%= " Buyer : " + company.name %>
             <% else %>
              <%= "Seller : " + company.name %>
             <% end %>
             <%= "<br>Contact : " + contact.name %>
             <%= "<br>Phone : " + contact.phone %>
             <%= "<br>Email : " + contact.email %>
            </li>
           <% end %>

         <% end %>

         <% if is_admin %>
          <% @suppliers.each do |company| %>


            <% company.contact_details.each do |contact| %>
             <li>
              <%= check_box_tag "machine_enquiry[messages_attributes[contact_detail_ids][]", contact.id, false, :class=>"checkbox_normal" %> <%= " Supplier : " + company.name %>
              <%= "<br>Contact : " + contact.name %>
              <%= "<br>Phone : " + contact.phone %>
              <%= "<br>Email : " + contact.email %>
             </li>
            <% end %>


          <% end %>
        <% end %>
       </ul> 
       </div>
       <a class="arrow back"></a>
       <a class="arrow forward"></a>
      </div>
     </div>

    <% end  %>

    <div class="form_full_holder span-18 last">
       <div class="form_full_label span-5">
      &nbsp;
      </div>
        <div class="form_full_label span-13 last">
         <%= me_form.submit 'Send message' %>

      </div>
     </div>


  <% end %>

and here is the message model. Message as you didnt know about is also an attribute in message model.

class Message < ActiveRecord::Base
    belongs_to :user
    belongs_to :enquiry
    belongs_to :messagable, :polymorphic => true

    has_many :distributes, :dependent => :destroy
    has_many :contact_details, :through=>:distributes

    validates_presence_of :title
    validates_presence_of :message


end

hope you can solve this problem for me.

thank you rick

richard moss 2009-10-12 22:29:53

BTW, the best thing to do would probably be to update your original question with this information instead of adding a comment.

Emily 2009-10-12 23:40:36

Answer 3

A:

I'm having a similar situation with polymorphic case, however Have a look at this great example: Complex form Examples.

khelll 2009-10-13 04:01:13

ansaurus

tags:

views:

answers:

Please help rails problem with stringify_keys error

related questions