Hi I would just like to ask if someone can point me to the right
direction. I have an order page which will need to be split into 3 views
before saving say in controller:
def new @order = Order.new
end
def process_new @order = Order.new(params[:order]) #do validation for new
redirect to no_of_attendees
end
def no_of_attendees @order = Order.new(params[:order])
end
def process_no_of_attendees @order = Order.new(params[:order]) #do validation #redirect to save page
end
Question: Is there aright way of doing this (carry params values between
views) or should I just place the data from new in session as we move to
forward to the third page create a hash from session and save in model?
I’m not sure if this is the right way, wht i did in the same case was,
converting it to JSON and pass the params. I didn’t like to store a
large params in session and so i did like this.
def process_new @order = Order.new(params[:order])
end
In the view…
<%= hidden_field_tag “order_json”, @order.to_json %>
Question: Is there aright way of doing this (carry params values between
views) or should I just place the data from new in session as we move to
forward to the third page create a hash from session and save in model?
If all you are passing back each time is the latest data for the
order, so you can do Order.new(params[:order]) as you have shown, then
there is no reason to complicate things by using the session.
You might like to consider whether the controller (and maybe the view)
would be simplified by combining the actions (and maybe views) and
passing up and back a phase value to indicate which phase of the
operation is current.
Make sure you do the validation in the model so that at each action
you are just asking the model to validate itself, rather than putting
validation code in the controller.
If all you are passing back each time is the latest data for the
order, so you can do Order.new(params[:order]) as you have shown, then
there is no reason to complicate things by using the session.
this is fine as long as you place the attributes in the previous step in a
hidden field so it will not be lost
when you submit the form.
It appears from the OP that even this is not necessary, as all the
required data is in params[:order].
You might like to consider whether the controller (and maybe the view)
would be simplified by combining the actions (and maybe views) and
passing up and back a phase value to indicate which phase of the
operation is current.
To do this would require a hidden field as you have pointed out.
@order = Order.new(params[:order]) #do validation
If all you are passing back each time is the latest data for the
required data is in params[:order].
this step asks for order’s no_of_attendees. so we’d have params[:order] =
{:no_of_attendees => no_of_attendees} available for next step.
STEP3 #process_no_of_attendees
in this action, we already lost the name attribute for order since we didn’t
add a hidden field for this attribute.
so eventhough we use params[:order] in all the actions, it’s still needed to
add hidden_field tags for attributes
filled up in the previous actions.
I see what you mean now, I was assuming the previous data would still
be visible, but I take your point, it may not be.
Question: Is there aright way of doing this (carry params values between
views) or should I just place the data from new in session as we move to
forward to the third page create a hash from session and save in model?
My view would be that the order gets saved to the DB, and you use
State Machine functionality to track which step a user is at.
Validation alwaysonly goes in the model.
Whether you have lots of “:if => Proc.new{|order| order.state =
:processing_attendees}” or have a separate OrderAttendeeRegister model
which has_many attendees and all the associated validation.
But if you store stuff in session, or hidden fields, you are blurring
your separation (and it always ended in pain when I’ve seen it done
those ways.
HTH
This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.