spree_multi_vendor 2.4.1 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 956455a7ff13df68945b50b576880f345e2dbd4f5c4c32dc99420f0a1c5698e7
-  data.tar.gz: 8daf1f4488fad039a96e4c43a2211521d5b15297c5e628f018368d38fca70121
+  metadata.gz: 739faebda6a3b0fea3e9bb528131c08de2dde2f692dbca6b7346e8eaf0b5d9a1
+  data.tar.gz: 656938bba90616428bd534d7e64d94d2673e99a3ced0a8a2dfc0515d6eb4c0c6
 SHA512:
-  metadata.gz: 6b10eaa1316ee480d4b8b2e456e1648b3da2b01561effa253c65671de1963e3cb7369c1a061a4a298d443f5be4c018a7d9fff45d0a0514a2115f17f400f4fbf6
-  data.tar.gz: ad4aac2625a258d534c9fa553a8261907601443a53859ef55e4e76523c6868c59083dac7904026741dc040b3b80b1da42a6aca9e35060fe82f17579ad111f0af
+  metadata.gz: 639389fb453a10c7fc98328a6bcf1f8b4d834063f1be4983a5177a00e2dafcb447afc6d2f67095dd80ba716214546b645481703f131bffca8abd1214fd007125
+  data.tar.gz: 4d8c28520b08cc36c0fb130e77d566ff4aaeee9a63d971c52b8af6c61443047cddb2f18df2543c3b35b47e22c461f9d65b1d89a82825cf1c7c82ec5b6199aebf

data/app/controllers/spree/admin/vendors_controller.rb

@@ -1,6 +1,7 @@
 module Spree
   module Admin
     class VendorsController < ResourceController
+      include Translatable
 
       def create
         if permitted_resource_params[:image]
@@ -13,7 +14,6 @@ module Spree
         if permitted_resource_params[:image]
           @vendor.create_image(attachment: permitted_resource_params.delete(:image))
         end
-        format_translations if defined? SpreeGlobalize
         super
       end
 
@@ -51,18 +51,6 @@ module Spree
           products: []
         }
       end
-
-      def format_translations
-        return if params[:vendor][:translations_attributes].blank?
-        params[:vendor][:translations_attributes].each do |_, data|
-          translation = @vendor.translations.find_or_create_by(locale: data[:locale])
-          translation.name = data[:name]
-          translation.about_us = data[:about_us]
-          translation.contact_us = data[:contact_us]
-          translation.slug = data[:slug]
-          translation.save!
-        end
-      end
     end
   end
 end

data/app/models/spree/vendor.rb

@@ -1,6 +1,11 @@
 module Spree
   class Vendor < Spree::Base
     extend FriendlyId
+    include TranslatableResource
+    include TranslatableResourceSlug
+
+    TRANSLATABLE_FIELDS = %i[name about_us contact_us slug].freeze
+    translates(*TRANSLATABLE_FIELDS)
 
     acts_as_paranoid
     acts_as_list column: :priority
@@ -48,16 +53,6 @@ module Spree
       update(notification_email: email)
     end
 
-    # Spree Globalize support
-    # https://github.com/spree-contrib/spree_multi_vendor/issues/104
-    if defined?(SpreeGlobalize)
-      attr_accessor :translations_attributes
-      translates :name,
-                 :about_us,
-                 :contact_us,
-                 :slug, fallbacks_for_empty_translations: true
-    end
-
     private
 
     def create_stock_location

data/app/views/spree/admin/vendors/index.html.erb

@@ -66,9 +66,9 @@
             <!-- Add translation icon support -->
             <%= link_to_with_icon 'translate',
                                   nil,
-                                  spree.admin_translations_path('vendor', vendor.id),
+                                  spree.translations_admin_vendor_path(vendor.id),
                                   title: Spree.t(:'i18n.translations'),
-                                  class: 'btn btn-sm btn-primary' if defined? (SpreeGlobalize)%>
+                                  class: 'btn btn-sm btn-primary' %>
           </td>
         </tr>
       <% end %>

data/app/views/spree/admin/vendors/translations.html.erb

@@ -0,0 +1,3 @@
+<div class="row" style="display: block">
+  <%= render 'spree/admin/translations/form', resource: @object %>
+</div>

data/config/routes.rb

@@ -1,6 +1,11 @@
 Spree::Core::Engine.routes.draw do
   namespace :admin do
     resources :vendors do
+      member do
+        get :translations
+        post :translations, to: 'vendors#edit_translations'
+      end
+
       collection do
         post :update_positions
       end

data/db/migrate/20231114090744_add_mobility_translations_to_vendor.rb

@@ -0,0 +1,25 @@
+class AddMobilityTranslationsToVendor < ActiveRecord::Migration[6.1]
+  def change
+    if ActiveRecord::Base.connection.table_exists?('spree_vendor_translations')
+      if ActiveRecord::Migration.connection.index_exists?(:spree_vendors_translations, :spree_vendor_id)
+        remove_index :spree_vendor_translations, column: :spree_vendor_id, if_exists: true
+      end
+    else
+      create_table :spree_vendor_translations do |t|
+        # Translated attribute(s)
+        t.string :name
+        t.text :about_us
+        t.text :contact_us
+        t.string :slug
+
+        t.string  :locale, null: false
+        t.references :spree_vendor, null: false, foreign_key: true, index: false
+
+        t.timestamps null: false
+      end
+
+      add_index :spree_vendor_translations, :locale, name: :index_spree_vendor_translations_on_locale
+      add_index :spree_vendor_translations, [:locale, :slug], unique: true, name: 'vendor_unique_slug_per_locale'
+    end
+  end
+end

data/db/migrate/20231114093444_transfer_vendor_data_to_translatable_tables.rb

@@ -0,0 +1,11 @@
+class TransferVendorDataToTranslatableTables < ActiveRecord::Migration[6.1]
+  TRANSLATION_MIGRATION = Spree::TranslationMigrations.new(Spree::Vendor, 'en')
+
+  def up
+    TRANSLATION_MIGRATION.transfer_translation_data
+  end
+
+  def down
+    TRANSLATION_MIGRATION.revert_translation_data_transfer
+  end
+end

data/lib/spree_multi_vendor/version.rb

@@ -1,5 +1,5 @@
 module SpreeMultiVendor
-  VERSION = '2.4.1'
+  VERSION = '2.5.0'
 
   module_function
 

data/spree_multi_vendor.gemspec

@@ -29,7 +29,7 @@ Gem::Specification.new do |s|
   s.require_path = 'lib'
   s.requirements << 'none'
 
-  spree_version = '>= 4.3.0.rc1'
+  spree_version = '>= 4.6.0'
   s.add_dependency 'spree', spree_version
   s.add_dependency 'spree_backend', spree_version
   s.add_dependency 'spree_emails', spree_version

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spree_multi_vendor
 version: !ruby/object:Gem::Version
-  version: 2.4.1
+  version: 2.5.0
 platform: ruby
 authors:
 - Spark Solutions
@@ -16,42 +16,42 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.3.0.rc1
+        version: 4.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.3.0.rc1
+        version: 4.6.0
 - !ruby/object:Gem::Dependency
   name: spree_backend
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.3.0.rc1
+        version: 4.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.3.0.rc1
+        version: 4.6.0
 - !ruby/object:Gem::Dependency
   name: spree_emails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.3.0.rc1
+        version: 4.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.3.0.rc1
+        version: 4.6.0
 - !ruby/object:Gem::Dependency
   name: spree_extension
   requirement: !ruby/object:Gem::Requirement
@@ -162,7 +162,6 @@ files:
 - app/overrides/spree/admin/orders/index.rb
 - app/overrides/spree/admin/products/form.rb
 - app/overrides/spree/admin/products/new.rb
-- app/overrides/spree/admin/shared/_main_menu.rb
 - app/overrides/spree/admin/shared/_order_summary.rb
 - app/overrides/spree/admin/users/_form.rb
 - app/serializers/spree/v2/storefront/vendor_image_serializer.rb
@@ -177,6 +176,7 @@ files:
 - app/views/spree/admin/vendors/edit.html.erb
 - app/views/spree/admin/vendors/index.html.erb
 - app/views/spree/admin/vendors/new.html.erb
+- app/views/spree/admin/vendors/translations.html.erb
 - app/views/spree/api/v1/vendors/index.rabl
 - app/views/spree/api/v1/vendors/show.rabl
 - app/views/spree/vendor_mailer/vendor_notification_email.html.erb
@@ -204,6 +204,8 @@ files:
 - db/migrate/20210728120649_change_vendorized_models_vendor_id_type.rb
 - db/migrate/20210728120910_change_vendor_id_and_user_id_type_for_spree_vendor_users.rb
 - db/migrate/20210728121024_change_vendor_id_and_order_id_type_for_spree_order_commissions.rb
+- db/migrate/20231114090744_add_mobility_translations_to_vendor.rb
+- db/migrate/20231114093444_transfer_vendor_data_to_translatable_tables.rb
 - lib/generators/spree_multi_vendor/install/install_generator.rb
 - lib/generators/spree_multi_vendor/mailers_preview/mailers_preview_generator.rb
 - lib/generators/spree_multi_vendor/mailers_preview/templates/mailers/previews/vendor_notification_preview.rb
@@ -250,9 +252,9 @@ licenses:
 - BSD-3-Clause
 metadata:
   bug_tracker_uri: https://github.com/spree-contrib/spree_multi_vendor/issues
-  changelog_uri: https://github.com/spree-contrib/spree_multi_vendor/releases/tag/v2.4.1
+  changelog_uri: https://github.com/spree-contrib/spree_multi_vendor/releases/tag/v2.5.0
   documentation_uri: https://guides.spreecommerce.org/
-  source_code_uri: https://github.com/spree-contrib/spree_multi_vendor/tree/v2.4.1
+  source_code_uri: https://github.com/spree-contrib/spree_multi_vendor/tree/v2.5.0
 post_install_message:
 rdoc_options: []
 require_paths:

data/app/overrides/spree/admin/shared/_main_menu.rb

@@ -1,6 +0,0 @@
-Deface::Override.new(
-  virtual_path: 'spree/admin/shared/_main_menu',
-  name: 'Display configuration tab for vendors',
-  replace: 'erb[silent]:contains("current_store")',
-  text: '<% if can?(:admin, current_store) || current_spree_user&.vendors&.any? %>'
-)