spree_mobility 1.0.0

data/config/initializers/mobility.rb

@@ -0,0 +1,118 @@
+require 'mobility'
+
+Mobility.configure do
+  # PLUGINS
+  plugins do
+    # Backend
+    #
+    # Sets the default backend to use in models. This can be overridden in models
+    # by passing +backend: ...+ to +translates+.
+    #
+    # To default to a different backend globally, replace +:key_value+ by another
+    # backend name.
+    #
+    backend :table
+
+    # ActiveRecord
+    #
+    # Defines ActiveRecord as ORM, and enables ActiveRecord-specific plugins.
+    active_record
+
+    # Accessors
+    #
+    # Define reader and writer methods for translated attributes. Remove either
+    # to disable globally, or pass +reader: false+ or +writer: false+ to
+    # +translates+ in any translated model.
+    #
+    reader
+    writer
+
+    # Backend Reader
+    #
+    # Defines reader to access the backend for any attribute, of the form
+    # +<attribute>_backend+.
+    #
+    backend_reader
+    #
+    # Or pass an interpolation string to define a different pattern:
+    # backend_reader "%s_translations"
+
+    # Query
+    #
+    # Defines a scope on the model class which allows querying on
+    # translated attributes. The default scope is named +i18n+, pass a different
+    # name as default to change the global default, or to +translates+ in any
+    # model to change it for that model alone.
+    #
+    query
+
+    # Cache
+    #
+    # Comment out to disable caching reads and writes.
+    #
+    cache
+
+    # Dirty
+    #
+    # Uncomment this line to include and enable globally:
+    dirty
+    #
+    # Or uncomment this line to include but disable by default, and only enable
+    # per model by passing +dirty: true+ to +translates+.
+    # dirty false
+
+    # Fallbacks
+    #
+    # Uncomment line below to enable fallbacks, using +I18n.fallbacks+.
+    fallbacks
+    #
+    # Or uncomment this line to enable fallbacks with a global default.
+    # fallbacks { :pt => :en }
+
+    # Presence
+    #
+    # Converts blank strings to nil on reads and writes. Comment out to
+    # disable.
+    #
+    presence
+
+    # Default
+    #
+    # Set a default translation per attributes. When enabled, passing +default:
+    # 'foo'+ sets a default translation string to show in case no translation is
+    # present. Can also be passed a proc.
+    #
+    # default 'foo'
+
+    # Fallthrough Accessors
+    #
+    # Uses method_missing to define locale-specific accessor methods like
+    # +title_en+, +title_en=+, +title_fr+, +title_fr=+ for each translated
+    # attribute. If you know what set of locales you want to support, it's
+    # generally better to use Locale Accessors (or both together) since
+    # +method_missing+ is very slow.  (You can use both fallthrough and locale
+    # accessor plugins together without conflict.)
+    #
+    # fallthrough_accessors
+
+    # Locale Accessors
+    #
+    # Uses +def+ to define accessor methods for a set of locales. By default uses
+    # +I18n.available_locales+, but you can pass the set of locales with
+    # +translates+ and/or set a global default here.
+    #
+    locale_accessors
+    #
+    # Or define specific defaults by uncommenting line below
+    # locale_accessors [:en, :ja]
+
+    # Attribute Methods
+    #
+    # Adds translated attributes to +attributes+ hash, and defines methods
+    # +translated_attributes+ and +untranslated_attributes+ which return hashes
+    # with translated and untranslated attributes, respectively. Be aware that
+    # this plugin can create conflicts with other gems.
+    #
+    # attribute_methods
+  end
+end

data/config/initializers/spree_ransack.rb

@@ -0,0 +1,31 @@
+# Core Spree is biased towards SpreeGlobalize, so we need to prepend our versions
+Rails.application.config.to_prepare do
+  module SpreeMobilityAdminTaxonSearch
+    private
+    def load_taxonomy
+      @taxonomy = scope.includes(:translations, taxons: [:translations]).find(params[:taxonomy_id])
+    end
+  end
+  Spree::Admin::TaxonsController.prepend SpreeMobilityAdminTaxonSearch
+
+  module SpreeMobilityAdminStoreSearch
+    def load_stores_by_query
+      @stores = 
+        stores_scope.joins(:translations).where("LOWER(#{Spree::Store::Translation.table_name}.name) LIKE LOWER(:query)",
+                                                query: "%#{params[:q]}%")
+    end
+  end
+  Spree::Admin::StoresController.prepend SpreeMobilityAdminStoreSearch
+
+  module SpreeMobilityApiV1TaxonSearch
+    private
+    def taxonomy
+      if params[:taxonomy_id].present?
+        @taxonomy ||=
+            Spree::Taxonomy.includes(:translations, taxons: [:translations]).
+            accessible_by(current_ability, :show).find(params[:taxonomy_id])
+      end
+    end
+  end
+  Spree::Api::V1::TaxonsController.prepend SpreeMobilityApiV1TaxonSearch
+end

data/config/locales/be.yml

@@ -0,0 +1,6 @@
+be:
+  spree:
+    mobility:
+      locales_displayed_on_translation_forms: "Мовы, адлюстрованныя на формах для перакладу"
+      no_translations_for_criteria: Пераклады па зададзенаму крытэру адсутнічаюць
+      store_translations: Пераклады

data/config/locales/bg.yml

@@ -0,0 +1,6 @@
+bg:
+  spree:
+    mobility:
+      locales_displayed_on_translation_forms: Преводи показани на формите
+      no_translations_for_criteria: Няма преводи по зададената критерия
+      store_translations: Преводи

data/config/locales/en.yml

@@ -0,0 +1,6 @@
+en:
+  spree:
+    mobility:
+      locales_displayed_on_translation_forms: Locales displayed on translation forms
+      no_translations_for_criteria: There are no translations for given criteria
+      store_translations: Store Translations

data/config/locales/it.yml

@@ -0,0 +1,6 @@
+it:
+  spree:
+    mobility:
+      locales_displayed_on_translation_forms: Lingue visualizzate nei form di traduzione
+      no_translations_for_criteria: Non ci sono traduzioni per i criteri selezionati
+      store_translations: Traduzioni per lo Store

data/config/locales/ru.yml

@@ -0,0 +1,6 @@
+ru:
+  spree:
+    mobility:
+      locales_displayed_on_translation_forms: "Языки, отображаемые на формах для перевода"
+      no_translations_for_criteria: Переводы по заданному критерию отсутствуют
+      store_translations: Переводы

data/config/locales/zh-TW.yml

@@ -0,0 +1,6 @@
+zh-TW:
+  spree:
+    mobility:
+      locales_displayed_on_translation_forms: 顯示於編輯翻譯欄位的文字
+      no_translations_for_criteria: 目前沒有適合的翻譯
+      store_translations: 商店翻譯

data/config/routes.rb

@@ -0,0 +1,7 @@
+Spree::Core::Engine.add_routes do
+  namespace :admin, path: Spree.admin_path do
+    get '/:resource/:resource_id/translations' => 'translations#index', as: :translations
+    patch '/option_values/:id' => 'option_values#update', as: :option_type_option_value
+    patch 'product/:id/product_properties/:id' => "product_properties#translate", as: :translate_product_property
+  end
+end

data/db/migrate/20220411041407_add_translations_to_main_models.rb

@@ -0,0 +1,83 @@
+class AddTranslationsToMainModels < SpreeExtension::Migration[4.2]
+  def up
+    unless table_exists?(:spree_product_translations)
+      create_helper :spree_product, { name: :string, description: :text, meta_title: :string,
+                 meta_description: :string, meta_keywords: :string, slug: :string, deleted_at: :datetime }
+      add_index :spree_product_translations, :deleted_at
+    end
+
+    unless table_exists?(:spree_promotion_translations)
+      create_helper :spree_promotion, { name: :string, description: :string }
+    end
+
+    unless table_exists?(:spree_option_type_translations)
+      create_helper :spree_option_type, { name: :string, presentation: :string }
+    end
+
+    unless table_exists?(:spree_property_translations)
+      create_helper :spree_property, { name: :string, presentation: :string }
+    end
+
+    unless table_exists?(:spree_taxonomy_translations)
+      create_helper :spree_taxonomy, { name: :string }
+    end
+
+    unless table_exists?(:spree_taxon_translations)
+      create_helper :spree_taxon, { name: :string, description: :text, meta_title: :string,
+                 meta_description: :string, meta_keywords: :string,
+                 permalink: :string }
+    end
+    
+    unless table_exists?(:spree_option_value_translations)
+      create_helper :spree_option_value, { name: :string, presentation: :string }
+    end
+    
+    unless table_exists?(:spree_product_property_translations)
+      create_helper :spree_product_property, { value: :string }
+    end
+    
+    unless table_exists?(:spree_store_translations)
+      create_helper :spree_store, { name: :string, meta_description: :text, meta_keywords: :text, seo_title: :string }
+    end
+    
+    unless table_exists?(:spree_shipping_method_translations)
+      create_helper :spree_shipping_method, { name: :string }
+    end
+    
+    add_column :friendly_id_slugs, :locale, :string
+  end
+
+  def down
+    drop_table :spree_product_translations
+    drop_table :spree_promotion_translations
+    drop_table :spree_option_type_translations
+    drop_table :spree_property_translations
+    drop_table :spree_taxonomy_translations
+    drop_table :spree_taxon_translations
+    drop_table :spree_option_value_translations
+    drop_table :spree_product_property_translations
+    drop_table :spree_store_translations
+    drop_table :spree_shipping_method_translations
+    remove_column :friendly_id_slugs, :locale
+  end
+  
+  protected
+  
+  def create_helper(table, params)
+    create_table :"#{table}_translations" do |t|
+
+      # Translated attribute(s)
+      params.each_pair do |attr, attr_type|
+        t.send attr_type, attr
+      end
+
+      t.string  :locale, null: false
+      t.references table, null: false, foreign_key: true, index: false
+
+      t.timestamps null: false
+    end
+
+    add_index :"#{table}_translations", :locale, name: :"index_#{table}_translations_on_locale"
+    add_index :"#{table}_translations", [:"#{table}_id", :locale], name: :"index_#{table}_translations_on_id_and_locale", unique: true
+  end
+end

data/db/migrate/20220412095648_remove_null_constraints_from_spree_tables.rb

@@ -0,0 +1,13 @@
+class RemoveNullConstraintsFromSpreeTables < SpreeExtension::Migration[4.2]
+  def up
+    change_column :spree_properties, :presentation, :string, null: true
+    change_column :spree_taxonomies, :name,         :string, null: true
+    change_column :spree_taxons,     :name,         :string, null: true
+  end
+
+  def down
+    change_column :spree_properties, :presentation, :string, null: false
+    change_column :spree_taxonomies, :name,         :string, null: false
+    change_column :spree_taxons,     :name,         :string, null: false
+  end
+end

data/db/migrate/20220413095648_migrate_translation_data.rb

@@ -0,0 +1,53 @@
+class MigrateTranslationData < SpreeExtension::Migration[4.2]
+  def up
+    create_helper Spree::Product, { name: :string, description: :text, meta_title: :string,
+               meta_description: :string, meta_keywords: :string, slug: :string, deleted_at: :datetime }
+
+    create_helper Spree::Promotion, { name: :string, description: :string }
+
+    create_helper Spree::OptionType, { name: :string, presentation: :string }
+
+    create_helper Spree::Property, { name: :string, presentation: :string }
+
+    create_helper Spree::Taxonomy, { name: :string }
+
+    create_helper Spree::Taxon, { name: :string, description: :text, meta_title: :string,
+               meta_description: :string, meta_keywords: :string,
+               permalink: :string }
+  
+    create_helper Spree::OptionValue, { name: :string, presentation: :string }
+  
+    create_helper Spree::ProductProperty, { value: :string }
+  
+    create_helper Spree::Store, { name: :string, meta_description: :text, meta_keywords: :text, seo_title: :string }
+  
+    create_helper Spree::ShippingMethod, { name: :string }
+  end
+
+  def down
+    # no need to do anything
+  end
+  
+  protected
+  
+  def create_helper(model_klass, fields)
+    store = Spree::Store.unscoped.first
+    return unless store
+    default_locale = store.default_locale.to_s
+    field_names = fields.keys + ['created_at', 'updated_at']
+    translation_table = "#{model_klass.table_name.singularize}_translations"
+    foreign_key = "#{model_klass.table_name.singularize}_id"
+    
+    # In case we are migrating from Globalize with existing translations, skip this step
+    return if ActiveRecord::Base.connection.execute("SELECT id FROM #{translation_table}").any?
+    ActiveRecord::Base.connection.execute("SELECT id, #{field_names.join(',')} FROM #{model_klass.table_name}").each do |r|
+      field_values =
+        field_names.each_with_object([]) do |field_name, a|
+          a << r[field_name.to_s]
+        end
+      
+      ActiveRecord::Base.connection.execute(ActiveRecord::Base.sanitize_sql_array(["INSERT INTO #{translation_table} (locale, #{foreign_key}, #{field_names.join(',')}) VALUES (?,?,#{(['?'] * field_names.size).join(',')})",
+        default_locale.to_s, r['id'], *field_values]))
+    end
+  end
+end

data/gemfiles/spree_4_2.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem 'spree', '~> 4.2.0.rc4'
+gem 'spree_i18n', github: 'spree-contrib/spree_i18n', branch: 'main'
+
+gemspec path: '../'

data/gemfiles/spree_master.gemfile

@@ -0,0 +1,10 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem 'spree', github: 'spree/spree', branch: 'main'
+gem 'spree_frontend', github: 'spree/spree', branch: 'main'
+gem 'spree_backend', github: 'spree/spree', branch: 'main'
+gem 'spree_i18n', github: 'spree-contrib/spree_i18n', branch: 'main'
+
+gemspec path: '../'

data/lib/generators/spree_mobility/install/install_generator.rb

@@ -0,0 +1,23 @@
+module SpreeMobility
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+      class_option :migrate, type: :boolean, default: true
+
+      def add_javascripts
+        append_file "vendor/assets/javascripts/spree/backend/all.js", "//= require spree/backend/spree_mobility\n"
+			end
+
+      def add_migrations
+        run 'bundle exec rake spree_mobility:install:migrations'
+      end
+
+      def run_migrations
+        if options[:migrate]
+          run 'bundle exec rake db:migrate'
+        else
+          puts "Skiping rake db:migrate, don't forget to run it!"
+        end
+      end
+    end
+  end
+end

data/lib/spree_mobility/configuration.rb

@@ -0,0 +1,12 @@
+module SpreeMobility
+  class Configuration < Spree::Preferences::Configuration
+    # These configs intend to, respectively:
+    #
+    #   Say which mobility inputs are displayed on backend
+    #   Set locales that should be available for end users
+    #
+    # e.g. If available_locales are [:en, :es] admin can translate model records
+    # to spanish as well. Once it's done :es can be added to supported_locales
+    preference :supported_locales, :array, default: [:en]
+  end
+end

data/lib/spree_mobility/core_ext/mobility/backends/active_record/table/mobility_acts_as_paranoid_decorator.rb

@@ -0,0 +1,31 @@
+module SpreeMobility::CoreExt::Mobility::Backends::ActiveRecord::Table
+  module MobilityActsAsParanoidDecorator
+    private
+    def using_acts_as_paranoid?
+      translation_class = model_class.const_get(subclass_name)
+      return false unless translation_class
+      translation_class.column_names.include?('deleted_at')
+    end
+
+    # If joining a table with a deleted_at column (acts_as_paranoid), then
+    # filter the join with deleted_at IS NULL
+    def join_translations(relation, locale, join_type)
+      if using_acts_as_paranoid?
+        # check if joins changed
+        old_joins = relation.joins_values
+        relation = super
+        new_joins = relation.joins_values - old_joins
+        return relation if new_joins.empty?
+        fail "join_translations new_joins.size > 1" unless new_joins.size == 1
+        # append deleted_at IS NULL to joins ON clause using Arel
+        new_join = new_joins.first
+        joined_table = model_class.const_get(subclass_name).arel_table.alias(table_alias(locale))
+        relation.joins_values -= [new_join]
+        new_join.right = new_join.right.and(joined_table[:deleted_at].eq(nil))
+        relation.joins(new_join)
+      else
+        super
+      end
+    end
+  end
+end

data/lib/spree_mobility/core_ext/spree/option_type_decorator.rb

@@ -0,0 +1,11 @@
+module SpreeMobility::CoreExt::Spree::OptionTypeDecorator
+  def self.prepended(base)
+    base.include SpreeMobility::Translatable
+    SpreeMobility.translates_for base, :name, :presentation
+    
+    base.translation_class.class_eval do
+      validates :name, presence: true, uniqueness: { scope: :locale, case_sensitive: false, allow_blank: true }
+      validates :presentation, presence: true
+    end
+  end
+end

data/lib/spree_mobility/core_ext/spree/option_value_decorator.rb

@@ -0,0 +1,30 @@
+module SpreeMobility::CoreExt::Spree::OptionValueDecorator
+  module TranslationMethods
+    def name_uniqueness_validation
+      return unless name.present?
+      return unless translated_model
+      check_scope =
+        ::Spree::OptionValue.
+        where.not(id: translated_model.id).
+        where(option_type_id: translated_model.option_type_id).
+        joins(:translations).
+        where(spree_option_value_translations: { locale: locale }).
+        where('LOWER(spree_option_value_translations.name) = LOWER(?)', name)
+      if check_scope.exists?
+        errors.add(:name, :taken, value: name)
+      end
+    end
+  end
+  
+  def self.prepended(base)
+    base.include SpreeMobility::Translatable
+    SpreeMobility.translates_for base, :name, :presentation
+    
+    base.translation_class.class_eval do
+      include TranslationMethods
+      validate :name_uniqueness_validation
+      validates :name, presence: true
+      validates :presentation, presence: true
+    end
+  end
+end

data/lib/spree_mobility/core_ext/spree/product_decorator.rb

@@ -0,0 +1,87 @@
+module SpreeMobility::CoreExt::Spree
+  module ProductDecorator
+
+
+    module ClassMethods
+      def search_by_name(query)
+        helper = SpreeMobility::TranslationQuery.new(all.model.mobility_backend_class(:name))
+
+        helper.add_joins(self.all).
+        where("LOWER(#{helper.col_name(:name)}) LIKE LOWER(:query)", query: "%#{query}%").distinct
+      end
+    
+      def search_by_name_or_sku(query)
+        helper = SpreeMobility::TranslationQuery.new(all.model.mobility_backend_class(:name))
+
+        helper.add_joins(self.all).
+        joins(:variants_including_master).
+        where("(LOWER(#{helper.col_name(:name)}) LIKE LOWER(:query)) OR (LOWER(#{::Spree::Variant.table_name}.sku) LIKE LOWER(:query))", query: "%#{query}%").distinct
+      end
+
+      def like_any(fields, values)
+        mobility_fields = fields.select { |field| mobility_attributes.include?(field.to_s) }
+        other_fields = fields - mobility_fields
+
+        helper = SpreeMobility::TranslationQuery.new(all.model.mobility_backend_class(:name))
+        conditions = mobility_fields.product(values).map do |(field, value)|
+          sanitize_sql_array(["LOWER(#{helper.col_name(field)}) LIKE LOWER(?)", "%#{value}%"])
+        end
+
+        scope = other_fields.empty? ? self.all : super(other_fields, values)
+
+        helper.add_joins(scope).where(conditions.join(' OR '))
+      end
+    end
+
+    def self.prepended(base)
+      base.include SpreeMobility::Translatable
+      SpreeMobility.translates_for base, :name, :description, :meta_title, :meta_description, :meta_keywords, :slug
+      base.friendly_id :slug_candidates, use: [:history, :mobility]
+      base.whitelisted_ransackable_scopes << 'search_by_name_or_sku'
+  
+      base.translation_class.class_eval do
+        acts_as_paranoid
+        after_destroy :punch_slug
+        default_scopes = []
+        validates :slug, presence: true, uniqueness: { scope: :locale, case_sensitive: false }
+
+        with_options length: { maximum: 255 }, allow_blank: true do
+          validates :meta_keywords
+          validates :meta_title
+        end
+        with_options presence: true do
+          validates :name
+        end
+      end
+
+      if RUBY_VERSION.to_f >= 2.5
+        base.translation_class.define_method(:punch_slug) { update(slug: "#{Time.current.to_i}_#{slug}"[0..254]) }
+      else
+        base.translation_class.send(:define_method, :punch_slug) { update(slug: "#{Time.current.to_i}_#{slug}"[0..254]) }
+      end
+    end
+
+    # Don't punch slug on original product as it prevents bulk deletion.
+    # Also we don't need it, as it is translated.
+    def punch_slug; end
+
+    def duplicate_extra(old_product)
+      duplicate_translations(old_product)
+    end
+
+    def property(property_name)
+      product_properties.joins(:property).find_by(spree_properties: { id: ::Spree::Property.find_by(name: property_name) }).try(:value)
+    end
+
+    private
+
+    def duplicate_translations(old_product)
+      self.translations.clear
+      old_product.translations.each do |translation|
+        translation.slug = nil # slug must be regenerated
+        translation.name = "COPY OF #{translation.name}" unless translation.name&.start_with?('COPY OF ')
+        self.translations << translation.dup
+      end
+    end
+  end
+end

data/lib/spree_mobility/core_ext/spree/product_property_decorator.rb

@@ -0,0 +1,6 @@
+module SpreeMobility::CoreExt::Spree::ProductPropertyDecorator
+  def self.prepended(base)
+    base.include SpreeMobility::Translatable
+    SpreeMobility.translates_for base, :value
+  end
+end

data/lib/spree_mobility/core_ext/spree/product_scopes_with_mobility_decorator.rb

@@ -0,0 +1,16 @@
+module SpreeMobility::CoreExt::Spree::ProductScopesWithMobilityDecorator
+  def ascend_by_taxons_min_position(taxon_ids)
+    # order() must not refer to select(), because select could be removed
+    # in Spree::Products::Find
+    joins(:classifications).
+      where(::Spree::Classification.table_name => { taxon_id: taxon_ids }).
+      select(
+        [
+          "#{::Spree::Product.table_name}.*",
+          "MIN(#{::Spree::Classification.table_name}.position) AS min_position"
+        ].join(', ')
+      ).
+      group(:id).
+      order(Arel.sql("MIN(#{::Spree::Classification.table_name}.position) ASC"))
+  end
+end

data/lib/spree_mobility/core_ext/spree/products/find_with_mobility_decorator.rb

@@ -0,0 +1,12 @@
+module SpreeMobility::CoreExt::Spree::Products::FindWithMobilityDecorator
+  # The issue here is that ordering by translated attr (e.g. name) will add
+  # an ORDER BY translations_table.name, but the query has a SELECT DISTINCT,
+  # which would require the translations_table.name to be added to the SELECT
+  # Instead of using DISTINCT, we select the ID only and use that as a subquery
+  # in a simple SELECT ... FROM products WHERE products.id IN (SUBQUERY).
+  # Performance-wise should be almost the same, or the same.
+  def execute
+    ::Spree::Product.where(id: super.except(:select).select(:id).distinct(false))
+  end
+end
+

data/lib/spree_mobility/core_ext/spree/promotion_decorator.rb

@@ -0,0 +1,11 @@
+module SpreeMobility::CoreExt::Spree::PromotionDecorator
+  def self.prepended(base)
+    base.include SpreeMobility::Translatable
+    SpreeMobility.translates_for base, :name, :description
+    
+    base.translation_class.class_eval do
+      validates :name, presence: true
+      validates :description, length: { maximum: 255 }, allow_blank: true
+    end
+  end
+end

data/lib/spree_mobility/core_ext/spree/property_decorator.rb

@@ -0,0 +1,26 @@
+module SpreeMobility::CoreExt::Spree::PropertyDecorator
+  def self.prepended(base)
+    base.include SpreeMobility::Translatable
+    SpreeMobility.translates_for base, :name, :presentation
+    
+    base.translation_class.class_eval do
+      validates :name, :presentation, presence: true
+    end
+  end
+
+  # Currently, mobilize does not yet support fallbacks in query scopes,
+  # so we update this method to fetch property IDs instead of value and then let
+  # mobilize logic determine the correct translation.
+  # The performance impact in this case should be minimal.
+  def uniq_values(product_properties_scope: nil)
+    with_uniq_values_cache_key(product_properties_scope) do
+      properties = product_properties
+      properties = properties.where(id: product_properties_scope) if product_properties_scope.present?
+      
+      filter_params_scope = properties.reorder(nil).group(:filter_param).select(::Arel.sql('MAX(id)'))
+      properties.where(id: filter_params_scope).
+      includes(:translations).sort_by(&:value).
+      map { |property| [property.filter_param, property.value] }
+    end
+  end
+end

data/lib/spree_mobility/core_ext/spree/shipping_method_decorator.rb

@@ -0,0 +1,10 @@
+module SpreeMobility::CoreExt::Spree::ShippingMethodDecorator
+  def self.prepended(base)
+    base.include SpreeMobility::Translatable
+    SpreeMobility.translates_for base, :name
+    
+    base.translation_class.class_eval do
+      validates :name, presence: true
+    end
+  end
+end