Re: [PATCH] [v2] toaster: Fix the download icon

[Michael Wood <michael.g.wood@intel.com>]

Thanks upstreamed and pushed to toaster-next with additions to make 
commit message more descriptive.

On 10/11/15 16:52, Belen Barros Pena wrote:
> From: Belen Barros Pena <belen.barros.pena@intel.com>
>
> The 'all builds' page was missing the download icon
> next to the outcome icon, which allows you to download
> a build log from the 'all builds' page.
>
> This patch brings it back.
>
> It also adds a check in the project builds page to make
> sure the download icon only appears if the build generates
> a cooker log, since builds that fail at the build request
> stage do not generate a cooker log.
>
> Signed-off-by: Belen Barros Pena <belen.barros.pena@intel.com>
> ---
>   bitbake/lib/toaster/toastergui/templates/builds.html        |  8 +++++++-
>   bitbake/lib/toaster/toastergui/templates/projectbuilds.html | 11 ++++++-----
>   2 files changed, 13 insertions(+), 6 deletions(-)
>
> diff --git a/bitbake/lib/toaster/toastergui/templates/builds.html b/bitbake/lib/toaster/toastergui/templates/builds.html
> index a27a121..c3cc266 100644
> --- a/bitbake/lib/toaster/toastergui/templates/builds.html
> +++ b/bitbake/lib/toaster/toastergui/templates/builds.html
> @@ -62,7 +62,13 @@
>           {% for build in objects %}
>           <tr class="data" data-table-build-result="{{ build.id }}">
>               <td class="outcome">
> -                <a href="{% url "builddashboard" build.id %}">{%if build.outcome == build.SUCCEEDED%}<i class="icon-ok-sign success"></i>{%elif build.outcome == build.FAILED%}<i class="icon-minus-sign error"></i>{%else%}{%endif%}</a> &nbsp;
> +                <a href="{% url "builddashboard" build.id %}">{%if build.outcome == build.SUCCEEDED%}<i class="icon-ok-sign success"></i>{%elif build.outcome == build.FAILED%}<i class="icon-minus-sign error"></i>{%else%}{%endif%}</a>
> +                {% if build.cooker_log_path %}
> +                  &nbsp;
> +                  <a href="{% url 'build_artifact' build.id "cookerlog" build.id %}">
> +                    <i class="icon-download-alt" title="Download build log"></i>
> +                  </a>
> +                {% endif %}
>               </td>
>               <td class="target">
>                   {% for t in build.target_set.all %}
> diff --git a/bitbake/lib/toaster/toastergui/templates/projectbuilds.html b/bitbake/lib/toaster/toastergui/templates/projectbuilds.html
> index bb38284..3402fc4 100644
> --- a/bitbake/lib/toaster/toastergui/templates/projectbuilds.html
> +++ b/bitbake/lib/toaster/toastergui/templates/projectbuilds.html
> @@ -63,11 +63,12 @@
>           {% for build in objects %} {# if we have a build, just display it #}
>           <tr class="data">
>               <td class="outcome"><a href="{% url "builddashboard" build.id %}">{%if build.outcome == build.SUCCEEDED%}<i class="icon-ok-sign success"></i>{%elif build.outcome == build.FAILED%}<i class="icon-minus-sign error"></i>{%else%}{%endif%}</a>
> -                   {% if build.project %}
> -                        &nbsp; <a href="{% url 'build_artifact' build.id "cookerlog" build.id %}">
> -                            <i class="icon-download-alt" title="" data-original-title="Download build log"></i>
> -                        </a>
> -                    {% endif %}
> +              {% if build.cooker_log_path %}
> +                &nbsp;
> +                <a href="{% url 'build_artifact' build.id "cookerlog" build.id %}">
> +                  <i class="icon-download-alt" title="Download build log"></i>
> +                </a>
> +              {% endif %}
>               </td>
>   
>               <td class="target">