[PATCH] BZ840882 fix indeterminate name column

Matt Wagner matt.wagner at redhat.com
Tue Jul 31 17:48:23 UTC 2012


On Mon, Jul 30, 2012 at 11:30:59AM -0400, Tzu-Mainn Chen wrote:
> ---
>  src/app/controllers/logs_controller.rb |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/app/controllers/logs_controller.rb b/src/app/controllers/logs_controller.rb
> index 06ff76a..72d0125 100644
> --- a/src/app/controllers/logs_controller.rb
> +++ b/src/app/controllers/logs_controller.rb
> @@ -212,7 +212,7 @@ class LogsController < ApplicationController
>                                t('logs.options.instance_event_type')]
>        @pool_options = [[t('logs.options.default_pools'), ""]]
>        PoolFamily.list_for_user(current_session, current_user, Privilege::VIEW).
> -        find(:all, :include => :pools, :order => "name",
> +        find(:all, :include => :pools, :order => "pool_families.name",
>               :select => ["id", "name"]).each do |pool_family|
>          @pool_options << [pool_family.name,
>                            "pool_family:" + pool_family.id.to_s]
> @@ -221,7 +221,7 @@ class LogsController < ApplicationController
>        end
>        @provider_options = [[t('logs.options.default_providers'), ""]]
>        Provider.list_for_user(current_session, current_user, Privilege::VIEW).
> -        find(:all, :include => :provider_accounts, :order => "name",
> +        find(:all, :include => :provider_accounts, :order => "providers.name",
>               :select => ["id", "name"]).each do |provider|
>          @provider_options << [provider.name, "provider:" + provider.id.to_s]
>          @provider_options += provider.provider_accounts.
> -- 
> 1.7.6.5

ACK! Looks good!

-- Matt



More information about the aeolus-devel mailing list