effective_orders 5.2.8 → 5.2.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7850176b4c32949c586aa453834799cb0063919a1eb7437e5111f2a280e19fc0
-  data.tar.gz: e5753a21c4ba634a779bce34c0dafae0d94492db53f51bbb89720f860ea17985
+  metadata.gz: e6fc5fddf4893dd467c30bd2ae4f505b1c4f4f191e819601364bc230d8530627
+  data.tar.gz: 6a157d58abcf4836b1324b17dbee3fcb502e23e2924d59daa1ef8a957cb91fbb
 SHA512:
-  metadata.gz: 6251dfd17f63fce47aa1484d26bfb68afaa24f080819f8f8a00ffc21e280e36396c0d25ee6d0fc400af01120539d39f71aaef91bdb8770d9f28a87dd5198b4d6
-  data.tar.gz: 65a6ad794d855f8057f5e14051ec1d0b7099e21e54b864238f61b5a020f17f0f1a93042a30c74dcd0b5b581385ac1e466eb075344054958506a9ee2ad7e7605a
+  metadata.gz: 9c98604d9238a92c62d41422b31f63001144fa47eaa5dbcf7359cf98ec849e8846d07db9be11e498df0024b66212bb674afcc646ab9f7bf4922c8b446a947ba5
+  data.tar.gz: 537d27c651f334bb17a189d4f3085eca6f0b33e9aaa742ea9ae270f5f3f6d00746d56007a10a5abbe2941ecbd083ae42a052bfbeecc627f0a79e31076b2dcb7c

data/app/helpers/effective_moneris_checkout_helper.rb

@@ -1,4 +1,5 @@
 module EffectiveMonerisCheckoutHelper
+  SCRUB = /[^\w\d#,\s]/
 
   def moneris_checkout_preload_request(order)
     # Make the Preload Request
@@ -19,8 +20,8 @@ module EffectiveMonerisCheckoutHelper
       language: 'en',
 
       contact_details: {
-        first_name: order.billing_first_name,
-        last_name: order.billing_last_name,
+        first_name: moneris_checkout_scrub(order.billing_first_name),
+        last_name: moneris_checkout_scrub(order.billing_last_name),
         email: order.email,
       }
     }
@@ -28,9 +29,9 @@ module EffectiveMonerisCheckoutHelper
     if (address = order.billing_address).present?
       params.merge!(
         billing_details: {
-          address_1: address.address1,
-          address_2: address.address2,
-          city: address.city,
+          address_1: moneris_checkout_scrub(address.address1),
+          address_2: moneris_checkout_scrub(address.address2),
+          city: moneris_checkout_scrub(address.city),
           province: address.state_code,
           country: address.country_code,
           postal_code: address.postal_code
@@ -41,8 +42,8 @@ module EffectiveMonerisCheckoutHelper
     if (address = order.shipping_address).present?
       params.merge!(
         shipping_details: {
-          address_1: address.address1,
-          address_2: address.address2,
+          address_1: moneris_checkout_scrub(address.address1),
+          address_2: moneris_checkout_scrub(address.address2),
           city: address.city,
           province: address.state_code,
           country: address.country_code,
@@ -62,4 +63,9 @@ module EffectiveMonerisCheckoutHelper
     }
   end
 
+  def moneris_checkout_scrub(value)
+    return value unless value.kind_of?(String)
+    value.gsub(SCRUB, '')
+  end
+
 end

data/app/models/effective/order.rb

@@ -190,6 +190,31 @@ module Effective
       self
     end
 
+    def remove(*items)
+      raise 'unable to alter a purchased order' if purchased?
+      raise 'unable to alter a declined order' if declined?
+
+      removed = items.map do |item|
+        order_item = if item.kind_of?(Effective::OrderItem)
+          order_items.find { |oi| oi == item }
+        else
+          order_items.find { |oi| oi.purchasable == item }
+        end
+
+        raise("Unable to find order item for #{item}") if order_item.blank?
+        order_item
+      end
+
+      removed.each { |order_item| order_item.mark_for_destruction }
+
+      # Make sure to reset stored aggregates
+      self.total = nil
+      self.subtotal = nil
+      self.tax = nil
+
+      removed.length == 1 ? removed.first : removed
+    end
+
     # Items can be an Effective::Cart, an Effective::order, a single acts_as_purchasable, or multiple acts_as_purchasables
     # add(Product.first) => returns an Effective::OrderItem
     # add(Product.first, current_cart) => returns an array of Effective::OrderItems
@@ -249,7 +274,7 @@ module Effective
       raise('already purchased') if purchased?
       raise('must be pending or confirmed') unless pending? || confirmed?
 
-      order_items.each do |item|
+      present_order_items.each do |item|
         purchasable = item.purchasable
 
         if purchasable.blank? || purchasable.marked_for_destruction?
@@ -318,9 +343,9 @@ module Effective
       Effective::Order.new(self)
     end
 
-    # For moneris and moneris_checkout. Just a unique value.
+    # For moneris and moneris_checkout. Just a unique value. Must be 50 characters or fewer or will raise moneris error.
     def transaction_id
-      [to_param, billing_name.to_s.parameterize.presence, Time.zone.now.to_i, rand(10000)].compact.join('-')
+      [to_param, billing_name.to_s.parameterize.first(20).presence, Time.zone.now.to_i, rand(1000..9999)].compact.join('-')
     end
 
     def billing_first_name
@@ -358,11 +383,11 @@ module Effective
     end
 
     def purchasables
-      order_items.map { |order_item| order_item.purchasable }
+      present_order_items.map { |order_item| order_item.purchasable }
     end
 
     def subtotal
-      self[:subtotal] || order_items.map { |oi| oi.subtotal }.sum
+      self[:subtotal] || present_order_items.map { |oi| oi.subtotal }.sum
     end
 
     def tax_rate
@@ -386,7 +411,7 @@ module Effective
     end
 
     def num_items
-      order_items.map { |oi| oi.quantity }.sum
+      present_order_items.map { |oi| oi.quantity }.sum
     end
 
     def send_order_receipt_to_admin?
@@ -586,7 +611,7 @@ module Effective
 
     def assign_order_totals
       self.subtotal = present_order_items.map { |oi| oi.subtotal }.sum
-      self.tax_rate = get_tax_rate() unless (tax_rate || 0) > 0
+      self.tax_rate = get_tax_rate()
       self.tax = get_tax()
       self.total = subtotal + (tax || 0)
     end

data/lib/effective_orders/version.rb

@@ -1,3 +1,3 @@
 module EffectiveOrders
-  VERSION = '5.2.8'.freeze
+  VERSION = '5.2.12'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_orders
 version: !ruby/object:Gem::Version
-  version: 5.2.8
+  version: 5.2.12
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-21 00:00:00.000000000 Z
+date: 2022-01-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails