From: Jan Provaznik jprovazn@redhat.com
--- src/app/controllers/provider_controller.rb | 12 ++++++++++-- src/app/views/provider/_form.html.erb | 2 +- 2 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/src/app/controllers/provider_controller.rb b/src/app/controllers/provider_controller.rb index 7422c95..8ead3f3 100644 --- a/src/app/controllers/provider_controller.rb +++ b/src/app/controllers/provider_controller.rb @@ -35,10 +35,18 @@ class ProviderController < ApplicationController def new require_privilege(Privilege::PROVIDER_MODIFY) @provider = Provider.new(params[:provider]) - if request.post? && @provider.set_cloud_type && - @provider.save && @provider.populate_hardware_profiles + condormatic_classads_sync + end + + def create + require_privilege(Privilege::PROVIDER_MODIFY) + @provider = Provider.new(params[:provider]) + if @provider.set_cloud_type && @provider.save && + @provider.populate_hardware_profiles flash[:notice] = "Provider added." redirect_to :action => "show", :id => @provider + else + render :action => "new" end condormatic_classads_sync end diff --git a/src/app/views/provider/_form.html.erb b/src/app/views/provider/_form.html.erb index 6058894..789f34b 100644 --- a/src/app/views/provider/_form.html.erb +++ b/src/app/views/provider/_form.html.erb @@ -2,7 +2,7 @@ <%= error_messages_for 'provider' %>
<h2>Add a cloud provider</h2><br /> - <% form_tag :controller => :provider, :action => 'new' do %> + <% form_tag :controller => :provider, :action => 'create' do %> <ul> <li><label>Name<span>Provide a descriptive name for this provider connection.</span></label><%= text_field :provider, :name, :class => "txtfield" %></li> <li><label>URL<span>Enter the URL of the cloud provider.</span></label><%= text_field:provider, :url, :class => "txtfield" %></li>