Hash does not contain all form fields

[Eggman]

I'm a ROR newbie. I am developing an app to provided web access to a database. I have a DB table with 4 columns. They are id, board_id, workorder_id, and SerialNumber. I have a view that contains a form to create a row in the table. The code is as follows:
<h1>New uut</h1>

<p>Work Order
<%= Workorder.find(params[:id]).WONumber %></p>

<% form_tag :action => 'create' do %>

<p>Board Number
<%= select("uut", "board_id", Board.findall).collect { |b| [b.RSI_Number, b.id] } ) %></p>
<p><%= hidden_field_tag "workorder_id", params[:id] %></p>
<%= render artial => 'form' %>
<%= submit_tag "Create" %>
<% end %>

<%= link_to 'Back', :action => 'list' %>

When I click on Create I get an error because the workorder_id is null. The parameters passes to the controller are:
{"commit"=>"Create", "workorder_id"=>"8", "uut"=>{"SerialNumber"=>"jjj", "board_id"=>"1"}}

I can work around this problem by getting the workorder_id parameter directly and updating the Uut class before saving it, but my question is - Why isn't workorder_id include as part of the uut hash?

Thanks,

[Eggman]

After further research I am getting a better understanding of this issue. I still don't fully have my answer but will continue to work on it. Ignore this post for the moment.

Regards

[ruby-lang]

You can replace your hidden_field_tag with:

<input type="hidden" id="uut_workorder_id" name="uut[workorder_id]" value="<&#37;= params[:id] %>" />

The problem was the hidden_field_tag wasn't creating the right name for the field.

[Eggman]

Thanks. That works. I try using input with type=hidden before but I must have had something else wrong as it did not work. Thanks again.

Eggman

[ReggieB]

This might be a better solution:

<&#37;= hidden_field "uut", "workorder_id", params[:id] -%>

Also this:

Workorder.find(params[:id]).WONumber

should be done at the controller with something like:

@workorder = Workorder.find(params[:id])

and then passed to the view. The view code them becomes:

<p>Work Order <%= @workorder.WONumber %></p>

That will then allow you to catch instances where an id isn't passed to the page. At the moment such an occurance is going to cause errors.

Likewise:

Board.findall

Should be done at the controller with something like:

@boards = Board.find(:all)

and then passed to the view where the code would become:

<%= select("uut", "board_id", @boards).collect { |b| [b.RSI_Number, b.id] } ) %>

[Eggman]

Thanks for the advice. However the first code you suggested did not work as show but I change it to:

<%= hidden_field "uut", "workorder_id", :value => params[:id] -%>

and it worked. By following your next suggestion to get the workorder in the controller I was able to change it to:

<%= hidden_field "uut", "workorder_id", :value  => @workorder.id] -%>

Jim

[ReggieB]

Yep. Should have tested the syntax before I posted.

However, I think what you've ended up with will be easier to manage and reuse.