> 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+unsubscr...@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.