mobility 0.8.8 → 1.0.0.alpha

data/Rakefile

@@ -2,7 +2,9 @@ require "bundler/gem_tasks"
 require "rspec/core/rake_task"
 require "yaml"
 
-RSpec::Core::RakeTask.new(:spec)
+RSpec::Core::RakeTask.new(:spec) do |task|
+  task.rspec_opts = '-f p'
+end
 
 task :default => :spec
 
@@ -18,7 +20,7 @@ namespace :db do
   desc "Create the database"
   task create: :setup do
     commands = {
-      "mysql"    => "mysql -u #{config['username']} -e 'create database #{config["database"]} default character set #{config["encoding"]} default collate #{config["collation"]};' >/dev/null",
+      "mysql"    => "mysql -h #{config['host']} -P #{config['port']} -u #{config['username']} --password=#{config['password']} -e 'create database #{config["database"]} default character set #{config["encoding"]} default collate #{config["collation"]};' >/dev/null",
       "postgres" => "psql -c 'create database #{config['database']};' -U #{config['username']} >/dev/null"
     }
     %x{#{commands[driver] || true}}
@@ -28,7 +30,7 @@ namespace :db do
   desc "Drop the database"
   task drop: :setup do
     commands = {
-      "mysql"    => "mysql -u #{config['username']} -e 'drop database #{config["database"]};' >/dev/null",
+      "mysql"    => "mysql -h #{config['host']} -P #{config['port']} -u #{config['username']} --password=#{config['password']} -e 'drop database #{config["database"]};' >/dev/null",
       "postgres" => "psql -c 'drop database #{config['database']};' -U #{config['username']} >/dev/null"
     }
     %x{#{commands[driver] || true}}
@@ -42,8 +44,8 @@ namespace :db do
 
     require orm
     require "database"
-    require "#{orm}/schema"
     DB = Mobility::Test::Database.connect(orm)
+    require "#{orm}/schema"
     Mobility::Test::Schema.up
   end
 

data/lib/mobility.rb

@@ -10,96 +10,36 @@ on a class. The {Mobility} module includes all necessary methods and modules to
 support defining backend accessors on a class.
 
 To enable Mobility on a class, simply include or extend the {Mobility} module,
-and define any attribute accessors using {Translates#mobility_accessor} (aliased to the
-value of {Mobility.accessor_method}, which defaults to +translates+).
+and define any attribute accessors using {Translates#translates}.
 
   class MyClass
     extend Mobility
     translates :title, backend: :key_value
   end
 
-When defining this module, Mobility attempts to +require+ various gems (for
-example, +active_record+ and +sequel+) to evaluate which are loaded. Loaded
-gems are tracked with dynamic subclasses of the {Loaded} module and referenced
-in backends to define gem-dependent behavior.
-
 =end
 module Mobility
-  require "mobility/attributes"
+  # A generic exception used by Mobility.
+  class Error < StandardError
+  end
+
   require "mobility/backend"
   require "mobility/backends"
-  require "mobility/backend_resetter"
-  require "mobility/configuration"
-  require "mobility/fallbacks"
-  require "mobility/loaded"
+  require "mobility/plugin"
   require "mobility/plugins"
-  require "mobility/translates"
+  require "mobility/translations"
 
   # General error for version compatibility conflicts
   class VersionNotSupportedError < ArgumentError; end
   CALL_COMPILABLE_REGEXP = /\A[a-zA-Z_]\w*[!?]?\z/
   private_constant :CALL_COMPILABLE_REGEXP
 
-  begin
-    require "rails"
-    Loaded::Rails = true
-  rescue LoadError => e
-    raise unless e.message =~ /rails/
-    Loaded::Rails = false
-  end
-
-  begin
-    require "active_record"
-    raise VersionNotSupportedError, "Mobility is only compatible with ActiveRecord 4.2 and greater" if ::ActiveRecord::VERSION::STRING < "4.2"
-    Loaded::ActiveRecord = true
-  rescue LoadError => e
-    raise unless e.message =~ /active_record/
-    Loaded::ActiveRecord = false
-  end
-
-  if Loaded::ActiveRecord
-    require "mobility/active_model"
-    require "mobility/active_record"
-    if Loaded::Rails
-      require "rails/generators/mobility/generators"
-    end
-  end
-
-  begin
-    require "sequel"
-    raise VersionNotSupportedError, "Mobility is only compatible with Sequel 4.0 and greater" if ::Sequel::MAJOR < 4
-    require "sequel/plugins/mobility"
-    unless defined?(ActiveSupport::Inflector)
-      # TODO: avoid automatically including the inflector extension
-      require "sequel/extensions/inflector"
-    end
-    require "sequel/plugins/dirty"
-    require "mobility/sequel"
-    Loaded::Sequel = true
-  rescue LoadError => e
-    raise unless e.message =~ /sequel/
-    Loaded::Sequel = false
-  end
+  require "rails/generators/mobility/generators" if defined?(Rails)
 
   class << self
     def extended(model_class)
-      return if model_class.respond_to? :mobility_accessor
-
-      model_class.extend Translates
-      model_class.extend ClassMethods
-      #TODO: Remove in v1.0
-      model_class.include InstanceMethods
-
-      if translates = Mobility.config.accessor_method
-        model_class.singleton_class.send(:alias_method, translates, :mobility_accessor)
-      end
-
-      if Loaded::ActiveRecord && model_class < ::ActiveRecord::Base
-        model_class.include(ActiveRecord)
-      end
-
-      if Loaded::Sequel && model_class < ::Sequel::Model
-        model_class.include(Sequel)
+      def model_class.translates(*args, **options)
+        include Mobility.translations_class.new(*args, **options)
       end
     end
 
@@ -110,6 +50,34 @@ module Mobility
       model_class.extend self
     end
 
+    # @return [Symbol,Class]
+    def default_backend
+      translations_class.defaults[:backend]
+    end
+
+    # Configure Mobility
+    # @yield [Mobility::Translations]
+    def configure
+      translates_with(Class.new(Translations)) unless @translations_class
+      yield translations_class
+    end
+    # @!endgroup
+
+    def translates_with(pluggable)
+      raise ArgumentError, "translations class must be a subclass of Module." unless Module === pluggable
+      @translations_class = pluggable
+    end
+
+    def translations_class
+      @translations_class ||
+        raise(Error, "Mobility has not been configured. "\
+              "Configure with Mobility.configure, or assign a translations class with Mobility.translates_with(<class>)")
+    end
+
+    def reset_translations_class
+      @translations_class = nil
+    end
+
     # @!group Locale Accessors
     # @return [Symbol] Mobility locale
     def locale
@@ -144,55 +112,6 @@ module Mobility
       RequestStore.store
     end
 
-    # @!group Configuration Methods
-    # @return [Mobility::Configuration] Mobility configuration
-    def config
-      @configuration ||= Configuration.new
-    end
-
-    # (see Mobility::Configuration#accessor_method)
-    # @!method accessor_method
-    #
-    # (see Mobility::Configuration#query_method)
-    # @!method query_method
-
-    # (see Mobility::Configuration#new_fallbacks)
-    # @!method new_fallbacks
-
-    # (see Mobility::Configuration#default_backend)
-    # @!method default_backend
-
-    # (see Mobility::Configuration#default_options)
-    # @!method default_options
-    #
-    # (see Mobility::Configuration#plugins)
-    # @!method plugins
-    #
-    # (see Mobility::Configuration#default_accessor_locales)
-    # @!method default_accessor_locales
-    %w[accessor_method query_method default_backend default_options plugins default_accessor_locales].each do |method_name|
-      define_method method_name do
-        config.public_send(method_name)
-      end
-    end
-
-    # TODO: Remove in v1.0
-    def default_fallbacks(*args)
-      config.public_send(:default_fallbacks, *args)
-    end
-
-    # TODO: Make private in v1.0
-    def new_fallbacks(*args)
-      config.public_send(:new_fallbacks, *args)
-    end
-
-    # Configure Mobility
-    # @yield [Mobility::Configuration] Mobility configuration
-    def configure
-      yield config
-    end
-    # @!endgroup
-
     # Return normalized locale
     # @param [String,Symbol] locale
     # @return [String] Normalized locale
@@ -228,17 +147,7 @@ module Mobility
     # @param [String,Symbol] locale
     # @raise [InvalidLocale] if locale is present but not available
     def enforce_available_locales!(locale)
-      # TODO: Remove conditional in v1.0
-      if I18n.enforce_available_locales
-        raise Mobility::InvalidLocale.new(locale) unless (I18n.locale_available?(locale) || locale.nil?)
-      else
-        warn <<-EOL
-WARNING: You called Mobility.enforce_available_locales! in a situation where
-I18n.enforce_available_locales is false. In the past, Mobility would do nothing
-in this case, but as of the next major release Mobility will ignore the I18n
-setting and enforce available locales whenever this method is called.
-EOL
-      end
+      raise Mobility::InvalidLocale.new(locale) unless (locale.nil? || available_locales.include?(locale.to_sym))
     end
 
     # Returns available locales. Defaults to I18n.available_locales, but will
@@ -250,7 +159,7 @@ EOL
     #   simply default to +I18n.available_locales+, we may define many more
     #   methods (in LocaleAccessors) than is really necessary.
     def available_locales
-      if Loaded::Rails && Rails.application
+      if defined?(Rails) && Rails.application
         Rails.application.config.i18n.available_locales || I18n.available_locales
       else
         I18n.available_locales
@@ -270,41 +179,6 @@ EOL
     end
   end
 
-  # TODO: Remove entire module in v1.0
-  module InstanceMethods
-    # Fetch backend for an attribute
-    # @deprecated Use mobility_backends[:<attribute>] instead.
-    # @param [String] attribute Attribute
-    def mobility_backend_for(attribute)
-      warn %{
-WARNING: mobility_backend_for is deprecated and will be removed in the next
-version of Mobility. Use <post>.<attribute>_backend instead.}
-      mobility_backends[attribute.to_sym]
-    end
-
-    def mobility
-      warn %{
-WARNING: <post>.mobility is deprecated and will be removed in the next
-version of Mobility. To get backends, use <post>.<attribute>_backend instead.}
-      @mobility ||= Adapter.new(self)
-    end
-
-    class Adapter < Struct.new(:model)
-      def backend_for(attribute)
-        model.mobility_backends[attribute.to_sym]
-      end
-    end
-    private_constant :Adapter
-  end
-
-  module ClassMethods
-    # Return translated attribute names on this model.
-    # @return [Array<String>] Attribute names
-    def mobility_attributes
-      []
-    end
-  end
-
   class InvalidLocale < I18n::InvalidLocale; end
   class NotImplementedError < StandardError; end
 end

data/lib/mobility/active_record/translation.rb

@@ -6,7 +6,7 @@ module Mobility
 
       belongs_to :translatable, polymorphic: true, touch: true
 
-      validates :key, presence: true, uniqueness: { scope: [:translatable_id, :translatable_type, :locale] }
+      validates :key, presence: true, uniqueness: { scope: [:translatable_id, :translatable_type, :locale], case_sensitive: true }
       validates :translatable, presence: true
       validates :locale, presence: true
     end

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

@@ -49,7 +49,7 @@ module Mobility
           case other
           when NilClass
             to_question.not
-          when Integer, Array, Hash
+          when Integer, Array, ::Hash
             to_dash_arrow.eq other.to_json
           when Jsonb
             to_dash_arrow.eq other.to_dash_arrow

data/lib/mobility/backend.rb

@@ -1,6 +1,4 @@
 # frozen-string-literal: true
-require "mobility/backend/orm_delegator"
-
 module Mobility
 =begin
 
@@ -9,10 +7,9 @@ Defines a minimum set of shared components included in any backend. These are:
 - a reader returning the +model+ on which the backend is defined ({#model})
 - a reader returning the +attribute+ for which the backend is defined
   ({#attribute})
-- a constructor setting these two elements (+model+, +attribute+), and
-  extracting fallbacks from the options hash ({#initialize})
+- a constructor setting these two elements (+model+, +attribute+)
 - a +setup+ method adding any configuration code to the model class
-  ({Setup#setup})
+  ({ClassMethods#setup})
 
 On top of this, a backend will normally:
 
@@ -50,7 +47,7 @@ On top of this, a backend will normally:
     end
   end
 
-@see Mobility::Attributes
+@see Mobility::Translations
 
 =end
 
@@ -66,9 +63,9 @@ On top of this, a backend will normally:
     # @!macro [new] backend_constructor
     #   @param model Model on which backend is defined
     #   @param [String] attribute Backend attribute
-    def initialize(model, attribute)
-      @model = model
-      @attribute = attribute
+    def initialize(*args)
+      @model = args[0]
+      @attribute = args[1]
     end
 
     # @!macro [new] backend_reader
@@ -118,17 +115,7 @@ On top of this, a backend will normally:
     # Extend included class with +setup+ method and other class methods
     def self.included(base)
       base.extend ClassMethods
-      def base.options
-        @options
-      end
-      base.option_reader :model_class
-    end
-
-    # @param [String] attribute
-    # @return [String] name of backend reader method
-    def self.method_name(attribute)
-      @backend_method_names ||= {}
-      @backend_method_names[attribute] ||= "#{attribute}_backend"
+      base.singleton_class.attr_reader :options, :model_class
     end
 
     # Defines setup hooks for backend to customize model class.
@@ -152,6 +139,7 @@ On top of this, a backend will normally:
       def inherited(subclass)
         subclass.instance_variable_set(:@setup_block, @setup_block)
         subclass.instance_variable_set(:@options, @options)
+        subclass.instance_variable_set(:@model_class, @model_class)
       end
 
       # Call setup block on a class with attributes and options.
@@ -166,14 +154,14 @@ On top of this, a backend will normally:
       # Build a subclass of this backend class for a given set of options
       # @note This method also freezes the options hash to prevent it from
       #   being changed.
+      # @param [Class] model_class Class
       # @param [Hash] options
       # @return [Class] backend subclass
-      def with_options(options = {}, &block)
-        configure(options) if respond_to?(:configure)
-        options.freeze
+      def build_subclass(model_class, options)
         Class.new(self) do
-          @options = options
-          class_eval(&block) if block_given?
+          @model_class = model_class
+          configure(options) if respond_to?(:configure)
+          @options = options.freeze
         end
       end
 
@@ -191,18 +179,6 @@ On top of this, a backend will normally:
         EOM
       end
 
-      # {Attributes} uses this method to get a backend class specific to the
-      # model using the backend. Backend classes can override this method to
-      # return a class specific to the model class using the backend (e.g.
-      # either an ActiveRecord or Sequel backend class depending on whether the
-      # model is an ActiveRecord model or a Sequel model.)
-      # @see OrmDelegator
-      # @see Attributes
-      # @return [self] returns itself
-      def for(_)
-        self
-      end
-
       # Called from plugins to apply custom processing for this backend.
       # Name is the name of the plugin.
       # @param [Symbol] name Name of plugin
@@ -220,10 +196,12 @@ On top of this, a backend will normally:
     end
 
     Translation = Struct.new(:backend, :locale) do
-      %w[read write].each do |accessor|
-        define_method accessor do |*args|
-          backend.send(accessor, locale, *args)
-        end
+      def read(options = {})
+        backend.read(locale, options)
+      end
+
+      def write(value, options = {})
+        backend.write(locale, value, options)
       end
     end
   end

data/lib/mobility/backends.rb

@@ -1,4 +1,24 @@
 module Mobility
   module Backends
+    @backends = {}
+
+    class << self
+      # @param [Symbol, Object] backend Name of backend to load.
+      def load_backend(name)
+        return name if Module === name
+
+        unless (backend = @backends[name])
+          require "mobility/backends/#{name}"
+          raise LoadError, "backend #{name} did not register itself correctly in Mobility::Backends" unless (backend = @backends[name])
+        end
+        backend
+      end
+    end
+
+    def self.register_backend(name, mod)
+      @backends[name] = mod
+    end
+
+    class LoadError < Error; end
   end
 end