Ruby on Rails

> Finally, your controller is much too large and complex. I am not
> going to wade through it trying to refactor it for you, but it is not
> suitable for purpose as it is. Likely large amounts of the code can
> be delegated to models, for example.
>
> Colin

Thanks colin,

I moved the method up in controller but now it gives me the error:

NoMethodError in Employee#add_rosterduty

Showing app/views/employee/add_rosterduty.html.erb where line #14
raised:

undefined method `roster_duties_path' for #<ActionView::Base:0x19dd9fc0>
Extracted source (around line #14):

11: <%=render_breadcrumbs%>
12: </div>
13:
14: <% form_for(@rosterduties) do |a| %>
15: <%= a.error_messages%>
16: <div id="admission1_form">

--
Posted via http://www.ruby-forum.com/.

--
You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-talk+unsubscribe@googlegroups.com.
To post to this group, send email to rubyonrails-talk@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/f7582a38ae300139077d8f8b300013e2%40ruby-forum.com.
For more options, visit https://groups.google.com/d/optout.