spree_core 3.0.10 → 3.1.0.rc1

data/lib/generators/spree/dummy/templates/rails/test.rb

@@ -27,7 +27,8 @@ Dummy::Application.configure do
   # The :test delivery method accumulates sent emails in the
   # ActionMailer::Base.deliveries array.
   config.action_mailer.delivery_method = :test
-  ActionMailer::Base.default :from => "spree@example.com"
+  ActionMailer::Base.default from: "spree@example.com"
+
 
   # Print deprecation notices to the stderr
   config.active_support.deprecation = :stderr

data/lib/spree/core.rb

@@ -31,6 +31,21 @@ module Spree
     end
   end
 
+  def self.admin_path
+    Spree::Config[:admin_path]
+  end
+
+  # Used to configure admin_path for Spree
+  #
+  # Example:
+  #
+  # write the following line in `config/initializers/spree.rb`
+  #   Spree.admin_path = '/custom-path'
+
+  def self.admin_path=(path)
+    Spree::Config[:admin_path] = path
+  end
+
   # Used to configure Spree.
   #
   # Example:
@@ -59,6 +74,7 @@ require 'spree/core/version'
 require 'spree/core/environment_extension'
 require 'spree/core/environment/calculators'
 require 'spree/core/environment'
+require 'spree/core/number_generator'
 require 'spree/promo/environment'
 require 'spree/migrations'
 require 'spree/core/engine'

data/lib/spree/core/controller_helpers/auth.rb

@@ -67,7 +67,7 @@ module Spree
         def redirect_unauthorized_access
           if try_spree_current_user
             flash[:error] = Spree.t(:authorization_failure)
-            redirect_to '/unauthorized'
+            redirect_to spree.forbidden_path
           else
             store_location
             if respond_to?(:spree_login_path)

data/lib/spree/core/controller_helpers/common.rb

@@ -51,9 +51,9 @@ module Spree
 
           def set_user_language
             locale = session[:locale]
-            locale ||= config_locale if respond_to?(:config_locale, true)
-            locale ||= Rails.application.config.i18n.default_locale
-            locale ||= I18n.default_locale unless I18n.available_locales.map(&:to_s).include?(locale)
+            locale = config_locale if respond_to?(:config_locale, true) && locale.blank?
+            locale = Rails.application.config.i18n.default_locale if locale.blank?
+            locale = I18n.default_locale unless I18n.available_locales.map(&:to_s).include?(locale.to_s)
             I18n.locale = locale
           end
 

data/lib/spree/core/controller_helpers/order.rb

@@ -81,11 +81,12 @@ module Spree
           options[:lock] ||= false
 
           # Find any incomplete orders for the guest_token
-          if with_adjustments
-            order = Spree::Order.incomplete.includes(:adjustments).lock(options[:lock]).find_by(current_order_params)
-          else
-            order = Spree::Order.incomplete.lock(options[:lock]).find_by(current_order_params)
-          end
+          incomplete_orders = Spree::Order.incomplete.includes(line_items: [variant: [:images, :option_values, :product]])
+          order = if with_adjustments
+                    incomplete_orders.includes(:adjustments).lock(options[:lock]).find_by(current_order_params)
+                  else
+                    incomplete_orders.lock(options[:lock]).find_by(current_order_params)
+                  end
 
           # Find any incomplete orders for the current user
           if order.nil? && try_spree_current_user

data/lib/spree/core/controller_helpers/search.rb

@@ -2,7 +2,7 @@ module Spree
   module Core
     module ControllerHelpers
       module Search
-        def build_searcher params
+        def build_searcher(params)
           Spree::Config.searcher_class.new(params).tap do |searcher|
             searcher.current_user = try_spree_current_user
             searcher.current_currency = current_currency

data/lib/spree/core/controller_helpers/store.rb

@@ -7,6 +7,7 @@ module Spree
         included do
           helper_method :current_currency
           helper_method :current_store
+          helper_method :current_price_options
         end
 
         def current_currency
@@ -16,6 +17,34 @@ module Spree
         def current_store
           @current_store ||= Spree::Store.current(request.env['SERVER_NAME'])
         end
+
+        # Return a Hash of things that influence the prices displayed in your shop.
+        #
+        # By default, the only thing that influences prices that is the current order's +tax_zone+
+        # (to facilitate differing prices depending on VAT rate for digital products in Europe, see
+        # https://github.com/spree/spree/pull/6295 and https://github.com/spree/spree/pull/6662).
+        #
+        # If your prices depend on something else, overwrite this method and add
+        # more key/value pairs to the Hash it returns.
+        #
+        # Be careful though to also patch the following parts of Spree accordingly:
+        #
+        # * `Spree::VatPriceCalculation#gross_amount`
+        # * `Spree::LineItem#update_price`
+        # * `Spree::Stock::Estimator#taxation_options_for`
+        # * Subclass the `DefaultTax` calculator
+        #
+        def current_price_options
+          {
+            tax_zone: current_tax_zone
+          }
+        end
+
+        private
+
+        def current_tax_zone
+          current_order.try(:tax_zone) || Spree::Zone.default_tax
+        end
       end
     end
   end

data/lib/spree/core/delegate_belongs_to.rb

@@ -75,7 +75,7 @@ module DelegateBelongsTo
 
   def delegator_for(association, attr, *args)
     return if self.class.column_names.include?(attr.to_s)
-    send("#{association}=", self.class.reflect_on_association(association).klass.new) if send(association).nil?
+    send "build_#{association}" if send(association).nil?
     if args.empty?
       send(association).send(attr)
     else
@@ -85,7 +85,7 @@ module DelegateBelongsTo
 
   def delegator_for_setter(association, attr, val)
     return if self.class.column_names.include?(attr.to_s)
-    send("#{association}=", self.class.reflect_on_association(association).klass.new) if send(association).nil?
+    send "build_#{association}" if send(association).nil?
     send(association).send("#{attr}=", val)
   end
   protected :delegator_for

data/lib/spree/core/engine.rb

@@ -5,38 +5,49 @@ module Spree
       engine_name 'spree'
 
       rake_tasks do
-        load File.join(root, "lib", "tasks", "exchanges.rake")
+        load File.join(root, 'lib', 'tasks', 'exchanges.rake')
       end
 
-      initializer "spree.environment", :before => :load_config_initializers do |app|
+      initializer 'spree.environment', before: :load_config_initializers do |app|
         app.config.spree = Spree::Core::Environment.new
-        Spree::Config = app.config.spree.preferences #legacy access
+        Spree::Config = app.config.spree.preferences # legacy access
       end
 
-      initializer "spree.register.calculators" do |app|
+      initializer 'spree.register.calculators' do |app|
         app.config.spree.calculators.shipping_methods = [
-            Spree::Calculator::Shipping::FlatPercentItemTotal,
-            Spree::Calculator::Shipping::FlatRate,
-            Spree::Calculator::Shipping::FlexiRate,
-            Spree::Calculator::Shipping::PerItem,
-            Spree::Calculator::Shipping::PriceSack]
-
-         app.config.spree.calculators.tax_rates = [
-            Spree::Calculator::DefaultTax]
+          Spree::Calculator::Shipping::FlatPercentItemTotal,
+          Spree::Calculator::Shipping::FlatRate,
+          Spree::Calculator::Shipping::FlexiRate,
+          Spree::Calculator::Shipping::PerItem,
+          Spree::Calculator::Shipping::PriceSack
+        ]
+
+        app.config.spree.calculators.tax_rates = [
+          Spree::Calculator::DefaultTax
+        ]
       end
 
-      initializer "spree.register.stock_splitters" do |app|
+      initializer 'spree.register.stock_splitters' do |app|
         app.config.spree.stock_splitters = [
           Spree::Stock::Splitter::ShippingCategory,
           Spree::Stock::Splitter::Backordered
         ]
       end
 
-      initializer "spree.register.payment_methods" do |app|
+      initializer 'spree.register.payment_methods', after: 'acts_as_list.insert_into_active_record' do |app|
         app.config.spree.payment_methods = [
-            Spree::Gateway::Bogus,
-            Spree::Gateway::BogusSimple,
-            Spree::PaymentMethod::Check ]
+          Spree::Gateway::Bogus,
+          Spree::Gateway::BogusSimple,
+          Spree::PaymentMethod::Check,
+          Spree::PaymentMethod::StoreCredit
+        ]
+      end
+
+      initializer 'spree.register.adjustable_adjusters' do |app|
+        app.config.spree.adjusters = [
+          Spree::Adjustable::Adjuster::Promotion,
+          Spree::Adjustable::Adjuster::Tax
+        ]
       end
 
       # We need to define promotions rules here so extensions and existing apps
@@ -91,7 +102,7 @@ module Spree
       end
 
       # filter sensitive information during logging
-      initializer "spree.params.filter" do |app|
+      initializer 'spree.params.filter' do |app|
         app.config.filter_parameters += [
           :password,
           :password_confirmation,
@@ -99,17 +110,11 @@ module Spree
           :verification_value]
       end
 
-      initializer "spree.core.checking_migrations" do |app|
+      initializer 'spree.core.checking_migrations' do
         Migrations.new(config, engine_name).check
       end
 
       config.to_prepare do
-        # Load spree locales before decorators
-        I18n.load_path += Dir.glob(
-          File.join(
-            File.dirname(__FILE__), '../../../config/locales', '*.{rb,yml}'
-          )
-        )
         # Load application's model / class decorators
         Dir.glob(File.join(File.dirname(__FILE__), '../../../app/**/*_decorator*.rb')) do |c|
           Rails.configuration.cache_classes ? require(c) : load(c)

data/lib/spree/core/environment.rb

@@ -4,7 +4,7 @@ module Spree
       include EnvironmentExtension
 
       attr_accessor :calculators, :payment_methods, :preferences,
-                    :stock_splitters
+                    :adjusters, :stock_splitters
 
       def initialize
         @calculators = Calculators.new

data/lib/spree/core/importer/order.rb

@@ -82,7 +82,7 @@ module Spree
                 shipment.shipped_at = s[:shipped_at]
                 shipment.state      = 'shipped'
               end
-
+ 
               shipment.save!
 
               shipping_method = Spree::ShippingMethod.find_by_name(s[:shipping_method]) || Spree::ShippingMethod.find_by_admin_name!(s[:shipping_method])
@@ -91,6 +91,8 @@ module Spree
               shipment.selected_shipping_rate_id = rate.id
               shipment.update_amounts
 
+              adjustments = s.delete(:adjustments_attributes)
+              create_adjustments_from_params(adjustments, order, shipment)
             rescue Exception => e
               raise "Order import shipments: #{e.message} #{s}"
             end
@@ -99,10 +101,10 @@ module Spree
 
         def self.create_line_items_from_params(line_items, order)
           return {} unless line_items
-          case line_items
+          iterator = case line_items
           when Hash
             ActiveSupport::Deprecation.warn(<<-EOS, caller)
-              Passing a hash is now deprecated and will be removed in Spree 3.1.
+              Passing a hash is now deprecated and will be removed in Spree 4.0.
               It is recommended that you pass it as an array instead.
 
               New Syntax:
@@ -127,53 +129,41 @@ module Spree
                 }
               }
             EOS
-
-            line_items.each_key do |k|
-              begin
-                extra_params = line_items[k].except(:variant_id, :quantity, :sku)
-                line_item = ensure_variant_id_from_params(line_items[k])
-                variant = Spree::Variant.find(line_item[:variant_id])
-                line_item = order.contents.add(variant, line_item[:quantity])
-                # Raise any errors with saving to prevent import succeeding with line items
-                # failing silently.
-                if extra_params.present?
-                  line_item.update_attributes!(extra_params)
-                else
-                  line_item.save!
-                end
-              rescue Exception => e
-                raise "Order import line items: #{e.message} #{line_item}"
-              end
-            end
+            :each_value
           when Array
-            line_items.each do |line_item|
-              begin
-                extra_params = line_item.except(:variant_id, :quantity, :sku)
-                line_item = ensure_variant_id_from_params(line_item)
-                variant = Spree::Variant.find(line_item[:variant_id])
-                line_item = order.contents.add(variant, line_item[:quantity])
-                # Raise any errors with saving to prevent import succeeding with line items
-                # failing silently.
-                if extra_params.present?
-                  line_item.update_attributes!(extra_params)
-                else
-                  line_item.save!
-                end
-              rescue Exception => e
-                raise "Order import line items: #{e.message} #{line_item}"
+            :each
+          end
+
+          line_items.send(iterator) do |line_item|
+            begin
+              adjustments = line_item.delete(:adjustments_attributes)
+              extra_params = line_item.except(:variant_id, :quantity, :sku)
+              line_item = ensure_variant_id_from_params(line_item)
+              variant = Spree::Variant.find(line_item[:variant_id])
+              line_item = order.contents.add(variant, line_item[:quantity])
+              # Raise any errors with saving to prevent import succeeding with line items
+              # failing silently.
+              if extra_params.present?
+                line_item.update_attributes!(extra_params)
+              else
+                line_item.save!
               end
+              create_adjustments_from_params(adjustments, order, line_item)
+            rescue Exception => e
+              raise "Order import line items: #{e.message} #{line_item}"
             end
           end
         end
 
-        def self.create_adjustments_from_params(adjustments, order)
+        def self.create_adjustments_from_params(adjustments, order, adjustable = nil)
           return [] unless adjustments
           adjustments.each do |a|
             begin
-              adjustment = order.adjustments.build(
-                order:  order,
+              adjustment = (adjustable || order).adjustments.build(
+                order: order,
                 amount: a[:amount].to_f,
-                label:  a[:label]
+                label: a[:label],
+                source_type: source_type_from_adjustment(a)
               )
               adjustment.save!
               adjustment.close!
@@ -281,6 +271,13 @@ module Spree
           end
         end
 
+        def self.source_type_from_adjustment(adjustment)
+          if adjustment[:tax]
+            'Spree::TaxRate'
+          elsif adjustment[:promotion]
+            'Spree::PromotionAction'
+          end
+        end
       end
     end
   end

data/lib/spree/core/number_generator.rb

@@ -0,0 +1,52 @@
+module Spree
+  module Core
+    class NumberGenerator < Module
+      BASE           = 10
+      DEFAULT_LENGTH = 9
+      NUMBERS        = (0..9).to_a.freeze
+      LETTERS        = ('A'..'Z').to_a.freeze
+
+      attr_accessor :prefix, :length
+
+      def initialize(options)
+        @random     = Random.new
+        @prefix     = options.fetch(:prefix)
+        @length     = options.fetch(:length, DEFAULT_LENGTH)
+        @candidates = NUMBERS + (options[:letters] ? LETTERS : [])
+      end
+
+      def included(host)
+        generator_method   = method(:generate_permalink)
+        generator_instance = self
+
+        host.class_eval do
+          validates(:number, presence: true, uniqueness: { allow_blank: true })
+
+          before_validation do |instance|
+            instance.number ||= generator_method.call(host)
+          end
+
+          define_singleton_method(:number_generator) { generator_instance }
+        end
+      end
+
+      private
+
+      def generate_permalink(host)
+        length = @length
+
+        loop do
+          candidate = new_candidate(length)
+          return candidate unless host.exists?(number: candidate)
+
+          # If over half of all possible options are taken add another digit.
+          length += 1 if host.count > Rational(BASE**length, 2)
+        end
+      end
+
+      def new_candidate(length)
+        @prefix + length.times.map { @candidates.sample(random: @random) }.join
+      end
+    end # Permalink
+  end # Core
+end # Spree

data/lib/spree/core/product_filters.rb

@@ -9,7 +9,7 @@ module Spree
     #   2) Add it to the config load path, or require it in an initializer, e.g...
     #
     #      # config/initializers/spree.rb
-    #      require 'spree/product_filters'
+    #      require 'spree/core/product_filters'
     #
 
     # set up some basic filters for use with products