From d1de12cac1eb08ee2c8133a3fe41c53ae7d34cee Mon Sep 17 00:00:00 2001
From: aszlig <aszlig@redmoonstudios.org>
Date: Sun, 26 Apr 2015 18:01:02 +0200
Subject: [PATCH] product-list: Handle channel along with "file".

We want to have contents and detauls of channel expressions as well and
we already have that in product.type == file, so why not reuse the same
for the channel expression?

Signed-off-by: aszlig <aszlig@redmoonstudios.org>
---
 src/root/product-list.tt | 22 ++++++++--------------
 1 file changed, 8 insertions(+), 14 deletions(-)

diff --git a/src/root/product-list.tt b/src/root/product-list.tt
index 5eef9640..17e2911e 100644
--- a/src/root/product-list.tt
+++ b/src/root/product-list.tt
@@ -117,19 +117,7 @@
           </tr>
         [% END %]
 
-      [% CASE "channel" %]
-
-      <tr class="product">
-        <td>
-          <a href="[% uri %]">
-            <img src="[% c.uri_for("/static/images/channel.png") %]" alt="Channel" />
-            Channel expression tarball <tt>[% product.name %]</tt>
-            [% IF product.subtype != "-" %]for <tt>[% product.subtype %]</tt>[% END %]
-          </a>
-        </td>
-      </tr>
-
-      [% CASE "file" %]
+      [% CASE ["file", "channel"] %]
 
       <tr class="product">
         <td>
@@ -148,7 +136,13 @@
           [% CASE "binary-dist" %]
             <img src="[% c.uri_for("/static/images/binary-dist.png") %]" alt="Binary distribution" /> Binary distribution <tt>[% product.name %]</tt>
           [% CASE DEFAULT %]
-            File <tt>[% product.name %]</tt> of type <tt>[% product.subtype %]</tt>
+            [% IF product.type == "channel" %]
+              <img src="[% c.uri_for("/static/images/channel.png") %]" alt="Channel" />
+              Channel expression tarball <tt>[% product.name %]</tt>
+              [% IF product.subtype != "-" %]for <tt>[% product.subtype %]</tt>[% END %]
+            [% ELSE %]
+              File <tt>[% product.name %]</tt> of type <tt>[% product.subtype %]</tt>
+            [% END %]
           [% END %]
         </a>
         [% WRAPPER makePopover title="Details" classes="btn-mini" %]