[PATCH] BZ 798548 - fixed redirect on image import fail

Matt Wagner matt.wagner at redhat.com
Tue Mar 6 14:37:55 UTC 2012


On Fri, Mar 02, 2012 at 06:54:06PM +0100, jprovazn at redhat.com wrote:
> From: Jan Provaznik <jprovazn at redhat.com>
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=798548
> 
> Added missing 'environment' param
> ---
>  src/app/controllers/images_controller.rb |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/app/controllers/images_controller.rb b/src/app/controllers/images_controller.rb
> index 26f12df..9a6c7b4 100644
> --- a/src/app/controllers/images_controller.rb
> +++ b/src/app/controllers/images_controller.rb
> @@ -210,7 +210,7 @@ class ImagesController < ApplicationController
>        else
>          flash[:error] = t("images.import.image_not_imported")
>        end
> -      redirect_to new_image_url(:tab => 'import')
> +      redirect_to new_image_url(:tab => 'import', :environment => @environment)
>      end
>    end
>  
> -- 
> 1.7.7.6

ACK, assuming you have run tests. This looks good and works properly,
but I haven't run automated tests against it.

-- Matt



More information about the aeolus-devel mailing list