filterrific 5.2.2 → 5.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 5226106f7944a5b6c08d8f568185d4e6fcf0c561
-  data.tar.gz: 2e4b24d697de15d10a5dddf6a7b444b5564950ed
+SHA256:
+  metadata.gz: a63c202db5dce43d16d986ad31a50abc21752cb7bd66771fce52c2ddc7a2614d
+  data.tar.gz: '090e98ad1d561910bbefd0095be81e8b0c622e29cd8cba5b12471379b9023b65'
 SHA512:
-  metadata.gz: 7829c794eb5e1a2e19a95cbb2e9651550f8c6cb5c123709a78c5fd5658c569a2c78963144f68b07cb30de9d28372ad3bfaf0087499a91f48f2e5c0597ed09d02
-  data.tar.gz: 4c7894ba6c9c1e9e5cde222bd69a088761dda82ccc01c0a1c12a1db4ed9e489b1f03b008bdc09dad8563355d95f138491cc17bd5c78174f1b9f6b46c3a4f0ec9
+  metadata.gz: 2d74dc358e308d4d84f04c6637bcd7c682b2ff671d9e11b36d9c9384434b4670d963e945ba80314e6e6e497fc54f7873b37e674bd5794d7f1b298c3912ac2190
+  data.tar.gz: b8731da2f59b0d70838c412fdacc1cbf103c76c1eba8668df5253a4d35948e7b60dd9f9bfbb8a021c1c11efe5ab7b945dca3f6490c0b58b66847a3a09dde717c

data/CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 Filterrific major versions match the Ruby on Rails major versions they work with.
 
+## [5.2.4] - Jan 30, 2023
+
+* Remove JS dependency on jquery
+* Make code compliant with standard.rb rules
+
+## [5.2.3] - Mar. 18, 2022
+
+* Added support for Rails 7
+
 ## [5.2.2] - Jul. 11, 2021
 
 * Fixed Ruby 2.7 deprecated warning when trying to regex match an Integer.

data/README.md

@@ -39,13 +39,13 @@ to find out more!
 
 Every commit to Filterrific is automatically tested against the following scenarios:
 
-|Filterrific version | Rails version  | Ruby environments              | Database adapters                  | Build status |
-|--------------------|----------------|--------------------------------|------------------------------------|--------------|
-| 5.x                | Rails 5.x, 6.x | MRI 2.0.0, 2.1.7, 2.2.3, 2.3.1 | mysql2, postgresql                 |[![Build Status](https://travis-ci.org/jhund/filterrific_demo.svg?branch=rails-5.x)](https://travis-ci.org/jhund/filterrific_demo)|
-| 4.x                | Rails 4.x      | MRI 2.0.0, 2.1.7, 2.2.3, 2.3.1 | mysql, mysql2, postgresql, sqlite3 |[![Build Status](https://travis-ci.org/jhund/filterrific_demo.svg?branch=rails-4.x)](https://travis-ci.org/jhund/filterrific_demo)|
-| 3.x                | Rails 3.2      | MRI 2.0.0, 2.1.7               | mysql, mysql2, postgresql, sqlite3 | Not tested|
-| 2.x                | Rails 3.2      | MRI 1.9.3                      | mysql, mysql2, postgresql, sqlite3 | Not tested|
-| 1.x                | < 3.2          | MRI <= 1.9.3                   | mysql, mysql2, postgresql, sqlite3 | Not tested|
+|Filterrific version | Rails version       | Ruby environments              | Database adapters                  | Build status |
+|--------------------|---------------------|--------------------------------|------------------------------------|--------------|
+| 5.x                | Rails 5.x, 6.x, 7.x | MRI 2.0.0, 2.1.7, 2.2.3, 2.3.1 | mysql2, postgresql                 |[![Build Status](https://travis-ci.org/jhund/filterrific_demo.svg?branch=rails-5.x)](https://travis-ci.org/jhund/filterrific_demo)|
+| 4.x                | Rails 4.x           | MRI 2.0.0, 2.1.7, 2.2.3, 2.3.1 | mysql, mysql2, postgresql, sqlite3 |[![Build Status](https://travis-ci.org/jhund/filterrific_demo.svg?branch=rails-4.x)](https://travis-ci.org/jhund/filterrific_demo)|
+| 3.x                | Rails 3.2           | MRI 2.0.0, 2.1.7               | mysql, mysql2, postgresql, sqlite3 | Not tested|
+| 2.x                | Rails 3.2           | MRI 1.9.3                      | mysql, mysql2, postgresql, sqlite3 | Not tested|
+| 1.x                | < 3.2               | MRI <= 1.9.3                   | mysql, mysql2, postgresql, sqlite3 | Not tested|
 
 ### Guidelines for submitting issues
 

data/Rakefile

@@ -1,17 +1,17 @@
 begin
-  require 'bundler/setup'
+  require "bundler/setup"
 rescue LoadError
-  puts 'You must `gem install bundler` and `bundle install` to run rake tasks'
+  puts "You must `gem install bundler` and `bundle install` to run rake tasks"
 end
 
 Bundler::GemHelper.install_tasks
 
-require 'rake/testtask'
+require "rake/testtask"
 Rake::TestTask.new do |test|
-  test.libs << 'spec'
-  test.pattern = 'spec/**/*_spec.rb'
+  test.libs << "spec"
+  test.pattern = "spec/**/*_spec.rb"
   test.verbose = true
 end
 
-require 'wwtd/tasks'
-task default: 'wwtd:local'
+require "wwtd/tasks"
+task default: "wwtd:local"

data/app/assets/javascripts/filterrific/filterrific.js

@@ -0,0 +1,172 @@
+/**
+ * Javascript behaviors for Filterrific.
+ * http://filterrific.clearcove.ca
+ *
+ * Released under the MIT license
+ *
+ */
+
+// Create global Filterrific namespace
+if (typeof Filterrific === 'undefined') {
+  var Filterrific = {};
+}
+
+// Define function to submit Filterrific filter form
+Filterrific.submitFilterForm = function() {
+  var form = Filterrific.findParents(this, '#filterrific_filter')[0];
+
+  // send before event
+  form.dispatchEvent(new Event('loadingFilterrificResults'));
+
+  // turn on spinner
+  document.querySelector('.filterrific_spinner').style.display = 'block';
+
+  // Abort previous XMLHttpRequest request
+  if (Filterrific.lastRequest && Filterrific.lastRequest.readyState != 4) {
+    Filterrific.lastRequest.abort();
+  }
+
+  // Submit XMLHttpRequest request
+  Filterrific.lastRequest = Filterrific.prepareRequest(form);
+  Filterrific.lastRequest.send();
+};
+
+Filterrific.prepareRequest = function(form) {
+  var url = form.getAttribute('action'),
+      formData = new FormData(form),
+      params = new URLSearchParams(formData),
+      xhr = new XMLHttpRequest();
+
+  if (url.indexOf('?') < 0) {
+    url += '?' + params;
+  } else {
+    url += '&' + params;
+  }
+
+  xhr.open("GET", url, true);
+  xhr.setRequestHeader('Accept', 'text/javascript, application/javascript, application/ecmascript, application/x-ecmascript, */*; q=0.01');
+  xhr.setRequestHeader('X-Requested-With', 'XMLHttpRequest');
+  xhr.onreadystatechange = function() {
+    if (xhr.readyState === XMLHttpRequest.DONE) {
+        return Filterrific.processResponse(form, xhr);
+      }
+  }
+
+  return xhr;
+}
+
+Filterrific.processResponse = function(form, xhr) {
+  var rawResponse = (_ref = xhr.response) != null ? _ref : xhr.responseText,
+      type = xhr.getResponseHeader('Content-Type'),
+      response;
+
+  if (typeof rawResponse === 'string' && typeof type === 'string') {
+      if (type.match(/\bjson\b/)) {
+        try {
+          response = JSON.parse(rawResponse);
+        } catch (_error) {}
+      } else if (type.match(/\b(?:java|ecma)script\b/)) {
+        script = document.createElement('script');
+        script.setAttribute('nonce', Filterrific.cspNonce());
+        script.text = rawResponse;
+        document.head.appendChild(script).parentNode.removeChild(script);
+      } else if (type.match(/\b(xml|html|svg)\b/)) {
+        parser = new DOMParser();
+        type = type.replace(/;.+/, '');
+        try {
+          response = parser.parseFromString(rawResponse, type);
+        } catch (_error) {}
+      }
+  }
+
+  // send after event
+  form.dispatchEvent(new Event('loadedFilterrificResults'));
+  document.querySelector('.filterrific_spinner').style.display = 'none';
+
+  return response;
+}
+
+Filterrific.cspNonce = function() {
+  return document.querySelector("meta[name=csp-nonce]")?.content
+}
+
+Filterrific.findParents = function(elem, selector) {
+  var elements = [];
+  var ishaveselector = selector !== undefined;
+
+  while ((elem = elem.parentElement) !== null) {
+    if (elem.nodeType !== Node.ELEMENT_NODE) {
+      continue;
+    }
+
+    if (!ishaveselector || elem.matches(selector)) {
+      elements.push(elem);
+    }
+  }
+
+  return elements;
+};
+
+Filterrific.observe_field = function(inputs_selector, frequency, callback) {
+  frequency = frequency * 1000;
+
+  document.querySelectorAll(inputs_selector).forEach(input => {
+    var prev = input.value;
+    var check = function() {
+      // if removed clear the interval and don't fire the callback
+      if (removed()) {
+        if(ti) clearInterval(ti);
+        return;
+      }
+      var val = input.value;
+      if (prev != val) {
+        prev = val;
+
+        // invokes the callback on $this
+        if (callback && typeof callback === 'function') {
+          callback.call(input);
+        }
+      }
+    };
+
+    var removed = function() {
+      return input.closest('html').length == 0
+    };
+
+    var reset = function() {
+      if (ti) {
+        clearInterval(ti);
+        ti = setInterval(check, frequency);
+      }
+    };
+    check();
+    var ti = setInterval(check, frequency); // invoke check periodically
+    // reset counter after user interaction
+    // mousemove is for selects
+    input.addEventListener('keyup', reset);
+    input.addEventListener('click', reset);
+    input.addEventListener('mousemove', reset);
+  })
+};
+
+
+Filterrific.init = function() {
+  // Add change event handler to all Filterrific filter inputs.
+  var filterrificForm = document.querySelector('#filterrific_filter');
+  filterrificForm.querySelectorAll('input, textarea, select, textarea').forEach(input => {
+    input.addEventListener('change', Filterrific.submitFilterForm)
+  })
+
+  // Add periodic observer to selected inputs.
+  // Use this for text fields you want to observe for change, e.g., a search input.
+  Filterrific.observe_field(
+    ".filterrific-periodically-observed",
+    0.5,
+    Filterrific.submitFilterForm
+  );
+};
+
+
+// Initialize event observers on document ready and turbolinks page:load
+document.addEventListener('turbolinks:load', Filterrific.init);
+document.addEventListener('DOMContentLoaded', Filterrific.init)

data/doc/meta.md

@@ -5,7 +5,6 @@ I use the gem-release gem
 For more info see: https://github.com/svenfuchs/gem-release#usage
 
 
-
 ## Steps for an update
 
 1. Update code and commit it.
@@ -22,11 +21,26 @@ For more info see: https://github.com/svenfuchs/gem-release#usage
    `gem tag`
 
 
-
 ## How to run specs
 
-`bundle exec rake`
+`bundle exec rake` in the filterrific repo
+
+
+## How to support a new major Rails version
+
+Follow these steps when starting support for a new Rails major version:
+
+* In `filterrific`
+   * Archive the current major rails version into a new branch off of master, e.g., when starting to support Rails 6, create a new branch for `5.x` from current master. This will be the branch used for ongoing 5.x support, and all new development for Rails 6 will happen in the `master` branch.
+   * Make all changes required to support a new version of Rails.
+   * Release the first `filterrific` version for Rails 6: `6.0.0`.
 
+* In `filterrific_demo`
+   * Following the same example for Rails 6:
+   * Make sure that the `5.x` branch is up-to-date with master, and with current filterrific.
+   * Create a new `6.x` branch. In that branch create a brand new rails app using a current version of Rails 6.
+   * Make sure that the app works with the relevant version of `filterrific`.
+   * Deploy demo app to heroku.
 
 
 ## Travis CI

data/doc/scratchpad.md

@@ -2,36 +2,8 @@
 
 ## 2021 Initiative
 
-* Update for Rails 6
-* Implement API only version (no frontend)
-* Update scope docs to use AREL
-* Implement/document various front end scenarios:
-    * React
-    * Stimulus
-    * jQuery
-    * sprockets/webpacker
-    * Thoughts: Support the rails defaults (webpacker or sprockets?) out of the box, batteries included. Document how to do everything else.
-* Work through issues and pull requests
-
-improve the filterrific frontend handling:
-
-* Remove invocation of init, document for devs to do it manually
-  https://github.com/jhund/filterrific/issues/199
-* Replace jquery with vanilla JS
-* Provide filterrific API version
-* Make compatible with Rails 6
-    * Remove jquery dependency, make it easier to use with webpacker, document how to do init
-      https://github.com/jhund/filterrific/issues/198
-
-* Support the following js scenarios:
-    * Sprockets
-    * Webpacker
-    * API only (e.g., for React)
-
-
 ## TODO
 
-* add check that no filter_name conflicts with existing methods on included ActiveRecord class (See https://github.com/jhund/filterrific/issues/17)
 * Update SW architecture and documentation according to Osterhuis Philosophy of software architecture/design book (mark advanced controller options as such to keep interface to learn simple)
 
 ## Travis

data/lib/filterrific/action_controller_extension.rb

@@ -1,13 +1,11 @@
-# -*- coding: utf-8 -*-
 #
 # Adds Filterrific methods ActionController instances
 #
 module Filterrific
   module ActionControllerExtension
-
     include HasResetFilterrificUrlMixin
 
-  protected
+    protected
 
     # @param model_class [Class]
     # @param filterrific_params [ActionController::Params, Hash] typically the
@@ -33,29 +31,29 @@ module Filterrific
     def initialize_filterrific(model_class, filterrific_params, opts = {})
       f_params = (filterrific_params || {}).stringify_keys
       opts = opts.stringify_keys
-      pers_id = if false == opts['persistence_id']
+      pers_id = if opts["persistence_id"] == false
         nil
       else
-        opts['persistence_id'] || compute_default_persistence_id
+        opts["persistence_id"] || compute_default_persistence_id
       end
 
-      if (f_params.delete('reset_filterrific'))
+      if f_params.delete("reset_filterrific")
         # Reset query and session_persisted params
-        session[pers_id] = nil  if pers_id
-        redirect_to url_for({})  and return false # requires `or return` in calling action.
+        session[pers_id] = nil if pers_id
+        redirect_to url_for({}) and return false # requires `or return` in calling action.
       end
 
       f_params = compute_filterrific_params(model_class, f_params, opts, pers_id)
 
       filterrific = Filterrific::ParamSet.new(model_class, f_params)
-      filterrific.select_options = opts['select_options']
-      session[pers_id] = filterrific.to_hash  if pers_id
+      filterrific.select_options = opts["select_options"]
+      session[pers_id] = filterrific.to_hash if pers_id
       filterrific
     end
 
     # Computes a default persistence id based on controller and action name
     def compute_default_persistence_id
-      [controller_name, action_name].join('#')
+      [controller_name, action_name].join("#")
     end
 
     # Computes filterrific params using a number of strategies. Limits params
@@ -68,17 +66,17 @@ module Filterrific
     #   Defaults to true.
     # @param persistence_id [String, nil]
     def compute_filterrific_params(model_class, filterrific_params, opts, persistence_id)
-      opts = { "sanitize_params" => true }.merge(opts.stringify_keys)
+      opts = {"sanitize_params" => true}.merge(opts.stringify_keys)
       r = (
         filterrific_params.presence || # start with passed in params
         (persistence_id && session[persistence_id].presence) || # then try session persisted params if persistence_id is present
-        opts['default_filter_params'] || # then use passed in opts
+        opts["default_filter_params"] || # then use passed in opts
         model_class.filterrific_default_filter_params # finally use model_class defaults
       ).stringify_keys
-      r.slice!(*opts['available_filters'].map(&:to_s))  if opts['available_filters']
+      r.slice!(*opts["available_filters"].map(&:to_s)) if opts["available_filters"]
       # Sanitize params to prevent reflected XSS attack
       if opts["sanitize_params"]
-        r.each { |k,v| r[k] = sanitize_filterrific_param(r[k]) }
+        r.each { |k, v| r[k] = sanitize_filterrific_param(r[k]) }
       end
       r
     end
@@ -94,7 +92,7 @@ module Filterrific
         val.map { |e| sanitize_filterrific_param(e) }
       when Hash
         # Return Hash
-        val.inject({}) { |m, (k,v)| m[k] = sanitize_filterrific_param(v); m }
+        val.each_with_object({}) { |(k, v), m| m[k] = sanitize_filterrific_param(v); }
       when NilClass
         # Nothing to do, use val as is
         val
@@ -105,6 +103,5 @@ module Filterrific
         val
       end
     end
-
   end
 end

data/lib/filterrific/action_view_extension.rb

@@ -1,10 +1,8 @@
-# -*- coding: utf-8 -*-
 #
 # Adds Filterrific view helpers to ActionView instances
 #
 module Filterrific
   module ActionViewExtension
-
     include HasResetFilterrificUrlMixin
 
     # Sets all options on form_for to defaults that work with Filterrific
@@ -17,8 +15,8 @@ module Filterrific
       options[:html][:method] ||= :get
       options[:html][:id] ||= :filterrific_filter
       options[:url] ||= url_for(
-        :controller => controller.controller_name,
-        :action => controller.action_name
+        controller: controller.controller_name,
+        action: controller.action_name
       )
       form_for(record, options, &block)
     end
@@ -27,7 +25,7 @@ module Filterrific
     def render_filterrific_spinner
       %(
         <span class="filterrific_spinner" style="display:none;">
-          #{ image_tag('filterrific/filterrific-spinner.gif', alt: '', role: 'presentation') }
+          #{image_tag("filterrific/filterrific-spinner.gif", alt: "", role: "presentation")}
         </span>
       ).html_safe
     end
@@ -65,23 +63,25 @@ module Filterrific
     #     Override the target URL attributes to be used for `url_for`. Default: {} (current URL).
     def filterrific_sorting_link(filterrific, sort_key, opts = {})
       opts = {
-        :active_column_class => 'filterrific_current_sort_column',
-        :inactive_column_class => 'filterrific_sort_column',
-        :ascending_indicator => '⬆',
-        :default_sort_direction => 'asc',
-        :descending_indicator => '⬇',
-        :html_attrs => {},
-        :label => sort_key.to_s.humanize,
-        :sorting_scope_name => :sorted_by,
-        :url_for_attrs => {},
-        :as => :filterrific
+        active_column_class: "filterrific_current_sort_column",
+        inactive_column_class: "filterrific_sort_column",
+        ascending_indicator: "⬆",
+        default_sort_direction: "asc",
+        descending_indicator: "⬇",
+        html_attrs: {},
+        label: sort_key.to_s.humanize,
+        sorting_scope_name: :sorted_by,
+        url_for_attrs: {},
+        as: :filterrific
       }.merge(opts)
       opts.merge!(
-        :html_attrs => opts[:html_attrs].with_indifferent_access,
-        :current_sorting => (current_sorting = filterrific.send(opts[:sorting_scope_name])),
-        :current_sort_key => current_sorting ? current_sorting.gsub(/_asc|_desc/, '') : nil,
-        :current_sort_direction => current_sorting ? (current_sorting =~ /_desc\z/ ? 'desc' : 'asc') : nil,
-        :current_sort_direction_indicator => (current_sorting =~ /_desc\z/ ? opts[:descending_indicator] : opts[:ascending_indicator]),
+        html_attrs: opts[:html_attrs].with_indifferent_access,
+        current_sorting: (current_sorting = filterrific.send(opts[:sorting_scope_name])),
+        current_sort_key: current_sorting ? current_sorting.gsub(/_asc|_desc/, "") : nil,
+        current_sort_direction: if current_sorting
+                                  current_sorting.end_with?("_desc") ? "desc" : "asc"
+                                end,
+        current_sort_direction_indicator: (current_sorting.end_with?("_desc") ? opts[:descending_indicator] : opts[:ascending_indicator])
       )
       new_sort_key = sort_key.to_s
       if new_sort_key == opts[:current_sort_key]
@@ -93,7 +93,7 @@ module Filterrific
       end
     end
 
-  protected
+    protected
 
     # Renders HTML to reverse sort order on currently sorted column.
     # @param filterrific [Filterrific::ParamSet]
@@ -102,20 +102,20 @@ module Filterrific
     # @return [String] an HTML fragment
     def filterrific_sorting_link_reverse_order(filterrific, new_sort_key, opts)
       # current sort column, toggle search_direction
-      new_sort_direction = 'asc' == opts[:current_sort_direction] ? 'desc' : 'asc'
-      new_sorting = safe_join([new_sort_key, new_sort_direction], '_')
+      new_sort_direction = opts[:current_sort_direction] == "asc" ? "desc" : "asc"
+      new_sorting = safe_join([new_sort_key, new_sort_direction], "_")
       css_classes = safe_join([
         opts[:active_column_class],
         opts[:html_attrs].delete(:class)
-      ].compact, ' ')
+      ].compact, " ")
       new_filterrific_params = filterrific.to_hash
-                                          .with_indifferent_access
-                                          .merge(opts[:sorting_scope_name] => new_sorting)
+        .with_indifferent_access
+        .merge(opts[:sorting_scope_name] => new_sorting)
       url_for_attrs = opts[:url_for_attrs].merge(opts[:as] => new_filterrific_params)
       link_to(
-        safe_join([opts[:label], opts[:current_sort_direction_indicator]], ' '),
+        safe_join([opts[:label], opts[:current_sort_direction_indicator]], " "),
         url_for(url_for_attrs),
-        opts[:html_attrs].reverse_merge(:class => css_classes, :method => :get, :remote => true)
+        opts[:html_attrs].reverse_merge(class: css_classes, method: :get, remote: true)
       )
     end
 
@@ -126,21 +126,20 @@ module Filterrific
     # @return [String] an HTML fragment
     def filterrific_sorting_link_new_column(filterrific, new_sort_key, opts)
       new_sort_direction = opts[:default_sort_direction]
-      new_sorting = safe_join([new_sort_key, new_sort_direction], '_')
+      new_sorting = safe_join([new_sort_key, new_sort_direction], "_")
       css_classes = safe_join([
         opts[:inactive_column_class],
         opts[:html_attrs].delete(:class)
-      ].compact, ' ')
+      ].compact, " ")
       new_filterrific_params = filterrific.to_hash
-                                          .with_indifferent_access
-                                          .merge(opts[:sorting_scope_name] => new_sorting)
+        .with_indifferent_access
+        .merge(opts[:sorting_scope_name] => new_sorting)
       url_for_attrs = opts[:url_for_attrs].merge(opts[:as] => new_filterrific_params)
       link_to(
         opts[:label],
         url_for(url_for_attrs),
-        opts[:html_attrs].reverse_merge(:class => css_classes, :method => :get, :remote => true)
+        opts[:html_attrs].reverse_merge(class: css_classes, method: :get, remote: true)
       )
     end
-
   end
 end

data/lib/filterrific/active_record_extension.rb

@@ -1,12 +1,10 @@
-# -*- coding: utf-8 -*-
 #
 # Adds Filterrific methods to ActiveRecord::Base model_class.
 #
-require 'filterrific/param_set'
+require "filterrific/param_set"
 
 module Filterrific
   module ActiveRecordExtension
-
     # Adds Filterrific behavior to class when called like so:
     #
     # filterrific(
@@ -34,7 +32,6 @@ module Filterrific
       validate_filterrific_available_filters
       assign_filterrific_default_filter_params(opts)
       validate_filterrific_default_filter_params
-
     end
 
     # Returns ActiveRecord relation based on filterrific_param_set.
@@ -46,7 +43,7 @@ module Filterrific
       unless filterrific_param_set.is_a?(Filterrific::ParamSet)
         raise(
           ArgumentError,
-          "Invalid Filterrific::ParamSet: #{ filterrific_param_set.inspect }"
+          "Invalid Filterrific::ParamSet: #{filterrific_param_set.inspect}"
         )
       end
 
@@ -69,7 +66,7 @@ module Filterrific
       ar_rel
     end
 
-  protected
+    protected
 
     # Defines a :sorted_by scope based on attrs
     # @param attrs [Hash] with keys as
@@ -83,7 +80,7 @@ module Filterrific
     # @return [void]
     def assign_filterrific_available_filters(opts)
       self.filterrific_available_filters = (
-        filterrific_available_filters + (opts['available_filters'] || [])
+        filterrific_available_filters + (opts["available_filters"] || [])
       ).map(&:to_s).uniq.sort
     end
 
@@ -97,7 +94,7 @@ module Filterrific
 
     def assign_filterrific_default_filter_params(opts)
       self.filterrific_default_filter_params = (
-        opts['default_filter_params'] || {}
+        opts["default_filter_params"] || {}
       ).stringify_keys
     end
 
@@ -106,9 +103,8 @@ module Filterrific
       if (
         inv_fdfps = filterrific_default_filter_params.keys - filterrific_available_filters
       ).any?
-        raise(ArgumentError, "Invalid default filter params: #{ inv_fdfps.inspect }")
+        raise(ArgumentError, "Invalid default filter params: #{inv_fdfps.inspect}")
       end
     end
-
   end
 end

data/lib/filterrific/engine.rb

@@ -1,16 +1,13 @@
-# -*- coding: utf-8 -*-
+require "filterrific/param_set"
 
-require 'filterrific/param_set'
+require "filterrific/has_reset_filterrific_url_mixin"
 
-require 'filterrific/has_reset_filterrific_url_mixin'
-
-require 'filterrific/action_controller_extension'
-require 'filterrific/action_view_extension'
-require 'filterrific/active_record_extension'
+require "filterrific/action_controller_extension"
+require "filterrific/action_view_extension"
+require "filterrific/active_record_extension"
 
 module Filterrific
   class Engine < ::Rails::Engine
-
     # It's an engine so that we can add javascript and image assets
     # to the asset pipeline.
 
@@ -27,6 +24,5 @@ module Filterrific
     ActiveSupport.on_load :active_record do
       extend Filterrific::ActiveRecordExtension
     end
-
   end
 end

data/lib/filterrific/has_reset_filterrific_url_mixin.rb

@@ -1,16 +1,13 @@
-# -*- coding: utf-8 -*-
 #
 # Adds reset_filterrific_url to controllers and views
 #
 module Filterrific
   module HasResetFilterrificUrlMixin
-
     # Returns a url that can be used to reset the Filterrific params
     def reset_filterrific_url(opts = {})
       url_for(
-        { filterrific: { reset_filterrific: true } }.merge(opts)
+        {filterrific: {reset_filterrific: true}}.merge(opts)
       )
     end
-
   end
 end