mobility 0.1.18 → 0.1.19

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 608d78d453d3a4a8c70d081851ad41b7469ea359
-  data.tar.gz: 25a56ec92b1f17a64b0f7ff2305709ac461a0b21
+  metadata.gz: cfaed87c8d2aa5d4c794e315b8b4080f94101f9a
+  data.tar.gz: d5fd629b130d538e419ac31522b9f6441d9d85b4
 SHA512:
-  metadata.gz: 0ee3ca1d2c69def85e662c064809c611cd1f93f4ec512ba85fdd011b4349de739175421cd6a1658060691e8ff8122c808056c254c24bfba2c7bfc803fd1fad7e
-  data.tar.gz: 4d32541308d245ea15bc1a2867876448e5fc637c30a794127807c5f63789c21f0f221160a54264777866a07416c7e09ff6d1d5df4e223ad3de74459fd54d788b
+  metadata.gz: 05b16517848c01ddc40f836ca085883b986dcff714ed74787e920e11e8bd376eafa7bcaf26ef74957ed0052c9c74d11b9acdf03ce68c4512afbc0c284a74e8bd
+  data.tar.gz: d593ee3f8c8bdc1b7a8f5ebc2928677dd89d4a146b0463839cba92e4d4360ed246317b67593e7c7a83c0f96840ef83a499e3e8b6280a1b7dbdbf3e3706d0189b

data/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## 0.1
 
+### 0.1.19 (July 16, 2017)
+* Partial support for AR 4.2 ([#46](https://github.com/shioyama/mobility/pull/46))
+* Fix issues with Sequel >= 4.46.0 ([#47](https://github.com/shioyama/mobility/pull/47))
+* Include anonymous modules instead of defining methods directly on class
+  ([049a5f](https://github.com/shioyama/mobility/commit/049a5f90fd898d82984d2fe1af1646fda48ad142),
+  [d8fe42](https://github.com/shioyama/mobility/commit/d8fe42f81211640125e6a50bf681d45dbaa71c40),
+  [9cc3d0](https://github.com/shioyama/mobility/commit/9cc3d0e8c3f813c15213848f305e363c4eec6b8e))
+
 ### 0.1.18 (June 21, 2017)
 * Fix deprecation warnings when using ActiveRecord 5.1
   ([#44](https://github.com/shioyama/mobility/pull/44))

data/Gemfile

@@ -5,17 +5,20 @@ gemspec
 
 group :development, :test do
   if ENV['ORM'] == 'active_record'
-    if ENV['RAILS_VERSION'] == '5.1'
-      gem 'activerecord', '>= 5.1', '< 5.2'
-    else
+    if ENV['RAILS_VERSION'] == '5.0'
       gem 'activerecord', '>= 5.0', '< 5.1'
+    elsif ENV['RAILS_VERSION'] == '4.2'
+      gem 'activerecord', '>= 4.2.6', '< 5.0'
+    else
+      gem 'activerecord', '>= 5.1', '< 5.2'
     end
     gem "generator_spec", '~> 0.9.4'
-  end
-
-  if ENV['ORM'] == 'sequel'
-    # some interal API changes are breaking specs, limit to 4.45.x for now
-    gem 'sequel', '>= 4.41.0', '< 4.46.0'
+  elsif ENV['ORM'] == 'sequel'
+    if ENV['SEQUEL_VERSION'] == '4.41'
+      gem 'sequel', '>= 4.41.0', '< 4.46.0'
+    else
+      gem 'sequel', '>= 4.46.0'
+    end
   end
 
   platforms :ruby do

data/Gemfile.lock

@@ -1,33 +1,33 @@
 PATH
   remote: .
   specs:
-    mobility (0.1.17)
+    mobility (0.2.0.alpha)
       i18n (>= 0.6.10, < 0.9)
       request_store (~> 1.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    actionpack (5.1.1)
-      actionview (= 5.1.1)
-      activesupport (= 5.1.1)
+    actionpack (5.1.2)
+      actionview (= 5.1.2)
+      activesupport (= 5.1.2)
       rack (~> 2.0)
       rack-test (~> 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.0.2)
-    actionview (5.1.1)
-      activesupport (= 5.1.1)
+    actionview (5.1.2)
+      activesupport (= 5.1.2)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.0.3)
-    activemodel (5.1.1)
-      activesupport (= 5.1.1)
-    activerecord (5.1.1)
-      activemodel (= 5.1.1)
-      activesupport (= 5.1.1)
+    activemodel (5.1.2)
+      activesupport (= 5.1.2)
+    activerecord (5.1.2)
+      activemodel (= 5.1.2)
+      activesupport (= 5.1.2)
       arel (~> 8.0)
-    activesupport (5.1.1)
+    activesupport (5.1.2)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (~> 0.7)
       minitest (~> 5.1)
@@ -39,7 +39,7 @@ GEM
     concurrent-ruby (1.0.5)
     database_cleaner (1.6.1)
     diff-lcs (1.3)
-    erubi (1.6.0)
+    erubi (1.6.1)
     ffi (1.9.18)
     formatador (0.2.5)
     generator_spec (0.9.4)
@@ -59,7 +59,7 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    i18n (0.8.4)
+    i18n (0.8.6)
     listen (3.1.5)
       rb-fsevent (~> 0.9, >= 0.9.4)
       rb-inotify (~> 0.9, >= 0.9.7)
@@ -93,14 +93,14 @@ GEM
       nokogiri (>= 1.6)
     rails-html-sanitizer (1.0.3)
       loofah (~> 2.0)
-    railties (5.1.1)
-      actionpack (= 5.1.1)
-      activesupport (= 5.1.1)
+    railties (5.1.2)
+      actionpack (= 5.1.2)
+      activesupport (= 5.1.2)
       method_source
       rake (>= 0.8.7)
       thor (>= 0.18.1, < 2.0)
     rake (10.5.0)
-    rb-fsevent (0.9.8)
+    rb-fsevent (0.10.2)
     rb-inotify (0.9.10)
       ffi (>= 0.5.0, < 2)
     request_store (1.3.2)
@@ -146,4 +146,4 @@ DEPENDENCIES
   yard (~> 0.9.0)
 
 BUNDLED WITH
-   1.15.0
+   1.15.1

data/README.md

@@ -47,7 +47,7 @@ Installation
 Add this line to your application's Gemfile:
 
 ```ruby
-gem 'mobility', '~> 0.1.18'
+gem 'mobility', '~> 0.1.19'
 ```
 
 To translate attributes on a model, include (or extend) `Mobility`, then call
@@ -58,9 +58,9 @@ To translate attributes on a model, include (or extend) `Mobility`, then call
 Requirements:
 - ActiveRecord >= 5.0
 
-(Support for some backends is also supported with ActiveRecord/Rails 4.2, see
-the [active_record-4.2
-branch](https://github.com/shioyama/mobility/tree/active_record_4.2).)
+(Support for most backends and features is also supported with
+ActiveRecord/Rails 4.2, but there are some tests still failing. To see exactly
+what might not work, check pending specs in Rails 4.2 Travis builds.)
 
 If using Mobility in a Rails project, you can run the generator to create an
 initializer and a migration to create shared translation tables for the

data/lib/generators/rails/mobility/active_record_migration_compatibility.rb

@@ -0,0 +1,14 @@
+require "rails/generators/active_record"
+require "active_record/migration"
+
+module Mobility
+  module ActiveRecordMigrationCompatibility
+    def activerecord_migration_class
+      if ::ActiveRecord::Migration.respond_to?(:current_version)
+        "ActiveRecord::Migration[#{::ActiveRecord::Migration.current_version}]"
+      else
+        "ActiveRecord::Migration"
+      end
+    end
+  end
+end

data/lib/generators/rails/mobility/backend_generators/base.rb

@@ -6,6 +6,7 @@ module Mobility
     class Base < ::Rails::Generators::NamedBase
       argument :attributes, type: :array, default: []
       include ::ActiveRecord::Generators::Migration
+      include ::Mobility::ActiveRecordMigrationCompatibility
 
       def create_migration_file
         if self.class.migration_exists?(migration_dir, migration_file)

data/lib/generators/rails/mobility/generators.rb

@@ -1,5 +1,6 @@
 require "rails/generators"
 
+require_relative "./active_record_migration_compatibility"
 require_relative "./install_generator"
 require_relative "./translations_generator"
 require_relative "./backend_generators/base"

data/lib/generators/rails/mobility/install_generator.rb

@@ -4,6 +4,7 @@ require "rails/generators/active_record"
 module Mobility
   class InstallGenerator < ::Rails::Generators::Base
     include ::Rails::Generators::Migration
+    include ::Mobility::ActiveRecordMigrationCompatibility
 
     desc "Generates migrations to add translations tables."
 

data/lib/generators/rails/mobility/templates/column_translations.rb

@@ -1,4 +1,4 @@
-class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
+class <%= migration_class_name %> < <%= activerecord_migration_class %>
   def change
 <% attributes.each do |attribute| -%>
 <% I18n.available_locales.each do |locale| -%>

data/lib/generators/rails/mobility/templates/create_string_translations.rb

@@ -1,4 +1,4 @@
-class CreateStringTranslations < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
+class CreateStringTranslations < <%= activerecord_migration_class %>
 
   def change
     create_table :mobility_string_translations do |t|

data/lib/generators/rails/mobility/templates/create_text_translations.rb

@@ -1,4 +1,4 @@
-class CreateTextTranslations < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
+class CreateTextTranslations < <%= activerecord_migration_class %>
 
   def change
     create_table :mobility_text_translations do |t|

data/lib/generators/rails/mobility/templates/table_migration.rb

@@ -1,9 +1,9 @@
-class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
+class <%= migration_class_name %> < <%= activerecord_migration_class %>
   def change
 <% attributes.each do |attribute| -%>
   <%- if attribute.reference? -%>
     add_reference :<%= table_name %>, :<%= attribute.name %><%= attribute.inject_options %>
-  <%- elsif attribute.token? -%>
+  <%- elsif attribute.respond_to?(:token?) && attribute.token? -%>
     add_column :<%= table_name %>, :<%= attribute.name %>, :string<%= attribute.inject_options %>
     add_index :<%= table_name %>, :<%= attribute.index_name %><%= attribute.inject_index_options %>, unique: true
   <%- else -%>

data/lib/generators/rails/mobility/templates/table_translations.rb

@@ -1,10 +1,10 @@
-class <%= migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::Migration.current_version %>]
+class <%= migration_class_name %> < <%= activerecord_migration_class %>
   def change
-    create_table :<%= table_name %><%= primary_key_type %> do |t|
+    create_table :<%= table_name %><%= primary_key_type if respond_to?(:primary_key_type) %> do |t|
 
       # Translated attribute(s)
 <% attributes.each do |attribute| -%>
-<% if attribute.token? -%>
+<% if attribute.respond_to?(:token?) && attribute.token? -%>
       t.string :<%= attribute.name %><%= attribute.inject_options %>
 <% else -%>
       t.<%= attribute.type %> :<%= attribute.name %><%= attribute.inject_options %>

data/lib/mobility.rb

@@ -48,7 +48,7 @@ module Mobility
 
   begin
     require "active_record"
-    raise VersionNotSupportedError, "Mobility is only compatible with ActiveRecord 5.0 and greater" if ::ActiveRecord::VERSION::MAJOR < 5
+    raise VersionNotSupportedError, "Mobility is only compatible with ActiveRecord 4.2 and greater" if ::ActiveRecord::VERSION::STRING < "4.2"
     autoload :ActiveModel,      "mobility/active_model"
     autoload :ActiveRecord,     "mobility/active_record"
     Loaded::ActiveRecord = true
@@ -79,7 +79,7 @@ module Mobility
   class << self
     def extended(model_class)
       return if model_class.respond_to? :mobility_accessor
-      model_class.class_eval do
+      mod = Module.new do
         # Fetch backend for an attribute
         # @param [String] attribute Attribute
         def mobility_backend_for(attribute)
@@ -90,14 +90,12 @@ module Mobility
           @mobility_backends = nil
           super
         end
+      end
+      model_class.include(mod)
 
-        class << self
-          include ClassMethods
-
-          if translates = Mobility.config.accessor_method
-            alias_method translates, :mobility_accessor
-          end
-        end
+      model_class.extend(ClassMethods)
+      if translates = Mobility.config.accessor_method
+        model_class.singleton_class.send(:alias_method, translates, :mobility_accessor)
       end
 
       if Loaded::ActiveRecord

data/lib/mobility/active_record/attribute_methods.rb

@@ -13,13 +13,15 @@ ancestors.
       # @return [Array<String>] Model attributes
       # @!method attributes
       def self.included(model)
-        model.class_eval do
-          alias_method :untranslated_attributes, :attributes
-
+        attributes_method = Module.new do
           def attributes
             super.merge(translated_attributes)
           end
         end
+        model.class_eval do
+          alias_method :untranslated_attributes, :attributes
+          include attributes_method
+        end
       end
 
       # Translated attributes defined on model.

data/lib/mobility/active_record/backend_resetter.rb

@@ -12,14 +12,12 @@ Backend resetter for ActiveRecord models. Adds hook on +reload+ event to
       def included(model_class)
         model_reset_method = @model_reset_method
 
-        model_class.class_eval do
-          mod = Module.new do
-            define_method :reload do |*args|
-              super(*args).tap { instance_eval(&model_reset_method) }
-            end
+        mod = Module.new do
+          define_method :reload do |*args|
+            super(*args).tap { instance_eval(&model_reset_method) }
           end
-          include mod
         end
+        model_class.include mod
       end
     end
   end

data/lib/mobility/backend/active_record/dirty.rb

@@ -1,5 +1,11 @@
 module Mobility
   module Backend
+=begin
+
+Dirty tracking for AR models. See {Mobility::Backend::ActiveModel::Dirty} for
+details on usage.
+
+=end
     module ActiveRecord::Dirty
       include ActiveModel::Dirty
 
@@ -9,22 +15,24 @@ module Mobility
         backend_class.extend(ClassMethods)
       end
 
+      # Adds hook after {Backend::Setup#setup_model} to patch AR so that it
+      # handles changes to translated attributes just like normal attributes.
       module ClassMethods
+        # (see Mobility::Backend::Setup#setup_model)
         def setup_model(model_class, attributes, **options)
           super
 
           method_name_regex = /\A(#{attributes.join('|'.freeze)})_([a-z]{2}(_[a-z]{2})?)(=?|\??)\z/.freeze
-          mod = Module.new do
+          has_attribute = Module.new do
             define_method :has_attribute? do |attr_name|
               super(attr_name) || !!method_name_regex.match(attr_name)
             end
           end
+          model_class.extend has_attribute
 
-          model_class.class_eval do
-            extend mod
-
-            method_name = ::ActiveRecord::VERSION::STRING < '5.1' ? :changes_applied : :changes_internally_applied
-            define_method method_name do
+          changes_applied_method = ::ActiveRecord::VERSION::STRING < '5.1' ? :changes_applied : :changes_internally_applied
+          mod = Module.new do
+            define_method changes_applied_method do
               @previously_changed = changes
               super()
             end
@@ -38,6 +46,7 @@ module Mobility
               (@previously_changed ||= ActiveSupport::HashWithIndifferentAccess.new).merge(super)
             end
           end
+          model_class.include mod
         end
       end
     end

data/lib/mobility/backend/active_record/hstore/query_methods.rb

@@ -25,7 +25,7 @@ module Mobility
               ops ? ops.and(op) : op
             }
 
-            opts.empty? ? where(i18n_query) : super(opts, *rest).where(i18n_query)
+            opts.empty? ? super(i18n_query) : super(opts, *rest).where(i18n_query)
           else
             super(opts, *rest)
           end

data/lib/mobility/backend/active_record/jsonb/query_methods.rb

@@ -26,7 +26,7 @@ module Mobility
               ops ? ops.and(op) : op
             }
 
-            opts.empty? ? where(i18n_query) : super(opts, *rest).where(i18n_query)
+            opts.empty? ? super(i18n_query) : super(opts, *rest).where(i18n_query)
           else
             super(opts, *rest)
           end

data/lib/mobility/backend/sequel/column/query_methods.rb

@@ -5,12 +5,16 @@ module Mobility
         super
         attributes_extractor = @attributes_extractor
 
-        define_method :_filter_or_exclude do |invert, clause, cond, &block|
-          if keys = attributes_extractor.call(cond)
-            cond = cond.dup
-            keys.each { |attr| cond[Column.column_name_for(attr)] = cond.delete(attr) }
+        %w[exclude or where].each do |method_name|
+          define_method method_name do |*conds, &block|
+            if keys = attributes_extractor.call(conds.first)
+              cond = conds.first.dup
+              keys.each { |attr| cond[Column.column_name_for(attr)] = cond.delete(attr) }
+              super(cond, &block)
+            else
+              super(*conds, &block)
+            end
           end
-          super(invert, clause, cond, &block)
         end
 
         attributes.each do |attribute|

data/lib/mobility/backend/sequel/hstore/query_methods.rb

@@ -1,34 +1,16 @@
+require 'mobility/backend/sequel/postgres_query_methods'
+
 Sequel.extension :pg_hstore, :pg_hstore_ops
 
 module Mobility
   module Backend
     class Sequel::Hstore::QueryMethods < Sequel::QueryMethods
+      include PostgresQueryMethods
+
       def initialize(attributes, **)
         super
-        attributes_extractor = @attributes_extractor
-
-        define_method :_filter_or_exclude do |invert, clause, *conds, &block|
-          if (clause == :where) && i18n_keys = attributes_extractor.call(conds.first)
-            locale = Mobility.locale.to_s
-            table_name = model.table_name
-            cond = conds.first
 
-            i18n_query = i18n_keys.inject(::Sequel.expr(!invert)) do |expr, attr|
-              value = cond.delete(attr)
-              attr_hstore = ::Sequel.hstore_op(attr)
-              contains_value = attr_hstore.contains({ locale => value.to_s })
-              has_key = attr_hstore.has_key?(locale)
-              if invert
-                expr.|(has_key & ~contains_value)
-              else
-                expr.&(value.nil? ? ~has_key : contains_value)
-              end
-            end
-            super(invert, clause, *conds, &block).where(i18n_query)
-          else
-            super(invert, clause, *conds, &block)
-          end
-        end
+        define_query_methods("hstore")
 
         attributes.each do |attribute|
           define_method :"first_by_#{attribute}" do |value|

data/lib/mobility/backend/sequel/jsonb/query_methods.rb

@@ -1,34 +1,16 @@
+require 'mobility/backend/sequel/postgres_query_methods'
+
 Sequel.extension :pg_json, :pg_json_ops
 
 module Mobility
   module Backend
     class Sequel::Jsonb::QueryMethods < Sequel::QueryMethods
+      include PostgresQueryMethods
+
       def initialize(attributes, **)
         super
-        attributes_extractor = @attributes_extractor
-
-        define_method :_filter_or_exclude do |invert, clause, *conds, &block|
-          if (clause == :where) && i18n_keys = attributes_extractor.call(conds.first)
-            locale = Mobility.locale.to_s
-            table_name = model.table_name
-            cond = conds.first
 
-            i18n_query = i18n_keys.inject(::Sequel.expr(!invert)) do |expr, attr|
-              value = cond.delete(attr)
-              attr_jsonb = ::Sequel.pg_jsonb_op(attr)
-              contains_value = attr_jsonb.contains({ locale => value })
-              has_key = attr_jsonb.has_key?(locale)
-              if invert
-                expr.|(has_key & ~contains_value)
-              else
-                expr.&(value.nil? ? ~has_key : contains_value)
-              end
-            end
-            super(invert, clause, *conds, &block).where(i18n_query)
-          else
-            super(invert, clause, *conds, &block)
-          end
-        end
+        define_query_methods("pg_jsonb")
 
         attributes.each do |attribute|
           define_method :"first_by_#{attribute}" do |value|

data/lib/mobility/backend/sequel/key_value/query_methods.rb

@@ -39,20 +39,20 @@ module Mobility
         #
         attributes_extractor = @attributes_extractor
 
-        define_method :_filter_or_exclude do |invert, clause, *cond, &block|
-          if i18n_keys = attributes_extractor.call(cond.first)
-            cond = cond.first.dup
-            i18n_nulls = i18n_keys.select { |key| cond[key].nil? }
-            i18n_keys.each { |attr| cond[::Sequel[:"#{attr}_#{association_name}"][:value]] = cond.delete(attr) }
-            super(invert, clause, cond, &block).
-              send("join_#{association_name}", *(i18n_keys - i18n_nulls)).
-              send("join_#{association_name}", *i18n_nulls, outer_join: true)
-          else
-            super(invert, clause, *cond, &block)
+        %w[exclude or where].each do |method_name|
+          define_method method_name do |*conds, &block|
+            if i18n_keys = attributes_extractor.call(conds.first)
+              cond = conds.first.dup
+              i18n_nulls = i18n_keys.select { |key| cond[key].nil? }
+              i18n_keys.each { |attr| cond[::Sequel[:"#{attr}_#{association_name}"][:value]] = cond.delete(attr) }
+              super(cond, &block).
+                send("join_#{association_name}", *(i18n_keys - i18n_nulls), outer_join: method_name == "or").
+                send("join_#{association_name}", *i18n_nulls, outer_join: true)
+            else
+              super(*conds, &block)
+            end
           end
         end
-        private :_filter_or_exclude
-
       end
     end
   end

data/lib/mobility/backend/sequel/postgres_query_methods.rb

@@ -0,0 +1,42 @@
+
+module Mobility
+  module Backend
+    module PostgresQueryMethods
+      private
+
+      def define_query_methods(column_type)
+        attributes_extractor = @attributes_extractor
+
+        %w[exclude or where].each do |method_name|
+          invert = method_name == "exclude"
+
+          define_method method_name do |*cond, &block|
+            if i18n_keys = attributes_extractor.call(cond.first)
+              locale = Mobility.locale.to_s
+              cond = cond.first
+
+              i18n_query = i18n_keys.inject(::Sequel.expr(!invert)) do |expr, attr|
+                value = cond.delete(attr)
+                op = ::Sequel.send(:"#{column_type}_op", attr)
+                contains_value = op.contains({ locale => value.to_s })
+                has_key = op.has_key?(locale)
+                if invert
+                  expr.|(has_key & ~contains_value)
+                else
+                  expr.&(value.nil? ? ~has_key : contains_value)
+                end
+              end
+              if method_name == "or"
+                cond.empty? ? super(i18n_query, &block) : super(::Sequel.&(cond, i18n_query), &block)
+              else
+                super(cond, &block).where(i18n_query)
+              end
+            else
+              super(*cond, &block)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/mobility/backend/sequel/table/query_methods.rb

@@ -35,14 +35,16 @@ module Mobility
         # See note in AR Table QueryMethods class about limitations of
         # query methods on translated attributes when searching on nil values.
         #
-        define_method :_filter_or_exclude do |invert, clause, *cond, &block|
-          if i18n_keys = attributes_extractor.call(cond.first)
-            cond = cond.first.dup
-            outer_join = i18n_keys.all? { |key| cond[key].nil? }
-            i18n_keys.each { |attr| cond[::Sequel[translation_class.table_name][attr]] = cond.delete(attr) }
-            super(invert, clause, cond, &block).send("join_#{association_name}", outer_join: outer_join)
-          else
-            super(invert, clause, *cond, &block)
+        %w[exclude or where].each do |method_name|
+          define_method method_name do |*conds, &block|
+            if i18n_keys = attributes_extractor.call(conds.first)
+              cond = conds.first.dup
+              outer_join = method_name == "or" || i18n_keys.all? { |key| cond[key].nil? }
+              i18n_keys.each { |attr| cond[::Sequel[translation_class.table_name][attr]] = cond.delete(attr) }
+              super(cond, &block).send("join_#{association_name}", outer_join: outer_join)
+            else
+              super(*conds, &block)
+            end
           end
         end
       end

data/lib/mobility/version.rb

@@ -1,3 +1,3 @@
 module Mobility
-  VERSION = "0.1.18"
+  VERSION = "0.1.19"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mobility
 version: !ruby/object:Gem::Version
-  version: 0.1.18
+  version: 0.1.19
 platform: ruby
 authors:
 - Chris Salzberg
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-06-22 00:00:00.000000000 Z
+date: 2017-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: request_store
@@ -135,6 +135,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- lib/generators/rails/mobility/active_record_migration_compatibility.rb
 - lib/generators/rails/mobility/backend_generators/base.rb
 - lib/generators/rails/mobility/backend_generators/column_backend.rb
 - lib/generators/rails/mobility/backend_generators/table_backend.rb
@@ -199,6 +200,7 @@ files:
 - lib/mobility/backend/sequel/jsonb/query_methods.rb
 - lib/mobility/backend/sequel/key_value.rb
 - lib/mobility/backend/sequel/key_value/query_methods.rb
+- lib/mobility/backend/sequel/postgres_query_methods.rb
 - lib/mobility/backend/sequel/query_methods.rb
 - lib/mobility/backend/sequel/serialized.rb
 - lib/mobility/backend/sequel/serialized/query_methods.rb
@@ -245,7 +247,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.12
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Pluggable Ruby translation framework