Just wondering if there is a better dry way to do this:
I have an inventory program, that I'm writing to learn rails. It has
multiple tables, one of the main ones is called parts. Parts has name,
manufacturer, vendor, etc columns.
Manufacturer and vendor columns are just storing ids of records from
tables Manufacturers and Vendors. So when I'm creating a form in the
view for "new" action with the dropdowns I need to first get data from
those two table in my controller then iterate in my view to show all the
options in the dropdown. This is all fine. My question is - when I save
the form it complains that I have no @vendors and @manufacturers in my
create method. So I have to do @vendors = Vendor.find(:all) again.
It's not a big performance hit, since I only might have potentially 50
records in each table, but it seems to me I should be able to do
@vendors = Vendor.find(:all) once in my controller and then use that in
many actions.
Please advise. Below are snippets from my code (please excuse crudity):
The reason for this is the render :action => 'new business. You see,
when you create a part, there are 2 ways to go; either you are
succesfull or not. If you are then everything is groovy and you
redirect_to wherever, the redirect_to, disposes of all your variables
(@part, @vendors...) and calls the new action method as if it was the
first time. If its not succesfull however, we cant redirect_to because
wed loose all the data that the user put in in the first place and
restart with a blank form. So we render, which means, grab the view,
and render it as if i was that method but do not call the method. that
is why you need to initialize the variables again, thats why ive put
them in the else statement, so that they only get loaded if you are
not succesfull.
try this, remove the "duplicate" varianble assignments and create a
part that is succesfull, it wont complain.
The reason for this is the render :action => 'new business. You see,
when you create a part, there are 2 ways to go; either you are
succesfull or not. If you are then everything is groovy and you
redirect_to wherever, the redirect_to, disposes of all your variables
(@part, @vendors...) and calls the new action method as if it was the
first time. If its not succesfull however, we cant redirect_to because
wed loose all the data that the user put in in the first place and
restart with a blank form. So we render, which means, grab the view,
and render it as if i was that method but do not call the method. that
is why you need to initialize the variables again, thats why ive put
them in the else statement, so that they only get loaded if you are
not succesfull.
try this, remove the "duplicate" varianble assignments and create a
part that is succesfull, it wont complain.
Thanks "Wolas!"
It does help in this particular case. But I was also wondering in
general what's the best way or any other way to do? Globals?
orders_by, submitted_by, and approved_by - all refer to Users table -
and potenttially one person, or multiple so how do I setup association
in this case?
orders_by, submitted_by, and approved_by - all refer to Users table -
and potenttially one person, or multiple so how do I setup association
in this case?
You can add many belongs_to clauses with different names all referring
to the same class:
and then it's possible to make the following queries on a
@purchase_order instance:
@purchase_order.submitter # => the user that is referenced by
submitted_by.
The convention in rails would be for submitted_by etc... to be submitted_by_id and for the association to be submitted_by (weird shit happens if an association and a column have exactly the same name)
Fred
