pay 6.2.0 → 6.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9b9bea4dde4674321c5fcc0315db2fbd79afa882e329b6ba922aa0cc01c1a91e
-  data.tar.gz: 0fe8b739284c825223d0ec377407ddf2f3fef5f846a5b11e57cf80ea144de2a8
+  metadata.gz: 10e2efeb851c80d2bb81878942929c95411fa43a781bfdfee41f3d9b99da61b4
+  data.tar.gz: 7080610378fd2efa259611084f41160143186ef06fb1c9d997e7d1cee72eb74d
 SHA512:
-  metadata.gz: 54184916de0ec236cdfb4eef54ff62add646b7ddc1f4cb43fa150e5b7ef517f1a780603a63807c626ef41567522d765cce9a15390b728f98a2701e3b3ddd44ea
-  data.tar.gz: 256c12615777ed0933b41290e7f33a52e155e9be192f034ad9e95a1ae301198e1f8c3d4ceae6568e021612981b8c32de1cae80a65a15f548bc41b68c318dd4b1
+  metadata.gz: 4d3032094987e54cf1834619b96064cf77335f03b2e45555de4eb059cee0c6b1e0783abb6077621fe138301dd5fe29b415a3cd4e829c63bf55493208275e0025
+  data.tar.gz: 6331154cae0f9f1c99a63b57c92e0b82e9197247ead694e9854a081eea19a2ff2859eef07d839b10b1d7204c00e42694ffce25e3f4f54f1e984e9f333dadc96e

data/app/models/pay/subscription.rb

@@ -11,7 +11,7 @@ module Pay
     scope :on_trial, -> { where.not(trial_ends_at: nil).where("#{table_name}.trial_ends_at > ?", Time.current) }
     scope :cancelled, -> { where.not(ends_at: nil) }
     scope :on_grace_period, -> { cancelled.where("#{table_name}.ends_at > ?", Time.current) }
-    scope :active, -> { where(status: ["trialing", "active"], ends_at: nil).pause_not_started.or(on_grace_period).or(on_trial) }
+    scope :active, -> { where(status: ["trialing", "active", "canceled"], ends_at: nil).pause_not_started.or(on_grace_period).or(on_trial) }
     scope :paused, -> { where(status: "paused").or(where("pause_starts_at <= ?", Time.current)) }
     scope :pause_not_started, -> { where("pause_starts_at IS NULL OR pause_starts_at > ?", Time.current) }
     scope :active_or_paused, -> { active.or(paused) }
@@ -98,6 +98,10 @@ module Pay
       canceled?
     end
 
+    def ended?
+      ends_at? && Time.current.after?(ends_at)
+    end
+
     def on_grace_period?
       (ends_at? && Time.current < ends_at) ||
         ((status == "paused" || pause_behavior == "void") && will_pause?)
@@ -108,11 +112,19 @@ module Pay
     end
 
     def pause_active?
-      (status == "paused" || pause_behavior == "void") && (pause_starts_at.nil? || pause_starts_at >= Time.current)
+      if status == "paused" # Paddle
+        true
+      elsif pause_behavior == "void"
+        pause_starts_at.nil? || Time.current.after?(pause_starts_at)
+      end
     end
 
+    # If you cancel during a trial, you should still retain access until the end of the trial
+    # Otherwise a subscription is active unless it has ended or is currently paused
+    # Check the subscription status so we don't accidentally consider "incomplete", "past_due", or other statuses as active
     def active?
-      ["trialing", "active"].include?(status) && (!(canceled? || paused?) || on_trial? || on_grace_period?)
+      ["trialing", "active", "canceled"].include?(status) &&
+        (!(canceled? || paused?) || on_trial? || on_grace_period?)
     end
 
     def past_due?

data/lib/pay/braintree/subscription.rb

@@ -32,12 +32,20 @@ module Pay
           created_at: object.created_at,
           current_period_end: object.billing_period_end_date,
           current_period_start: object.billing_period_start_date,
-          ends_at: (object.updated_at if object.status == "Canceled"),
           processor_plan: object.plan_id,
           status: object.status.underscore,
           trial_ends_at: (object.created_at + object.trial_duration.send(object.trial_duration_unit) if object.trial_period)
         }
 
+        # Canceled subscriptions should have access through the paid_through_date or updated_at
+        if object.status == "Canceled"
+          attributes[:ends_at] = object.updated_at
+
+        # Set grace period for subscriptions that are marked to be canceled
+        elsif object.status == "Active" && object.number_of_billing_cycles
+          attributes[:ends_at] = object.paid_through_date.end_of_day
+        end
+
         pay_subscription = pay_customer.subscriptions.find_by(processor_id: object.id)
         if pay_subscription
           pay_subscription.with_lock { pay_subscription.update!(attributes) }
@@ -66,29 +74,21 @@ module Pay
       end
 
       def cancel(**options)
-        subscription = processor_subscription
-
-        if on_trial?
-          gateway.subscription.cancel(processor_subscription.id)
-          pay_subscription.update(status: :canceled, ends_at: trial_ends_at)
+        result = if on_trial?
+          gateway.subscription.cancel(processor_id)
         else
           gateway.subscription.update(subscription.id, {
             number_of_billing_cycles: subscription.current_billing_cycle
           })
-          pay_subscription.update(status: :canceled, ends_at: subscription.billing_period_end_date.to_date)
         end
+        pay_subscription.sync!(object: result.subscription)
       rescue ::Braintree::BraintreeError => e
         raise Pay::Braintree::Error, e
       end
 
       def cancel_now!(**options)
-        gateway.subscription.cancel(processor_subscription.id)
-        ends_at = Time.current
-        pay_subscription.update!(
-          status: :canceled,
-          trial_ends_at: (ends_at if pay_subscription.trial_ends_at?),
-          ends_at: ends_at
-        )
+        result = gateway.subscription.cancel(processor_id)
+        pay_subscription.sync!(object: result.subscription)
       rescue ::Braintree::BraintreeError => e
         raise Pay::Braintree::Error, e
       end
@@ -97,10 +97,6 @@ module Pay
         raise NotImplementedError, "Braintree does not support setting quantity on subscriptions"
       end
 
-      def on_grace_period?
-        canceled? && Time.current < ends_at
-      end
-
       def paused?
         false
       end

data/lib/pay/version.rb

@@ -1,3 +1,3 @@
 module Pay
-  VERSION = "6.2.0"
+  VERSION = "6.2.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pay
 version: !ruby/object:Gem::Version
-  version: 6.2.0
+  version: 6.2.2
 platform: ruby
 authors:
 - Jason Charnes
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-12-23 00:00:00.000000000 Z
+date: 2023-01-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -220,7 +220,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.3.26
 signing_key:
 specification_version: 4
 summary: Payments engine for Ruby on Rails