[email protected] wrote: > From: martyntaylor <[email protected]> > > --- > src/app/controllers/provider_controller.rb | 18 ++++++++++++++++++ > src/app/models/cloud_account.rb | 19 ++++++++++++++++++- > src/features/step_definitions/pool_steps.rb | 2 +- > .../controllers/cloud_accounts_controller_spec.rb | 9 +++++++-- > src/spec/factories/cloud_account.rb | 4 ++-- > src/spec/models/cloud_account_spec.rb | 16 ++++++++++++++++ > src/spec/models/instance_spec.rb | 12 +++++++----- > src/spec/spec_helper.rb | 1 + > 8 files changed, 70 insertions(+), 11 deletions(-) > > diff --git a/src/app/controllers/provider_controller.rb > b/src/app/controllers/provider_controller.rb > index 7a4cefc..4955979 100644 > --- a/src/app/controllers/provider_controller.rb > +++ b/src/app/controllers/provider_controller.rb > @@ -67,4 +67,22 @@ class ProviderController < ApplicationController > require_privilege(Privilege::PROVIDER_VIEW, @provider) > end > > + def new_account > + @provider = Provider.find(params[:id]) > + require_privilege(Privilege::ACCOUNT_MODIFY, @provider) > + end > + > + def create_account > + require_privilege(Privilege::ACCOUNT_MODIFY) > + @acct = CloudAccount.find_or_create(params[:cloud_account]) > + @provider = Provider.find(params[:cloud_account][:provider_id]) > + if @acct.save > + @provider.cloud_accounts << @acct > + redirect_to :action => 'accounts', :id => @provider.id > + else > + flash[:notice] = @acct.errors.full_messages[0] > + redirect_to :action => 'new_account', :id => @provider.id > + end > + end > + > You should make the required changes to these actions to CloudAccountsController.new/create -- my last patch removed ProviderController.{new|create}_account since we had a redundant set of new account actions/views.
Scot _______________________________________________ deltacloud-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/deltacloud-devel
