> 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.

Reply via email to