From 19346158bfeb502e953364d0e413276fe689c917 Mon Sep 17 00:00:00 2001
From: Toby <toby.junk@gmail.com>
Date: Thu, 24 May 2012 17:01:01 +0100
Subject: [PATCH] 2418 click removes chosen facet + remove duplicate facet
 error

---
 ckan/templates/snippets/facet_list.html | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/ckan/templates/snippets/facet_list.html b/ckan/templates/snippets/facet_list.html
index 660f7e3bcab..399c74057ca 100644
--- a/ckan/templates/snippets/facet_list.html
+++ b/ckan/templates/snippets/facet_list.html
@@ -29,23 +29,24 @@ <h2 class="heading">${c.facet_titles.get(name)} <a href="${c.remove_field(name)}
     <nav py:if="items">
       <ul class="unstyled nav nav-simple nav-facet">
         <py:for each="item in items">
-        <!--! I'd rather not do this but it seems to be the only way genshi is happy -->
         <py:choose test="">
           <py:if test="item.active">
             <li class="active">
-            <a href="${c.drill_down_url(**{name: item.name})}">
+            <a href="${c.remove_field(name, item.name)}">
               ${label_function(item) if vars().has_key('label_function') else h.truncate(item.display_name, 22)}
               ${count_label(item['count']) if vars().has_key('count_label') else ('(%d)' % item['count'])}
             </a>
           </li>
           </py:if>
           <py:otherwise>
+          <py:if test="not item.active">
             <li>
             <a href="${c.drill_down_url(**{name: item.name})}">
               ${label_function(item) if vars().has_key('label_function') else h.truncate(item.display_name, 22)}
               ${count_label(item['count']) if vars().has_key('count_label') else ('(%d)' % item['count'])}
             </a>
           </li>
+          </py:if>
           </py:otherwise>
         </py:choose>
         </py:for>