Fix issue where has_active_filters were not being looked up correctly

cfpb · Jan 17, 2019 · f0a1825 · f0a1825
1 parent 347a2d4
commit f0a1825
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/cfgov/jinja2/v1/_includes/organisms/filterable-list-controls.html b/cfgov/jinja2/v1/_includes/organisms/filterable-list-controls.html
@@ -190,11 +190,13 @@
 
    fragment_id: A unique ID for the filter list control.
 
+   has_active_filters: Whether the filter has active filters or not.
+
    ========================================================================== #}
-{% macro render(value, form, fragment_id) %}
+{% macro render(value, form, fragment_id, has_active_filters) %}
     {% from 'organisms/expandable.html' import expandable with context %}
     <div id="{{ fragment_id }}" class="o-filterable-list-controls">
-        {% if value.has_active_filters %}
+        {% if has_active_filters %}
             {% do value.update({'is_expanded':true}) %}
         {% endif %}
 

diff --git a/cfgov/jinja2/v1/_includes/organisms/filterable-list.html b/cfgov/jinja2/v1/_includes/organisms/filterable-list.html
@@ -53,7 +53,12 @@
         {# ADD THE FILTER CONTROLS. #}
 
         {% set fragment_id = 'o-filterable-list-controls' %}
-        {{ filterable_list_controls.render(value, form, fragment_id) }}
+        {{ filterable_list_controls.render(
+            value,
+            form,
+            fragment_id,
+            has_active_filters)
+        }}
 
         {# ADD THE NOTIFICATION, IF NEEDED. #}