[PATCH conductor] BZ797208: Enable push all button on autoupdate

Petr Blaho pblaho at redhat.com
Mon Mar 5 14:23:45 UTC 2012


On Monday, March 05, 2012 02:00:23 PM ifarkas at redhat.com wrote:
> From: Imre Farkas <ifarkas at redhat.com>
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=797208
> ---
>  src/app/stylesheets/layout.scss          |    5 ++++-
>  src/app/views/images/show.html.haml      |   13 +++++++++++--
>  src/public/javascripts/backbone/views.js |   14 ++++++++++++++
>  3 files changed, 29 insertions(+), 3 deletions(-)
> 
> diff --git a/src/app/stylesheets/layout.scss b/src/app/stylesheets/layout.scss
> index c58d93a..da3f9d9 100644
> --- a/src/app/stylesheets/layout.scss
> +++ b/src/app/stylesheets/layout.scss
> @@ -2920,6 +2920,10 @@ ul.catalog_pretty {
>  /********************************************* Image show view */
>  /************************************************************************** */
>  
> +#push-all-btn {
> +  display: inline;
> +}
> +
>  ul.image_builds{
>    border: 1px solid #ccc;
>    -webkit-border-radius: 7px 7px 7px 7px; -moz-border-radius: 7px 7px 7px 7px; border-radius: 7px 7px 7px 7px;
> @@ -2972,7 +2976,6 @@ ul.image_builds{
>    }
>  }
>  
> -
>  /* ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>  Flat Table Component -- v.0.0.1 [flat_table] (flat_table.scss)
>  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ */
> diff --git a/src/app/views/images/show.html.haml b/src/app/views/images/show.html.haml
> index fe9faaa..38e34cf 100644
> --- a/src/app/views/images/show.html.haml
> +++ b/src/app/views/images/show.html.haml
> @@ -45,8 +45,10 @@
>            - if @account_groups.size > 1
>              - build_text << '_all'
>            = button_to t(build_text), rebuild_all_image_path(@image.id), :class => 'button pill'
> -          - if @build and @build.id == @latest_build
> -            = button_to t('.push_all'), push_all_image_path(@image.id, :build_id => @build.id), :class => 'button pill'
> +          #push-all-btn
> +            - if @build and @build.id == @latest_build
> +              = button_to t('.push_all'), push_all_image_path(@image.id, :build_id => @build.id), :class => 'button pill'
> +
>    .content
>      %ul.image_builds
>        - @account_groups.each do |driver, group|
> @@ -257,3 +259,10 @@
>          </li>
>        {{/if}}
>      {{/each}}
> +
> +%script#pushAllButtonTemplate{ :type => 'text/x-jquery-tmpl' }
> +  :plain
> +    #{path = push_all_image_path('replace_image_id', :build_id => 'replace_build_id')
> +      path = path.sub('replace_image_id', '${image[\'uuid\']}')
> +      path = path.sub('replace_build_id', '${build[\'uuid\']}')
> +      button_to(t('images.show.push_all'), path, :class => 'button pill')}
> diff --git a/src/public/javascripts/backbone/views.js b/src/public/javascripts/backbone/views.js
> index 2d6da0d..6a1d823 100644
> --- a/src/public/javascripts/backbone/views.js
> +++ b/src/public/javascripts/backbone/views.js
> @@ -208,5 +208,19 @@ Conductor.Views.ImagesShow = Backbone.View.extend({
>      $builds.empty();
>  
>      $('#imageBuildsTemplate').tmpl(this.model.toJSON()).appendTo($builds);
> +
> +    // Enable/Disable Push All button
> +    var $buttonTemplate = $('#pushAllButtonTemplate');
> +    var $pushAllBtn = $('#push-all-btn');
> +
> +    var buildId = this.model.get("build") ? this.model.get("build")['uuid'] : null;
> +    var latestBuildId = this.model.get("latest_build_id");
> +
> +    if (buildId && buildId == latestBuildId) {
> +      $pushAllBtn.html($buttonTemplate.tmpl(this.model.toJSON()));
> +    }
> +    else {
> +      $pushAllBtn.empty();
> +    }
>    }
>  });
> 

Ack.

-- 

With regards
Petr Blaho



More information about the aeolus-devel mailing list