[PATCH conductor] BZ#785223 Limit export instances scope

Matt Wagner matt.wagner at redhat.com
Tue Feb 7 18:54:57 UTC 2012


On Tue, Feb 07, 2012 at 03:42:02PM +0100, tsedovic at redhat.com wrote:
> From: Tomas Sedovic <tomas at sedovic.cz>
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=785223
> 
> When exporting instances from the Deployments Details page, only the instances
> belonging to that deployment should be included in the CSV.
> ---
>  src/app/views/instances/_list.html.haml |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/app/views/instances/_list.html.haml b/src/app/views/instances/_list.html.haml
> index e120e68..675efc2 100644
> --- a/src/app/views/instances/_list.html.haml
> +++ b/src/app/views/instances/_list.html.haml
> @@ -1,5 +1,5 @@
>  - content_for :form_header do
> -  %li= link_to "#{t'instances.list.export_events'}", export_events_instances_path, :class => 'button'
> +  %li= link_to "#{t'instances.list.export_events'}", export_events_instances_path(:deployment_id => @deployment), :class => 'button'
>    %li= restful_submit_tag "#{t'instances.list.stop_selected_instances'}", 'stop', multi_stop_instances_path, 'GET', :class => 'button', :id => 'stop_selected_instances'
>    %li= restful_submit_tag "#{t'instances.list.reboot_selected_instances'}", 'reboot', multi_reboot_instances_path, 'GET', :class => 'button', :id => 'reboot_selected_instances'
>    = hidden_field_tag 'backlink', request.url
> -- 
> 1.7.6.5

ACK. Since it's past the end of day for you and we're approaching code
freeze, I went ahead and pushed this to the staging repo as well. I'll
mark the BZ as modified momentarily.

-- Matt



More information about the aeolus-devel mailing list