mobility 1.3.0.rc3 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd0b53980f62d9538827053f6e08b8ffd287d44a7a4aa87602c5d85e87bf30fc
-  data.tar.gz: 540af4290175ec08b109a8cc0e783c1a634c66fe87bf97fd014d5c5a0f78b79c
+  metadata.gz: 488c8b1ec0f603d226b7a0dd54064a27ede153b1c8c3bd1ae078bcfec0dd74e2
+  data.tar.gz: ebce70b5815c7618b908459c59bc514647b55f2e792641bc3d400547fb5ae533
 SHA512:
-  metadata.gz: 25fc7f6a40125020f7425691a3d183a290bfaa3381594d1ad218a924797a0329f76ca39494742f380a3310fbe9d3c879849e366de20bd695219b247c90161d17
-  data.tar.gz: 8b02e9620070979a64e71d2774495a1be0054e92180116e4e40790575d05faff38fb78001ab460581de2524598af3823f312734b67804d2debe1de428737801a
+  metadata.gz: 8d8dc37e4dd38ead1eb16b593ca153876179770a23b32a63b2c6850509326590c8c80596351328b48c6e40e25df48b99f106e6cbcddd0b940381eaf0c7e41fd6
+  data.tar.gz: b0afa4c0962c240658c0e5fe109c76e6db533b7f1ad1679b4ed38094c8dd48cb581fb15f8ac50d57078238d98cade3198b562ee64d2ed1ca9c9af3f3bda5e72c

data/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## 1.3
 
+### 1.3.0
+
+- Only support ActiveRecord >= 7.0
+- Allow `I18n.available_locales` to contain Strings
+  ([#612](https://github.com/shioyama/mobility/pull/612))
+- Update CI config, add support for Rails 8
+  ([#653](https://github.com/shioyama/mobility/pull/653)), thanks
+  [d-rodriguez](https://github.com/n-rodriguez)!
+- Fix broken count statements in Active Record 8.0
+  ([#655](https://github.com/shioyama/mobility/pull/655)), thanks
+  [jukra](https://github.com/jukra)!
+
 ### 1.3.0.rc3
 
 - Don't try to load generators if Rails is loaded but AR is not

data/Gemfile

@@ -10,7 +10,7 @@ group :development, :test do
   when 'active_record'
     orm_version ||= '7.0'
     case orm_version
-    when '6.1', '7.0', '7.1'
+    when '6.1', '7.0', '7.1', '7.2', '8.0'
       gem 'activerecord', "~> #{orm_version}.0"
     when 'edge'
       git 'https://github.com/rails/rails.git', branch: 'main' do
@@ -45,19 +45,15 @@ group :development, :test do
     gem 'pry-byebug'
     case ENV['DB']
     when 'sqlite3'
-      if orm == 'active_record' && orm_version < '5.2'
-        gem 'sqlite3', '~> 1.3.13'
+      if orm == 'active_record' && orm_version >= '8.0'
+        gem 'sqlite3', '>= 2.1.0'
       else
-        gem 'sqlite3', '~> 1.4.1'
+        gem 'sqlite3', '~> 1.5.0'
       end
     when 'mysql'
       gem 'mysql2'
     when 'postgres'
-      if orm == 'active_record' && orm_version < '5.0'
-        gem 'pg', '< 1.0'
-      else
-        gem 'pg'
-      end
+      gem 'pg'
     end
   end
 end

data/Gemfile.lock

@@ -1,22 +1,46 @@
 PATH
   remote: .
   specs:
-    mobility (1.3.0.rc1)
+    mobility (1.3.0.rc3)
       i18n (>= 0.6.10, < 2)
       request_store (~> 1.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    benchmark-ips (2.13.0)
+    activemodel (7.1.5)
+      activesupport (= 7.1.5)
+    activerecord (7.1.5)
+      activemodel (= 7.1.5)
+      activesupport (= 7.1.5)
+      timeout (>= 0.4.0)
+    activesupport (7.1.5)
+      base64
+      benchmark (>= 0.3)
+      bigdecimal
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      connection_pool (>= 2.2.5)
+      drb
+      i18n (>= 1.6, < 2)
+      logger (>= 1.4.2)
+      minitest (>= 5.1)
+      mutex_m
+      securerandom (>= 0.3)
+      tzinfo (~> 2.0)
+    base64 (0.2.0)
+    benchmark (0.4.0)
+    benchmark-ips (2.14.0)
+    bigdecimal (3.1.8)
     byebug (11.1.3)
     coderay (1.1.3)
-    concurrent-ruby (1.2.3)
+    concurrent-ruby (1.3.4)
+    connection_pool (2.4.1)
     database_cleaner (1.99.0)
     diff-lcs (1.5.1)
-    ffi (1.16.3)
+    drb (2.2.1)
+    ffi (1.17.0-x86_64-linux-gnu)
     formatador (1.1.0)
-    guard (2.18.1)
+    guard (2.19.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
       lumberjack (>= 1.0.12, < 2.0)
@@ -30,13 +54,17 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    i18n (1.14.4)
+    i18n (1.14.6)
       concurrent-ruby (~> 1.0)
     listen (3.9.0)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
+    logger (1.6.1)
     lumberjack (1.2.10)
-    method_source (1.0.0)
+    method_source (1.1.0)
+    mini_portile2 (2.8.8)
+    minitest (5.25.2)
+    mutex_m (0.3.0)
     nenv (0.3.0)
     notiffany (0.1.3)
       nenv (~> 0.1)
@@ -47,35 +75,41 @@ GEM
     pry-byebug (3.10.1)
       byebug (~> 11.0)
       pry (>= 0.13, < 0.15)
-    rack (3.0.9.1)
+    rack (3.1.8)
     rake (12.3.3)
     rb-fsevent (0.11.2)
-    rb-inotify (0.10.1)
+    rb-inotify (0.11.1)
       ffi (~> 1.0)
-    request_store (1.6.0)
+    request_store (1.7.0)
       rack (>= 1.4)
     rspec (3.13.0)
       rspec-core (~> 3.13.0)
       rspec-expectations (~> 3.13.0)
       rspec-mocks (~> 3.13.0)
-    rspec-core (3.13.0)
+    rspec-core (3.13.2)
       rspec-support (~> 3.13.0)
-    rspec-expectations (3.13.0)
+    rspec-expectations (3.13.3)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-mocks (3.13.0)
+    rspec-mocks (3.13.2)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
     rspec-support (3.13.1)
+    securerandom (0.3.2)
     shellany (0.0.1)
-    sqlite3 (1.4.4)
-    thor (1.3.1)
-    yard (0.9.36)
+    sqlite3 (1.5.4)
+      mini_portile2 (~> 2.8.0)
+    thor (1.3.2)
+    timeout (0.4.2)
+    tzinfo (2.0.6)
+      concurrent-ruby (~> 1.0)
+    yard (0.9.37)
 
 PLATFORMS
   x86_64-linux
 
 DEPENDENCIES
+  activerecord (~> 7.1.0)
   benchmark-ips
   database_cleaner (~> 1.5, >= 1.5.3)
   guard-rspec
@@ -83,7 +117,7 @@ DEPENDENCIES
   pry-byebug
   rake (~> 12, >= 12.2.1)
   rspec (~> 3.0)
-  sqlite3 (~> 1.4.1)
+  sqlite3 (~> 1.5.0)
   yard (~> 0.9.0)
 
 BUNDLED WITH

data/README.md

@@ -60,7 +60,7 @@ gem 'mobility', '~> 1.3.0.rc3'
 ### ActiveRecord (Rails)
 
 Requirements:
-- ActiveRecord >= 6.1
+- ActiveRecord >= 7.0
 
 To translate attributes on a model, extend `Mobility`, then call `translates`
 passing in one or more attributes as well as a hash of options (see below).

data/lib/mobility/backends/active_record/container.rb

@@ -86,25 +86,6 @@ Implements the {Mobility::Backends::Container} backend for ActiveRecord models.
         end
       end
 
-      setup do |_attributes, options|
-        # Fix for duping depth-2 jsonb column in AR < 5.0
-        if ::ActiveRecord::VERSION::STRING < '5.0'
-          column_name = options[:column_name]
-          module_name = "MobilityArContainer#{column_name.to_s.camelcase}"
-          unless const_defined?(module_name)
-            dupable = Module.new do
-              class_eval <<-EOM, __FILE__, __LINE__ + 1
-                def initialize_dup(source)
-                  super
-                  self.#{column_name} = source.#{column_name}.deep_dup
-                end
-              EOM
-            end
-            include const_set(module_name, dupable)
-          end
-        end
-      end
-
       private
 
       def model_translations(locale)

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

@@ -79,28 +79,24 @@ locale suffix, so +title_en+, +title_pt_br+, etc.)
         )
 
         module InstanceMethods
-          if ::ActiveRecord::VERSION::STRING >= '5.1' # define patterns added in 5.1
-            def saved_changes
-              super.merge(mutations_from_mobility.previous_changes)
-            end
+          def saved_changes
+            super.merge(mutations_from_mobility.previous_changes)
+          end
 
-            def changes_to_save
-              super.merge(mutations_from_mobility.changes)
-            end
+          def changes_to_save
+            super.merge(mutations_from_mobility.changes)
+          end
 
-            def changed_attribute_names_to_save
-              super + mutations_from_mobility.changed
-            end
+          def changed_attribute_names_to_save
+            super + mutations_from_mobility.changed
+          end
 
-            def attributes_in_database
-              super.merge(mutations_from_mobility.changed_attributes)
-            end
+          def attributes_in_database
+            super.merge(mutations_from_mobility.changed_attributes)
+          end
 
-            if ::ActiveRecord::VERSION::STRING >= '6.0'
-              def has_changes_to_save?
-                super || mutations_from_mobility.changed?
-              end
-            end
+          def has_changes_to_save?
+            super || mutations_from_mobility.changed?
           end
 
           def reload(*)

data/lib/mobility/plugins/active_record/query.rb

@@ -137,19 +137,19 @@ enabled for any one attribute on the model.
           end
 
           def order(opts, *rest)
-            return super unless @klass.respond_to?(:mobility_attribute?)
+            return super unless klass.respond_to?(:mobility_attribute?)
 
             case opts
             when Symbol, String
-              @klass.mobility_attribute?(opts) ? order({ opts => :asc }, *rest) : super
+              klass.mobility_attribute?(opts) ? order({ opts => :asc }, *rest) : super
             when ::Hash
-              i18n_keys, keys = opts.keys.partition(&@klass.method(:mobility_attribute?))
+              i18n_keys, keys = opts.keys.partition(&klass.method(:mobility_attribute?))
               return super if i18n_keys.empty?
 
               base = keys.empty? ? self : super(opts.slice(keys))
 
               i18n_keys.inject(base) do |query, key|
-                backend_class = @klass.mobility_backend_class(key)
+                backend_class = klass.mobility_backend_class(key)
                 dir, node = opts[key], backend_node(key)
                 backend_class.apply_scope(query, node).order(node.send(dir.downcase))
               end
@@ -158,29 +158,27 @@ enabled for any one attribute on the model.
             end
           end
 
-          if ::ActiveRecord::VERSION::STRING >= '5.0'
-            %w[pluck group select].each do |method_name|
-              define_method method_name do |*attrs, &block|
-                return super(*attrs, &block) if (method_name == 'select' && block.present?)
+          %w[pluck group select].each do |method_name|
+            define_method method_name do |*attrs, &block|
+              return super(*attrs, &block) if (method_name == 'select' && block.present?)
 
-                return super(*attrs, &block) unless @klass.respond_to?(:mobility_attribute?)
+              return super(*attrs, &block) unless klass.respond_to?(:mobility_attribute?)
 
-                return super(*attrs, &block) unless attrs.any?(&@klass.method(:mobility_attribute?))
+              return super(*attrs, &block) unless attrs.any?(&klass.method(:mobility_attribute?))
 
-                keys = attrs.dup
+              keys = attrs.dup
 
-                base = keys.each_with_index.inject(self) do |query, (key, index)|
-                  next query unless @klass.mobility_attribute?(key)
-                  keys[index] = backend_node(key)
-                  if method_name == "select"
-                    keys[index] = keys[index]
-                      .as(::Mobility::Plugins::ActiveRecord::Query.attribute_alias(key.to_s))
-                  end
-                  @klass.mobility_backend_class(key).apply_scope(query, backend_node(key))
+              base = keys.each_with_index.inject(self) do |query, (key, index)|
+                next query unless klass.mobility_attribute?(key)
+                keys[index] = backend_node(key)
+                if method_name == "select" && query.order_values.any?
+                  keys[index] = keys[index]
+                    .as(::Mobility::Plugins::ActiveRecord::Query.attribute_alias(key.to_s))
                 end
-
-                base.public_send(method_name, *keys, &block)
+                klass.mobility_backend_class(key).apply_scope(query, backend_node(key))
               end
+
+              base.public_send(method_name, *keys, &block)
             end
           end
 
@@ -189,7 +187,7 @@ enabled for any one attribute on the model.
           # @param [Symbol] locale Locale
           # @return [Arel::Node] Arel node for this attribute in given locale
           def backend_node(name, locale = Mobility.locale)
-            @klass.mobility_backend_class(name)[name, locale]
+            klass.mobility_backend_class(name)[name, locale]
           end
 
           class WhereChain < ::ActiveRecord::QueryMethods::WhereChain

data/lib/mobility/plugins/arel/nodes/pg_ops.rb

@@ -26,17 +26,6 @@ module Mobility
           end)
         end
 
-        # Needed for AR 4.2, can be removed when support is deprecated
-        if ::ActiveRecord::VERSION::STRING < '5.0'
-          [JsonbDashDoubleArrow, HstoreDashArrow].each do |klass|
-            klass.class_eval do
-              def quoted_node other
-                other && super
-              end
-            end
-          end
-        end
-
         class Jsonb  < JsonbDashDoubleArrow
           def to_dash_arrow
             JsonbDashArrow.new left, right

data/lib/mobility/version.rb

@@ -9,7 +9,7 @@ module Mobility
     MAJOR = 1
     MINOR = 3
     TINY = 0
-    PRE = "rc3"
+    PRE = nil
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join(".")
   end

data/lib/mobility.rb

@@ -228,9 +228,9 @@ module Mobility
     #   methods (in LocaleAccessors) than is really necessary.
     def available_locales
       if defined?(Rails) && Rails.respond_to?(:application) && Rails.application
-        Rails.application.config.i18n.available_locales&.map(&:to_sym) || I18n.available_locales
+        Rails.application.config.i18n.available_locales&.map(&:to_sym) || I18n.available_locales.map(&:to_sym)
       else
-        I18n.available_locales
+        I18n.available_locales.map(&:to_sym)
       end
     end
 

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

@@ -78,7 +78,7 @@ Mobility.configure do
     # fallbacks
     #
     # Or uncomment this line to enable fallbacks with a global default.
-    # fallbacks { :pt => :en }
+    # fallbacks(pt: :en)
 
     # Presence
     #

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mobility
 version: !ruby/object:Gem::Version
-  version: 1.3.0.rc3
+  version: 1.3.0
 platform: ruby
 authors:
 - Chris Salzberg
@@ -35,7 +35,7 @@ cert_chain:
   3I5AfnFAG4/1IwhadqwF5cl3jOUa7n3mS2OJl3tRCGuPvwAA9MV10hmwbQTXMrNK
   tD9kfT9eseUE4mfPnIaHOs4FiIoHniA7zdtjB7GIQ4cEpB6o
   -----END CERTIFICATE-----
-date: 2024-03-31 00:00:00.000000000 Z
+date: 2024-11-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: request_store
@@ -270,9 +270,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.4.6
 signing_key: