Project

General

Profile

« Previous | Next » 

Revision a387cbcb

Added by Dominic Cleal over 10 years ago

fixes #3889 - don't reload nested lookup keys during render, so validation errors aren't lost

View differences:

app/controllers/puppetclasses_controller.rb
def find_by_name
not_found and return if params[:id].blank?
@puppetclass = (params[:id] =~ /\A\d+\Z/) ? Puppetclass.find(params[:id]) : Puppetclass.find_by_name(params[:id])
pc = Puppetclass.includes(:class_params => [:environment_classes, :environments, :lookup_values])
@puppetclass = (params[:id] =~ /\A\d+\Z/) ? pc.find(params[:id]) : pc.find_by_name(params[:id])
not_found and return unless @puppetclass
end
end
app/views/puppetclasses/_form.html.erb
</ul>
<div class="tab-content span7 smart-var-content">
<%= f.fields_for :class_params, f.object.class_params.includes(:environment_classes, :environments, :lookup_values) do |lookup_keys_form| %>
<%= f.fields_for :class_params do |lookup_keys_form| %>
<%= render 'lookup_keys/fields', :f => lookup_keys_form %>
<% end %>
</div>

Also available in: Unified diff