Re: [PATCH conductor] Image import - rake task fix

[Jason Guiditta]

On Fri, 2011-04-01 at 17:54 +0200, jprovazn(a)redhat.com wrote:
>  From: Jan Provaznik <jprovazn(a)redhat.com&gt;
>  
>  This patch fixes bug in image import rake task - rake task used old provider
>  account username/password model.
>  ---
>   src/app/models/image.rb |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>  
>  diff --git a/src/app/models/image.rb b/src/app/models/image.rb
>  index 504a189..923dd23 100644
>  --- a/src/app/models/image.rb
>  +++ b/src/app/models/image.rb
>  @@ -232,7 +232,8 @@ class Image < ActiveRecord::Base
>         raise "There is not provider with name '#{providername}'"
>       end
>   
>  -    account = ProviderAccount.new(:provider => provider, :username => username,
> :password => password)
>  +    account = ProviderAccount.new(:provider => provider)
>  +    account.credentials_hash = {:username => username, :password => password}
>   
>       unless account.valid_credentials?
>         raise "Invalid credentials for provider '#{providername}'"
ACK and pushed, this makes import work again with rake.