Merge pull request sonata-project#2530 from caponica/empty_filters

Fix empty filter list
mushkin-v · Jan 3, 2015 · fd09943 · fd09943
2 parents c394991 + be12d1a
commit fd09943
Showing 1 changed file with 18 additions and 16 deletions.
diff --git a/Resources/views/CRUD/base_list.html.twig b/Resources/views/CRUD/base_list.html.twig
@@ -188,22 +188,24 @@ file that was distributed with this source code.
 {% endblock %}
 
 {% block list_filters_actions %}
-    <ul class="nav navbar-nav navbar-right">
-
-        <li class="dropdown sonata-actions">
-            <a href="#" class="dropdown-toggle sonata-ba-action" data-toggle="dropdown">{{ 'link_filters'|trans({}, 'SonataAdminBundle') }} <b class="caret"></b></a>
-
-            <ul class="dropdown-menu" role="menu">
-                {% for filter in admin.datagrid.filters if filter.options['show_filter']%}
-                    <li>
-                        <a href="#" class="sonata-toggle-filter sonata-ba-action" filter-target="filter-{{ admin.uniqid }}-{{ filter.name }}" filter-container="filter-container-{{ admin.uniqid() }}">
-                            <i class="fa {{ filter.isActive() ? 'fa-check-square-o' : 'fa-square-o' }}"></i>{{ admin.trans(filter.label, {}, filter.translationDomain) }}
-                        </a>
-                    </li>
-                {% endfor %}
-            </ul>
-        </li>
-    </ul>
+    {%- if admin.datagrid.filters|length %}
+        <ul class="nav navbar-nav navbar-right">
+
+            <li class="dropdown sonata-actions">
+                <a href="#" class="dropdown-toggle sonata-ba-action" data-toggle="dropdown">{{ 'link_filters'|trans({}, 'SonataAdminBundle') }} <b class="caret"></b></a>
+
+                <ul class="dropdown-menu" role="menu">
+                    {% for filter in admin.datagrid.filters if filter.options['show_filter']%}
+                        <li>
+                            <a href="#" class="sonata-toggle-filter sonata-ba-action" filter-target="filter-{{ admin.uniqid }}-{{ filter.name }}" filter-container="filter-container-{{ admin.uniqid() }}">
+                                <i class="fa {{ filter.isActive() ? 'fa-check-square-o' : 'fa-square-o' }}"></i>{{ admin.trans(filter.label, {}, filter.translationDomain) }}
+                            </a>
+                        </li>
+                    {% endfor %}
+                </ul>
+            </li>
+        </ul>
+    {% endif -%}
 {% endblock %}
 
 {% block list_filters %}