[PATCH conductor] BZ 804697 - User status: Inactive option does not function

Jirka Tomasek jtomasek at redhat.com
Tue Aug 28 15:14:01 UTC 2012


On 08/22/2012 10:31 AM, jzigmund at redhat.com wrote:
> From: Jozef Zigmund <jzigmund at redhat.com>
>
> User status radio buttons were removed, because it was fake
> (the attribute didn't exist in User model).
>
> https://bugzilla.redhat.com/show_bug.cgi?id=804697
> ---
>   src/app/views/users/_form.html.haml | 11 -----------
>   1 file changed, 11 deletions(-)
>
> diff --git a/src/app/views/users/_form.html.haml b/src/app/views/users/_form.html.haml
> index 9ac1a80..3185ab0 100644
> --- a/src/app/views/users/_form.html.haml
> +++ b/src/app/views/users/_form.html.haml
> @@ -36,17 +36,6 @@
>           .input
>             = quota_form.text_field :maximum_running_instances
>   
> -  %fieldset
> -    %legend=t'users.form.user_status'
> -    .field
> -      %ul.radio-group
> -        %li
> -          = label_tag "user_status_active", t("users.form.user_status_active")
> -          = radio_button_tag "user_status", t("users.form.user_status_active"), true
> -        %li
> -          = label_tag "user_status_inactive", t("users.form.user_status_inactive")
> -          = radio_button_tag "user_status", t("users.form.user_status_inactive"), false, :disabled => true
> -
>   %fieldset.options
>     = form.submit "#{t'users.form.save_user'}",  :class => "submit button pill"
>   
Conditional ACK, please also remove related string translations which 
are not used any more. (user_status_active, user_status_inactive)

Jirka



More information about the aeolus-devel mailing list