effective_orders 5.1.15 → 5.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 889ffac4a7ea9e9402e87b841af4d28b6720e23f512dde7ba4f51978b3ebb045
-  data.tar.gz: 23a247aeb17a3dc13416ac428e9fdaf00bf4d58384b20f3f45d55e705924028c
+  metadata.gz: 7b2e8598ea6f759b602e70d61c5942080c2e528fc8c123b93e3396cbc2a8a26a
+  data.tar.gz: 004e0db3016e081294d9c6fe49a02eb800a13f49f7917f0d83c445e5a615ebdc
 SHA512:
-  metadata.gz: cc23bc2d1e8dd91ea835e18a2e8bb2a0af4d9401b8d4dcdd178a62bf8a36f233a30287c848d97aaceb39875986f00616533d24c610dcdc328548883908e2b68e
-  data.tar.gz: 1d9c8baf3fb9f140123d7829225f4180e2d202a911f5fc695f9f39289e89af585ae60b569900b7c551ddc3e2eee40c78c4d8579975b702a41c9987c706ae24a2
+  metadata.gz: 071a7b38a0c5817e3165ac17b2193288e435b0e92c3a54163bfa7997c7132f9f5c02329421198ad51e9078dc351e77a5984aafa96a2f51bef518913886bf0870
+  data.tar.gz: d2825cd0a8f2c66c6975bcdaf926ab9ac35081383547ad41965813d26a4d2ea059426340a1bba723f22f1ada37c19d2e16a0d34c23423189981fb8611ffdbbbd

data/README.md

@@ -399,8 +399,6 @@ On the Checkout page (`effective_orders.new_order_path`) a new `Effective::Order
 
 If the configuration options `config.billing_address` and/or `config.shipping_address` options are `true` then the user will be prompted for the appropriate addresses, based on [effective_addresses](https://github.com/code-and-effect/effective_addresses/).
 
-If `config.use_address_full_name` is set to `true` then appropriate form field will be shown and the user will be prompted for the appropriate address full name during the checkout process, based on [effective_addresses](https://github.com/code-and-effect/effective_addresses/).
-
 When the user submits the form on this screen, a POST to `effective_orders.order_path` is made, and the `Effective::Order` object is validated and created.
 
 On this final checkout screen, links to all configured payment providers are displayed, and the user may choose which payment processor should be used to make a payment.

data/app/controllers/effective/providers/free.rb

@@ -20,8 +20,7 @@ module Effective
           payment: 'free order. no payment required.',
           provider: 'free',
           card: 'none',
-          purchased_url: free_params[:purchased_url],
-          email: false
+          purchased_url: free_params[:purchased_url]
         )
       end
 

data/app/models/concerns/acts_as_purchasable_parent.rb

@@ -0,0 +1,34 @@
+# Mostly for the callbacks
+
+module ActsAsPurchasableParent
+  extend ActiveSupport::Concern
+
+  module Base
+    def acts_as_purchasable_parent(*options)
+      @acts_as_purchasable_parent = options || []
+      include ::ActsAsPurchasableParent
+    end
+  end
+
+  module ClassMethods
+    def acts_as_purchasable_parent?; true; end
+
+    def before_purchase(&block)
+      send :define_method, :before_purchase do |order| self.instance_exec(order, &block) end
+    end
+
+    def after_purchase(&block)
+      send :define_method, :after_purchase do |order| self.instance_exec(order, &block) end
+    end
+
+    def after_decline(&block)
+      send :define_method, :after_decline do |order| self.instance_exec(order, &block) end
+    end
+  end
+
+  included do
+    has_many :orders, -> { order(:id) }, as: :parent, class_name: 'Effective::Order', dependent: :nullify
+    accepts_nested_attributes_for :orders
+  end
+
+end

data/app/models/concerns/acts_as_subscribable.rb

@@ -24,9 +24,7 @@ module ActsAsSubscribable
       raise :abort unless (subscripter.destroy! rescue false)
     end
 
-    if EffectiveOrders.trial?
-      validates :trialing_until, presence: true
-    end
+    validates :trialing_until, presence: true, if: -> { EffectiveOrders.trial? }
 
     validates :subscription_status, inclusion: { allow_nil: true, in: EffectiveOrders::STATUSES.keys }
 

data/app/models/effective/order.rb

@@ -12,13 +12,11 @@ module Effective
     self.table_name = EffectiveOrders.orders_table_name.to_s
 
     if EffectiveOrders.obfuscate_order_ids
+      raise('unsupported obfuscation with tenant') if defined?(Tenant)
       acts_as_obfuscated format: '###-####-###'
     end
 
-    acts_as_addressable(
-      billing: { singular: true, use_full_name: EffectiveOrders.use_address_full_name },
-      shipping: { singular: true, use_full_name: EffectiveOrders.use_address_full_name }
-    )
+    acts_as_addressable(billing: { singular: true }, shipping: { singular: true })
 
     attr_accessor :terms_and_conditions # Yes, I agree to the terms and conditions
     attr_accessor :confirmed_checkout   # Set on the Checkout Step 1
@@ -68,8 +66,8 @@ module Effective
     before_validation { assign_order_totals }
     before_validation { assign_billing_name }
     before_validation { assign_email }
-    before_validation { assign_last_address }
     before_validation { assign_user_address }
+    before_validation { assign_last_address }
 
     before_validation(if: -> { confirmed_checkout }) do
       assign_attributes(state: EffectiveOrders::CONFIRMED) if pending?
@@ -88,11 +86,14 @@ module Effective
     validates :state, inclusion: { in: EffectiveOrders::STATES.keys }
     validates :subtotal, presence: true
 
-    if EffectiveOrders.minimum_charge.to_i > 0
-      validates :total, presence: true, numericality: {
-        greater_than_or_equal_to: EffectiveOrders.minimum_charge.to_i,
-        message: "must be $#{'%0.2f' % (EffectiveOrders.minimum_charge.to_i / 100.0)} or more. Please add additional items."
-      }, unless: -> { (free? && EffectiveOrders.free?) || (refund? && EffectiveOrders.refund?) }
+    with_options(if: -> { EffectiveOrders.minimum_charge.to_i > 0 }) do
+      validates :total, presence: true
+
+      validate(unless: -> { (free? && EffectiveOrders.free?) || (refund? && EffectiveOrders.refund?) }) do
+        if total.present? && total < EffectiveOrders.minimum_charge
+          self.errors.add(:total, "must be $#{'%0.2f' % (EffectiveOrders.minimum_charge.to_i / 100.0)} or more. Please add additional items.")
+        end
+      end
     end
 
     validate(if: -> { tax_rate.present? }) do
@@ -106,17 +107,9 @@ module Effective
       validates :tax_rate, presence: { message: "can't be determined based on billing address" }
       validates :tax, presence: true
 
-      if EffectiveOrders.billing_address
-        validates :billing_address, presence: true
-      end
-
-      if EffectiveOrders.shipping_address
-        validates :shipping_address, presence: true
-      end
-
-      if EffectiveOrders.collect_note_required
-        validates :note, presence: true
-      end
+      validates :billing_address, presence: true, if: -> { EffectiveOrders.billing_address }
+      validates :shipping_address, presence: true, if: -> { EffectiveOrders.shipping_address }
+      validates :note, presence: true, if: -> { EffectiveOrders.collect_note_required }
     end
 
     # When Purchased
@@ -124,12 +117,21 @@ module Effective
       validates :purchased_at, presence: true
       validates :payment, presence: true
 
-      validates :payment_provider, presence: true, inclusion: { in: EffectiveOrders.payment_providers }
+      validates :payment_provider, presence: true
+
+      validate do
+        self.errors.add(:payment_provider, "unknown payment provider") unless EffectiveOrders.payment_providers.include?(payment_provider)
+      end
+
       validates :payment_card, presence: true
     end
 
     with_options if: -> { deferred? } do
-      validates :payment_provider, presence: true, inclusion: { in: EffectiveOrders.deferred_providers }
+      validates :payment_provider, presence: true
+
+      validate do
+        self.errors.add(:payment_provider, "unknown deferred payment provider") unless EffectiveOrders.deferred_providers.include?(payment_provider)
+      end
     end
 
     scope :deep, -> { includes(:addresses, :user, order_items: :purchasable) }
@@ -636,6 +638,7 @@ module Effective
 
     def run_purchasable_callbacks(name)
       order_items.each { |oi| oi.purchasable.public_send(name, self, oi) if oi.purchasable.respond_to?(name) }
+      parent.public_send(name, self) if parent.respond_to?(name)
     end
 
     def send_email(email, *args)

data/app/models/effective/subscripter.rb

@@ -11,11 +11,9 @@ module Effective
     validates :subscribable, presence: true, if: -> { stripe_plan_id.present? }
     validates :customer, presence: true
 
-    validates :stripe_plan_id, inclusion: {
-      allow_blank: true,
-      in: EffectiveOrders.stripe_plans.map { |plan| plan[:id] },
-      message: 'unknown plan'
-    }
+    validate(if: -> { stripe_plan_id.present? }) do
+      self.errors.add(:stripe_plan_id, 'unknown plan') unless EffectiveOrders.stripe_plans.map { |plan| plan[:id] }.include?(stripe_plan_id)
+    end
 
     validate(if: -> { stripe_plan_id && plan && plan[:amount] > 0 }) do
       self.errors.add(:stripe_token, 'updated payment card required') if stripe_token.blank? && token_required?

data/app/models/effective/subscription.rb

@@ -47,7 +47,10 @@ module Effective
     validates :subscribable, presence: true
 
     validates :stripe_plan_id, presence: true
-    validates :stripe_plan_id, inclusion: { in: EffectiveOrders.stripe_plans.map { |plan| plan[:id] } }
+
+    validate(if: -> { stripe_plan_id.present? }) do
+      self.errors.add(:stripe_plan_id, 'unknown plan') unless EffectiveOrders.stripe_plans.map { |plan| plan[:id] }.include?(stripe_plan_id)
+    end
 
     validates :stripe_subscription_id, presence: true
 

data/app/views/effective/orders/_order_shipping.html.haml

@@ -9,7 +9,7 @@
     %tbody
       %tr
         %td
-          - if order.billing_address.blank? || !EffectiveOrders.use_address_full_name
+          - if order.billing_address.blank? || order.billing_address.full_name.blank?
             = order.billing_name
             %br
 

data/config/effective_orders.rb

@@ -19,7 +19,6 @@ EffectiveOrders.setup do |config|
   # Require these addresses when creating a new Order.  Works with effective_addresses gem
   config.billing_address = true
   config.shipping_address = false
-  config.use_address_full_name = true
 
   # Use effective_obfuscation gem to change order.id into a seemingly random 10-digit number
   config.obfuscate_order_ids = false

data/lib/effective_orders/engine.rb

@@ -6,6 +6,7 @@ module EffectiveOrders
     initializer 'effective_orders.active_record' do |app|
       ActiveSupport.on_load :active_record do
         ActiveRecord::Base.extend(ActsAsPurchasable::Base)
+        ActiveRecord::Base.extend(ActsAsPurchasableParent::Base)
         ActiveRecord::Base.extend(ActsAsSubscribable::Base)
         ActiveRecord::Base.extend(ActsAsSubscribableBuyer::Base)
       end

data/lib/effective_orders/version.rb

@@ -1,3 +1,3 @@
 module EffectiveOrders
-  VERSION = '5.1.15'.freeze
+  VERSION = '5.2.1'.freeze
 end

data/lib/effective_orders.rb

@@ -36,7 +36,7 @@ module EffectiveOrders
       :layout, :mailer_class_name, :mailer,
       :orders_collection_scope, :order_tax_rate_method,
       :obfuscate_order_ids, :use_effective_qb_sync,
-      :billing_address, :shipping_address, :use_address_full_name,
+      :billing_address, :shipping_address,
       :collect_note, :collect_note_required, :collect_note_message,
       :terms_and_conditions, :terms_and_conditions_label, :minimum_charge,
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_orders
 version: !ruby/object:Gem::Version
-  version: 5.1.15
+  version: 5.2.1
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-20 00:00:00.000000000 Z
+date: 2021-11-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -215,6 +215,7 @@ files:
 - app/helpers/effective_subscriptions_helper.rb
 - app/mailers/effective/orders_mailer.rb
 - app/models/concerns/acts_as_purchasable.rb
+- app/models/concerns/acts_as_purchasable_parent.rb
 - app/models/concerns/acts_as_subscribable.rb
 - app/models/concerns/acts_as_subscribable_buyer.rb
 - app/models/effective/cart.rb