From: Jan Provaznik <jprovazn(a)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>
--
1.7.0.1