effective_orders 5.9.3 → 5.9.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c21ba05da7239366ba59d58f08bc997193192c6b9cb83c93caade90b61072bd
-  data.tar.gz: 965fe919c2c7a6537dd7b15427291cef1119c768ccd38cf1fdbcaf282b1fd95a
+  metadata.gz: 35bd45132852365a2cc570b4b6a7fbbf5cc62d42cd9f506710da80a0c77daf82
+  data.tar.gz: e97f9869c710bdfb91cfa35b6b0bc091b429b175cdc7af9b066283d02d90bac6
 SHA512:
-  metadata.gz: 344e2cdaa0de66f512335a2c9da024de369713f34c84480a1b813b59874343bc4fc2ac81579f35ce718153346eb9e53826f46e657a5def99a6ff266b0f0aa45e
-  data.tar.gz: 51f49f85b475d26fea9cfc593f627ed4e1c1761075d6fb508264419246485035995272e638da63cb5ed99635a9d7ff4b1aad0c1c34b8bc7c385e26eda737f069
+  metadata.gz: 02036ff52d79eaa0c47e6f2353f23b33d6168222abe6337a77880fb7c48d0c7de7b1960235ed2b2dcb9e304aac36b8bddbbf0f1132fee7395bed3638be80c2bd
+  data.tar.gz: 1929601f0588f3cefde7eec8ceec3f796b03822406e9ae1533507a783c3f0867a5c408f21a9954c12eaa225c912817044102c869d1a334660f328a80a9381a2c

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

@@ -13,7 +13,7 @@ module Effective
 
         payment = validate_stripe_payment(stripe_params[:payment_intent_id])
 
-        if payment.blank? || !payment.kind_of?(Hash)
+        if payment.blank?
           return order_declined(payment: payment, provider: 'stripe', declined_url: stripe_params[:declined_url])
         end
 
@@ -37,36 +37,35 @@ module Effective
       end
 
       def validate_stripe_payment(payment_intent_id)
-        begin
-          intent = EffectiveOrders.with_stripe { ::Stripe::PaymentIntent.retrieve(payment_intent_id) }
-
-          raise('status is not succeeded') unless intent.status == 'succeeded'
-          raise('charges are not present') unless intent.charges.present?
-
-          charge = intent.charges.data.first
-          raise('charge not succeeded') unless charge.status == 'succeeded'
-
-          card = charge.payment_method_details.try(:card) || {}
-          active_card = "**** **** **** #{card['last4']} #{card['brand']} #{card['exp_month']}/#{card['exp_year']}" if card.present?
-
-          {
-            charge_id: charge.id,
-            payment_method_id: charge.payment_method,
-            payment_intent_id: intent.id,
-
-            active_card: active_card,
-            card: card['brand'],
-
-            amount: charge.amount,
-            created: charge.created,
-            currency: charge.currency,
-            customer: charge.customer,
-            status: charge.status
-          }.compact
-
-        rescue => e
-          e.message
-        end
+        intent = EffectiveOrders.with_stripe { ::Stripe::PaymentIntent.retrieve(payment_intent_id) }
+        raise('expected stripe intent to be present') if intent.blank?
+        return unless intent.status == 'succeeded'
+
+        # Stripe API version 2022-11-15 and 2022-08-01
+        charge_id = intent.try(:latest_charge) || (intent.charges.data.first.id rescue nil)
+        raise('expected stripe charge_id to be present') if charge_id.blank?
+
+        charge = EffectiveOrders.with_stripe { ::Stripe::Charge.retrieve(charge_id) }
+        raise('expected stripe charge to be present') if charge.blank?
+        return unless charge.status == 'succeeded'
+
+        card = charge.payment_method_details.try(:card) || {}
+        active_card = "**** **** **** #{card['last4']} #{card['brand']} #{card['exp_month']}/#{card['exp_year']}" if card.present?
+
+        {
+          charge_id: charge.id,
+          payment_method_id: charge.payment_method,
+          payment_intent_id: intent.id,
+
+          active_card: active_card,
+          card: card['brand'],
+
+          amount: charge.amount,
+          created: charge.created,
+          currency: charge.currency,
+          customer: charge.customer,
+          status: charge.status
+        }.compact
       end
 
     end

data/lib/effective_orders/version.rb

@@ -1,3 +1,3 @@
 module EffectiveOrders
-  VERSION = '5.9.3'.freeze
+  VERSION = '5.9.4'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_orders
 version: !ruby/object:Gem::Version
-  version: 5.9.3
+  version: 5.9.4
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-11-29 00:00:00.000000000 Z
+date: 2022-12-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails