alchemy-solidus 2.5.1 → 3.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 902f1162c27c48166435b147674f8b9da9fd5c0fa9d3d95b60acb4cc62cf900c
-  data.tar.gz: acd5cabe30ca1080ef6790ad7f3e47b553b02b7eb3ff8fe68c82ca7bb6c8c178
+  metadata.gz: 0a6e12d7505ec9bc89770302454888523f6c5fe4853bc46f49596b2f2c0380aa
+  data.tar.gz: 6a57eba2ac492aae30aa9df6ae32a9a48c3237210f0bcf6028b2e270ac25bebe
 SHA512:
-  metadata.gz: a8b2ce4bcf0f1ef9a8e1e1461c6843f404c7ff7397a7cef4f71dc8d3efbe8cd5a31979b83d29cb2a42279afd7f0f10261064c52e9ee31ef04e8826f4c60718f6
-  data.tar.gz: 2b479960fe66bbe12d2168ca95eedf62c798a00e790647d80fa6dc1ac7c6aadc453a6d6b2e7ec28405aec2e534b1ad5a675884df4481f3bc0adea83b9e8faf08
+  metadata.gz: 2f25bad30b4d80f28b066d1631ea7cbd0dd750490c8e3bdf8b731c83bcbf87080a273e6067b499295c5017cdf87d317190aed4fa207477cd166f45a6169aa018
+  data.tar.gz: 63b56091e7f023645b5de9c7212587852741fe2e6dbae826e12f1dc79e8c268f1b4248e5498c97d2aebb4bc9a7c5d782dfe61d5d6300f66c57b8b0ec9316de1a

data/README.md

@@ -41,6 +41,8 @@ $ bundle install
 
 ## Automated setup
 
+**Recommended**
+
 We ship a Rails generator that helps you to install this gem into your existing application.
 
 ```bash
@@ -61,9 +63,11 @@ To upgrade update the Gemfile and run the install generator again
 $ bin/rails g alchemy:solidus:install
 ```
 
-## Manual setup
+**NOTE** Please make sure to remove the `Alchemy::Modules.register_module` part from your `config/initializer/alchemy.rb` file if upgrading from 2.5.
+
+## Manual setup (for advanced users)
 
-If you want to have full control over the integration you can also set this up manually.
+For regular setups we recommend the [automated installer](#automated-setup) mentioned above. But if you know what you are doing and want to have full control over the integration you can also set this up manually.
 
 ### Authentication system installation
 
@@ -178,26 +182,30 @@ and follow the on screen instructions.
 
 ### Render Alchemy Content in Solidus Layout
 
-If you plan to render the Alchemy site in the Solidus layout add the following
-to your initializer:
+~~If you plan to render the Alchemy site in the Solidus layout add the following
+to your initializer:~~
 
 ```ruby
 # config/initializers/alchemy.rb
 require 'alchemy/solidus/use_solidus_layout'
 ```
 
+**NOTE:** Since v2.5.2 this is done automatically for you. If you upgraded from an older version you can safely remove this from your initializers.
+
 ### Render Alchemy Content in Solidus views
 
-If you plan to render Alchemy content in your Solidus views (ie. a global header
+~~If you plan to render Alchemy content in your Solidus views (ie. a global header
 or footer section), you need to include the Alchemy view helpers and language
 store in your Solidus controllers with the following addition to your
-initializer:
+initializer:~~
 
 ```ruby
 # config/initializers/alchemy.rb
 require 'alchemy/solidus/alchemy_in_solidus'
 ```
 
+**NOTE:** Since v2.5.2 this is done automatically for you. If you upgraded from an older version you can safely remove this from your initializers.
+
 ### Routing
 
 For routing you have a few options.

data/Rakefile

@@ -20,7 +20,7 @@ task :test_setup do
     system <<-SETUP.strip_heredoc
       export RAILS_ENV=test && \
       bin/rake db:environment:set db:drop && \
-      bin/rails g spree:install --force --quiet --auto-accept --no-seed --no-sample && \
+      bin/rails g spree:install --force --auto-accept --no-seed --no-sample && \
       bin/rails g alchemy:solidus:install --auto-accept --force
     SETUP
     exit($?.exitstatus) unless $?.success?

data/app/views/alchemy/essences/_essence_spree_product_editor.html.erb

@@ -1,3 +1,5 @@
+<% content ||= local_assigns[:content] || local_assigns[:essence_spree_product_editor] %>
+
 <div class="content_editor essence_spree_product" id="<%= content.dom_id %>" data-content-id="<%= content.id %>">
   <%= content_label(content) %>
   <%= text_field_tag(

data/app/views/alchemy/essences/_essence_spree_taxon_editor.html.erb

@@ -1,3 +1,5 @@
+<% content ||= local_assigns[:content] || local_assigns[:essence_spree_taxon_editor] %>
+
 <div class="content_editor essence_spree_taxon" id="<%= content.dom_id %>" data-content-id="<%= content.id %>">
   <%= content_label(content) %>
   <%= text_field_tag(

data/app/views/alchemy/essences/_essence_spree_variant_editor.html.erb

@@ -1,3 +1,5 @@
+<% content ||= local_assigns[:content] || local_assigns[:essence_spree_variant_editor] %>
+
 <div class="content_editor essence_spree_variant" id="<%= content.dom_id %>" data-content-id="<%= content.id %>">
   <%= content_label(content) %>
   <%= text_field_tag(

data/config/initializers/alchemy.rb

@@ -42,6 +42,14 @@ if defined?(Spree::Auth::Engine)
       name: 'Users'
     }
   )
+  Alchemy.user_class_name = 'Spree::User'
+  Alchemy.current_user_method = :spree_current_user
+
+  if Alchemy.respond_to?(:logout_method)
+    Rails.application.config.after_initialize do
+      Alchemy.logout_method = Devise.sign_out_via
+    end
+  end
 end
 
 Alchemy::Modules.register_module(alchemy_module)

data/lib/alchemy/solidus/alchemy_in_solidus.rb

@@ -1,16 +1,15 @@
-# Make sure we have everything loaded before patching classes
-Rails.application.config.to_prepare do
-  # Allows to render Alchemy content within Solidus' controller views
-  Spree::BaseController.include Alchemy::ControllerActions
-  Spree::BaseController.include Alchemy::ConfigurationMethods
+# Allows to render Alchemy content within Solidus' controller views
+Spree::StoreController.include(
+  Alchemy::ControllerActions,
+  Alchemy::ConfigurationMethods
+)
 
-  # Hook into SolidusAuthDevise controllers if present
-  if defined? Spree::Auth::Engine
-    Spree::UserPasswordsController.include Alchemy::ControllerActions
-    Spree::UserConfirmationsController.include Alchemy::ControllerActions
-    Spree::UserRegistrationsController.include Alchemy::ControllerActions
-    Spree::UserSessionsController.include Alchemy::ControllerActions
-  end
+# Hook into SolidusAuthDevise controllers if present
+if defined? Spree::Auth::Engine
+  Spree::UserPasswordsController.include Alchemy::ControllerActions
+  Spree::UserConfirmationsController.include Alchemy::ControllerActions
+  Spree::UserRegistrationsController.include Alchemy::ControllerActions
+  Spree::UserSessionsController.include Alchemy::ControllerActions
 end
 
 # Do not prefix element view partials with `spree` namespace.

data/lib/alchemy/solidus/engine.rb

@@ -12,12 +12,12 @@ module Alchemy
         Alchemy.register_ability ::Spree::Ability
         ::Spree::Ability.register_ability ::Alchemy::Permissions
 
-        if Alchemy.user_class_name == 'Spree::User'
+        if Alchemy.user_class.name == 'Spree::User'
           require 'alchemy/solidus/spree_user_extension'
           Spree::User.include Alchemy::Solidus::SpreeUserExtension
         end
 
-        if Alchemy.user_class_name == 'Alchemy::User'
+        if Alchemy.user_class.name == 'Alchemy::User'
           require 'alchemy/solidus/alchemy_user_extension'
           Alchemy::User.include Alchemy::Solidus::AlchemyUserExtension
           require 'alchemy/solidus/spree_admin_unauthorized_redirect'
@@ -27,18 +27,26 @@ module Alchemy
           require 'alchemy/solidus/spree_custom_user_generator_fix'
           require 'alchemy/solidus/spree_install_generator_fix'
         end
+
+        if SolidusSupport.frontend_available?
+          # Allows to render Alchemy content within Solidus' controller views
+          require_dependency 'alchemy/solidus/alchemy_in_solidus'
+        end
+
+        # Allows to use Solidus helpers within Alchemys controller views
+        require_dependency 'alchemy/solidus/use_solidus_layout'
       end
 
       # Fix for +belongs_to :bill_address+ in {Spree::UserAddressBook}
       # Solidus has this set to +false+ in {Spree::Base}, but {Alchemy::User} does not inherit from it.
       initializer 'alchemy_solidus.belongs_bill_address_fix' do
-        if Alchemy.user_class_name == 'Alchemy::User'
+        if Alchemy.user_class.name == 'Alchemy::User'
           ActiveSupport.on_load(:active_record) do
             Alchemy::User.belongs_to_required_by_default = false
           end
         end
       end
-      
+
       # In versions of Solidus prior to 2.8, we override the tabs partial
       # to pass a match_path value to each tab. (Version 2.8 is already
       # passing this option.) This option is used to configure the paths

data/lib/alchemy/solidus/spree_admin_unauthorized_redirect.rb

@@ -4,6 +4,6 @@ Spree::Admin::BaseController.unauthorized_redirect = -> do
     redirect_to spree.root_path
   else
     store_location
-    redirect_to alchemy.login_path
+    redirect_to Alchemy.login_path
   end
 end

data/lib/alchemy/solidus/use_solidus_layout.rb

@@ -1,18 +1,19 @@
-# Make sure we have everything loaded before patching classes
-Rails.application.config.to_prepare do
-  # Allows to use Solidus helpers within Alchemys controller views
-  Alchemy::BaseHelper.include Spree::BaseHelper
-  Alchemy::BaseHelper.include Spree::CheckoutHelper
-  Alchemy::BaseHelper.include Spree::ProductsHelper
-  Alchemy::BaseHelper.include Spree::StoreHelper
-  Alchemy::BaseHelper.include Spree::TaxonsHelper
+# Allows to use Solidus helpers within Alchemys controller views
+Alchemy::BaseHelper.include(
+  Spree::BaseHelper,
+  Spree::CheckoutHelper,
+  Spree::ProductsHelper,
+  Spree::StoreHelper,
+  Spree::TaxonsHelper
+)
 
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::Auth
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::Common
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::Order
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::PaymentParameters
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::Pricing
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::Search
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::Store
-  Alchemy::BaseController.include Spree::Core::ControllerHelpers::StrongParameters
-end
+Alchemy::BaseController.include(
+  Spree::Core::ControllerHelpers::Auth,
+  Spree::Core::ControllerHelpers::Common,
+  Spree::Core::ControllerHelpers::Order,
+  Spree::Core::ControllerHelpers::PaymentParameters,
+  Spree::Core::ControllerHelpers::Pricing,
+  Spree::Core::ControllerHelpers::Search,
+  Spree::Core::ControllerHelpers::Store,
+  Spree::Core::ControllerHelpers::StrongParameters
+)

data/lib/alchemy/solidus/version.rb

@@ -1,5 +1,5 @@
 module Alchemy
   module Solidus
-    VERSION = "2.5.1"
+    VERSION = "3.1.1"
   end
 end

data/lib/generators/alchemy/solidus/install/files/db/seeds/alchemy/pages.yml

@@ -0,0 +1,3 @@
+- name: Home
+  page_layout: index
+  public_on: 1970-01-01

data/lib/generators/alchemy/solidus/install/install_generator.rb

@@ -1,5 +1,12 @@
 require 'rails/generators'
-require 'rails/generators/alchemy/install/install_generator'
+require 'alchemy/version'
+
+if Alchemy.gem_version >= Gem::Version.new("5.0.0.a")
+  require 'generators/alchemy/install/install_generator'
+else
+  require 'rails/generators/alchemy/install/install_generator'
+end
+
 require 'generators/spree/custom_user/custom_user_generator'
 require 'solidus_support'
 
@@ -25,38 +32,36 @@ module Alchemy
       class_option :auto_accept, default: false, type: :boolean,
         desc: 'Set true if run from a automated script (ie. on a CI)'
 
-      source_root File.expand_path('templates', __dir__)
+      source_root File.expand_path('files', __dir__)
 
       def run_alchemy_installer
         unless options[:skip_alchemy_installer]
           arguments = options[:auto_accept] ? ['--skip-demo-files', '--force'] : []
           Alchemy::Generators::InstallGenerator.start(arguments)
+          rake('railties:install:migrations', abort_on_failure: true)
+          rake('db:migrate', abort_on_failure: true)
         end
       end
 
       def run_alchemy_devise_installer
-        if Kernel.const_defined?('Alchemy::Devise') && !options[:skip_alchemy_devise_installer]
+        if alchemy_devise_present? && !options[:skip_alchemy_devise_installer]
           arguments = options[:auto_accept] ? ['--force'] : []
           Alchemy::Devise::Generators::InstallGenerator.start(arguments)
         end
       end
 
       def run_spree_custom_user_generator
-        if Kernel.const_defined?('Alchemy::Devise') && !options[:skip_spree_custom_user_generator]
+        if alchemy_devise_present? && !options[:skip_spree_custom_user_generator]
           arguments = options[:auto_accept] ? ['Alchemy::User', '--force'] : ['Alchemy::User']
           Spree::CustomUserGenerator.start(arguments)
           gsub_file 'lib/spree/authentication_helpers.rb', /main_app\./, 'Alchemy.'
           if SolidusSupport.solidus_gem_version < Gem::Version.new('2.5.0')
             gsub_file 'config/initializers/spree.rb', /Spree\.user_class.?=.?.+$/, 'Spree.user_class = "Alchemy::User"'
           end
-          rake 'db:migrate'
+          rake('db:migrate', abort_on_failure: true)
         end
       end
 
-      def copy_alchemy_initializer
-        template "alchemy.rb.tt", "config/initializers/alchemy.rb"
-      end
-
       def inject_admin_tab
         inject_into_file 'config/initializers/spree.rb', {after: "Spree::Backend::Config.configure do |config|\n"} do
           <<~ADMIN_TAB
@@ -75,7 +80,7 @@ module Alchemy
       end
 
       def create_admin_user
-        if Kernel.const_defined?('Alchemy::Devise') && !options[:skip_alchemy_user_generator] && Alchemy::User.count.zero?
+        if alchemy_devise_present? && !options[:skip_alchemy_user_generator] && Alchemy::User.count.zero?
           login = ENV.fetch('ALCHEMY_ADMIN_USER_LOGIN', 'admin')
           email = ENV.fetch('ALCHEMY_ADMIN_USER_EMAIL', 'admin@example.com')
           password = ENV.fetch('ALCHEMY_ADMIN_USER_PASSWORD', 'test1234')
@@ -85,14 +90,17 @@ module Alchemy
             password = ask("Enter the password for the admin user", default: password)
           end
 
+          # This is a bit strange, but without the double save this fails with a failed validation.
           Alchemy::User.create!(
             login: login,
             email: email,
             password: password,
             password_confirmation: password,
-            alchemy_roles: 'admin',
-            spree_roles: [Spree::Role.find_or_create_by!(name: 'admin')]
-          )
+            alchemy_roles: 'admin'
+          ).tap do |user|
+            user.spree_roles = [Spree::Role.find_or_create_by!(name: 'admin')]
+            user.save!
+          end
         end
       end
 
@@ -114,21 +122,17 @@ module Alchemy
 
       def set_root_route
         routes_file_path = Rails.root.join('config', 'routes.rb')
-        if options[:auto_accept] || ask("\nDo you want Alchemy to handle the root route ('/')?", default: true)
-          if File.read(routes_file_path).match SPREE_MOUNT_REGEXP
-            sentinel = SPREE_MOUNT_REGEXP
-          else
-            sentinel = "Rails.application.routes.draw do\n"
-          end
+        if options[:auto_accept] || yes?("\nDo you want Alchemy to handle the root route '/'? (y/n)")
+          sentinel = "Rails.application.routes.draw do\n"
           inject_into_file routes_file_path, {after: sentinel} do
             <<~ROOT_ROUTE
-              \n
               \  # Let AlchemyCMS handle the root route
-              \  Spree::Core::Engine.routes.draw do
-              \    root to: '/alchemy/pages#index'
-              \  end
+              \  root to: 'alchemy/pages#index'
             ROOT_ROUTE
           end
+          copy_file('db/seeds/alchemy/pages.yml')
+          append_file(Rails.root.join('db', 'seeds.rb'), "Alchemy::Seeder.seed!\n")
+          rake('alchemy:db:seed', abort_on_failure: true)
         end
       end
 
@@ -136,6 +140,12 @@ module Alchemy
         append_file "vendor/assets/javascripts/alchemy/admin/all.js",
           "//= require alchemy/solidus/admin.js"
       end
+
+      private
+
+      def alchemy_devise_present?
+        defined?(Alchemy::Devise::Engine)
+      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: alchemy-solidus
 version: !ruby/object:Gem::Version
-  version: 2.5.1
+  version: 3.1.1
 platform: ruby
 authors:
 - Thomas von Deyen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-12-13 00:00:00.000000000 Z
+date: 2020-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: alchemy_cms
@@ -239,8 +239,8 @@ files:
 - lib/alchemy/solidus/spree_user_extension.rb
 - lib/alchemy/solidus/use_solidus_layout.rb
 - lib/alchemy/solidus/version.rb
+- lib/generators/alchemy/solidus/install/files/db/seeds/alchemy/pages.yml
 - lib/generators/alchemy/solidus/install/install_generator.rb
-- lib/generators/alchemy/solidus/install/templates/alchemy.rb.tt
 - lib/views/spree/admin/shared/_tabs.html.erb
 homepage: https://github.com/AlchemyCMS/alchemy-solidus
 licenses:

data/lib/generators/alchemy/solidus/install/templates/alchemy.rb.tt

@@ -1,44 +0,0 @@
-Alchemy::Modules.register_module({
-  engine_name: 'spree',
-  name: 'solidus',
-  navigation: {
-    controller: 'spree/admin/orders',
-    action: 'index',
-    name: 'Store',
-    inline_image: '<svg version="1.1" width="24px" height="24px" viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg" style="vertical-align: middle">
-      <path d="M 6.687 21.839 L 17.314 21.839 L 17.314 2.162 L 6.687 2.162 Z M 19.476 24 L 4.525 24 L 4.525 0 L 19.476 0 Z M 13.89 17 C 13.307 16.662 13.078 16.287 13.05 15.313 L 13.05 8.574 C 13.05 6.554 12.05 5.814 9.911 4.794 L 8.99 6.434 L 10.111 7.074 C 10.693 7.412 10.923 7.786 10.951 8.761 L 10.951 15.5 C 10.951 17.52 11.951 18.26 14.09 19.279 L 15.01 17.64 Z" fill="currentColor"/>
-    </svg>',
-    data: { turbolinks: false },
-    sub_navigation: [
-      {
-        controller: 'spree/admin/orders',
-        action: 'index',
-        name: 'Orders'
-      },
-      {
-        controller: 'spree/admin/products',
-        action: 'index',
-        name: 'Products'
-      },
-      {
-        controller: 'spree/admin/promotions',
-        action: 'index',
-        name: 'Promotions'
-      },
-      {
-        controller: 'spree/admin/stock_items',
-        action: 'index',
-        name: 'Stock'
-      <%- if defined?(Spree::Auth::Engine) -%>
-      },
-      {
-        controller: 'spree/admin/users',
-        action: 'index',
-        name: 'Users'
-      }
-      <%- else -%>
-      }
-      <%- end -%>
-    ]
-  }
-})