Merge f1b710f into aace5dd

FortySource · Nov 29, 2018 · b63b620 · b63b620
2 parents aace5dd + f1b710f
commit b63b620
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/lib/forty_facets/filter/scope_filter_definition.rb b/lib/forty_facets/filter/scope_filter_definition.rb
@@ -2,7 +2,7 @@ module FortyFacets
   class ScopeFilterDefinition < FilterDefinition
     class ScopeFilter < Filter
       def active?
-        definition.options[:pass_value] ? value.present? : '1'
+        definition.options[:pass_value] ? value.present? : value == '1'
       end
 
       def selected 
@@ -28,7 +28,7 @@ def remove(value = nil)
         search_instance.class.new_unwrapped(new_params, search_instance.root)
       end
 
-      def add(value = nil)
+      def add(value = '1')
         new_params = search_instance.params || {}
         new_params[definition.request_param] = value
         search_instance.class.new_unwrapped(new_params, search_instance.root)
@@ -38,7 +38,5 @@ def add(value = nil)
     def build_filter(search_instance, value)
       ScopeFilter.new(self, search_instance, value)
     end
-
   end
 end
-