pay 3.0.18 → 3.0.22

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a37f4ee9640c9f48545166512e3ce5dfd8f17555c6181794807db5a6cc88e4ac
-  data.tar.gz: 994889fa6c1e3c55b836d3597f95046eaed46d1c551154c57c83543312153965
+  metadata.gz: 2dfae5ada22ac1b2cecc080e154223aed66740ce53327fcaf012fa28ed2e6580
+  data.tar.gz: cfdf57b6ee3c1e011a67bf67923d55ca8da9bb6b0cffa8334af4713296aac33c
 SHA512:
-  metadata.gz: d6e97f172fb3a5fbb9b62e3867fb29e363f9c5ea3f1fc385ab22a9ea59f27547ea84ca357eac1473e2ca4b25584d0c28f58f9124ca80e729cd5fa1da465d5b71
-  data.tar.gz: f455c3cc02a3b93f7eb5dd04c2edf0c67d0f72c3e2bddd3ced8e509a8b557705609b20dab00b0b40daab19734b46e097f2c4fcbe2f6aaf5df79f8291b75f6d3d
+  metadata.gz: fbe399383317007d063340568d983e7fcdc928f352ccfe3126ea2ec686e5c7ed96978ba576af9774226d1c702e4b4f8c2c3dee6cb8be0e5d823f2a267bfff3a2
+  data.tar.gz: 00eb2d8b09de4457a7aa3fe861e393b212d87ad74065817a4fffe53f04eb6e6fc05ba9d4d7bb6ed8a483a01c8f3334f438ecd4cc75d93c1e03e3f6b682400861

data/app/jobs/pay/customer_sync_job.rb

@@ -5,7 +5,7 @@ module Pay
     def perform(pay_customer_id)
       Pay::Customer.find(pay_customer_id).update_customer!
     rescue ActiveRecord::RecordNotFound
-      Rails.logger.info "Couldn't find a #{class_name} with ID = #{id}"
+      Rails.logger.info "Couldn't find a Pay::Customer with ID = #{id}"
     end
   end
 end

data/app/views/pay/payments/show.html.erb

@@ -66,7 +66,7 @@
 <script type="module">
   window.stripe = Stripe('<%= Pay::Stripe.public_key %>');
 
-  import { Application, Controller } from 'https://cdn.skypack.dev/stimulus'
+  import { Application, Controller } from 'https://cdn.skypack.dev/@hotwired/stimulus'
   const application = Application.start()
 
   application.register('payment-intent', class extends Controller {

data/app/views/pay/user_mailer/receipt.html.erb

@@ -10,7 +10,7 @@ Amount: <%= params[:charge].amount_with_currency %><br/>
 Charged to: <%= params[:charge].charged_to %><br/>
 Transaction ID: <%= params[:charge].id %><br/>
 Date: <%= params[:charge].created_at %><br/>
-<% if params[:charge].customer.owner.extra_billing_info? %>
+<% if params[:charge].customer.owner.try(:extra_billing_info?) %>
 <%= params[:charge].customer.owner.extra_billing_info %><br/>
 <% end %>
 <br/>

data/app/views/pay/user_mailer/refund.html.erb

@@ -11,7 +11,7 @@ Amount: <%= params[:charge].amount_refunded_with_currency %><br/>
 Refunded to: <%= params[:charge].charged_to %><br/>
 Transaction ID: <%= params[:charge].id %><br/>
 Date: <%= params[:charge].created_at %><br/>
-<% if params[:charge].customer.owner.extra_billing_info? %>
+<% if params[:charge].customer.owner.try(:extra_billing_info?) %>
 <%= params[:charge].customer.owner.extra_billing_info %><br/>
 <% end %>
 <br/>

data/lib/pay/braintree/billable.rb

@@ -30,7 +30,8 @@ module Pay
 
           customer
         else
-          result = gateway.customer.create(email: email, first_name: try(:first_name), last_name: try(:last_name), payment_method_nonce: payment_method_token)
+          first_name, last_name = customer_name.split(" ", 2)
+          result = gateway.customer.create(email: email, first_name: first_name, last_name: last_name, payment_method_nonce: payment_method_token)
           raise Pay::Braintree::Error, result unless result.success?
           pay_customer.update!(processor_id: result.customer.id)
 
@@ -47,6 +48,13 @@ module Pay
         raise Pay::Braintree::Error, e
       end
 
+      # Syncs name and email to Braintree::Customer
+      def update_customer!
+        return unless processor_id?
+        first_name, last_name = customer_name.split(" ", 2)
+        gateway.customer.update(processor_id, first_name: first_name, last_name: last_name, email: email)
+      end
+
       def charge(amount, options = {})
         args = {
           amount: amount.to_i / 100.0,

data/lib/pay/fake_processor/billable.rb

@@ -19,6 +19,10 @@ module Pay
         pay_customer
       end
 
+      def update_customer!
+        # pass
+      end
+
       def charge(amount, options = {})
         # Make to generate a processor_id
         customer

data/lib/pay/paddle/billable.rb

@@ -18,6 +18,10 @@ module Pay
         # pass
       end
 
+      def update_customer!
+        # pass
+      end
+
       def charge(amount, options = {})
         subscription = pay_customer.subscription
         return unless subscription.processor_id

data/lib/pay/paddle/subscription.rb

@@ -75,7 +75,14 @@ module Pay
       end
 
       def cancel
-        ends_at = on_trial? ? trial_ends_at : processor_subscription.next_payment[:date]
+        ends_at = if on_trial?
+          trial_ends_at
+        elsif paused?
+          paddle_paused_from
+        else
+          processor_subscription.next_payment&.fetch(:date) || Time.current
+        end
+
         PaddlePay::Subscription::User.cancel(processor_id)
         pay_subscription.update(status: :canceled, ends_at: ends_at)
 
@@ -106,7 +113,7 @@ module Pay
       def pause
         attributes = {pause: true}
         response = PaddlePay::Subscription::User.update(processor_id, attributes)
-        pay_subscription.update(paddle_paused_from: Time.zone.parse(response[:next_payment][:date]))
+        pay_subscription.update(paddle_paused_from: Time.zone.parse(response.dig(:next_payment, :date)))
       rescue ::PaddlePay::PaddlePayError => e
         raise Pay::Paddle::Error, e
       end

data/lib/pay/receipts.rb

@@ -21,7 +21,7 @@ module Pay
         [I18n.t("pay.receipt.amount"), Pay::Currency.format(amount, currency: currency)],
         [I18n.t("pay.receipt.charged_to"), charged_to]
       ]
-      line_items << [I18n.t("pay.receipt.additional_info"), customer.owner.extra_billing_info] if customer.owner.extra_billing_info?
+      line_items << [I18n.t("pay.receipt.additional_info"), customer.owner.extra_billing_info] if customer.owner.try(:extra_billing_info?)
       line_items << [I18n.t("pay.receipt.refunded"), Pay::Currency.format(amount_refunded, currency: currency)] if refunded?
 
       defaults = {
@@ -49,7 +49,7 @@ module Pay
 
     def invoice_pdf(**options)
       bill_to = [customer.owner.name]
-      bill_to += [customer.owner.extra_billing_info] if customer.owner.extra_billing_info?
+      bill_to += [customer.owner.extra_billing_info] if customer.owner.try(:extra_billing_info?)
       bill_to += [nil, customer.owner.email]
 
       total = Pay::Currency.format(amount, currency: currency)

data/lib/pay/stripe/billable.rb

@@ -44,6 +44,12 @@ module Pay
         raise Pay::Stripe::Error, e
       end
 
+      # Syncs name and email to Stripe::Customer
+      def update_customer!
+        return unless processor_id?
+        ::Stripe::Customer.update(processor_id, {name: customer_name, email: email}, stripe_options)
+      end
+
       def charge(amount, options = {})
         add_payment_method(payment_method_token, default: true) if payment_method_token?
 

data/lib/pay/version.rb

@@ -1,3 +1,3 @@
 module Pay
-  VERSION = "3.0.18"
+  VERSION = "3.0.22"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pay
 version: !ruby/object:Gem::Version
-  version: 3.0.18
+  version: 3.0.22
 platform: ruby
 authors:
 - Jason Charnes
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-15 00:00:00.000000000 Z
+date: 2021-10-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -123,14 +123,7 @@ files:
 - config/locales/en.yml
 - config/routes.rb
 - db/migrate/1_create_pay_tables.rb
-- lib/generators/active_record/billable_generator.rb
-- lib/generators/active_record/merchant_generator.rb
-- lib/generators/active_record/templates/billable_migration.rb
-- lib/generators/active_record/templates/merchant_migration.rb
-- lib/generators/pay/billable_generator.rb
 - lib/generators/pay/email_views_generator.rb
-- lib/generators/pay/merchant_generator.rb
-- lib/generators/pay/orm_helpers.rb
 - lib/generators/pay/views_generator.rb
 - lib/pay.rb
 - lib/pay/adapter.rb

data/lib/generators/active_record/billable_generator.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-require "rails/generators/active_record"
-require "generators/pay/orm_helpers"
-
-module ActiveRecord
-  module Generators
-    class BillableGenerator < ActiveRecord::Generators::Base
-      include Pay::Generators::OrmHelpers
-      source_root File.expand_path("../templates", __FILE__)
-
-      def copy_pay_billable_migration
-        if (behavior == :invoke && model_exists?) || (behavior == :revoke && migration_exists?(table_name))
-          migration_template "billable_migration.rb", "#{migration_path}/add_pay_billable_to_#{table_name}.rb", migration_version: migration_version
-        else
-          say "#{model_path} does not exist.", :red
-          say "⚠️  Make sure the #{name} model exists before running this generator."
-        end
-      end
-
-      # If the file already contains the contents, the user will receive this warning:
-      #
-      #   File unchanged! The supplied flag value not found!
-      #
-      # This can be ignored as it just means the contents already exist and the file is unchanged.
-      # Thor will be updated to improve this message: https://github.com/rails/thor/issues/706
-      def inject_pay_billable_content
-        return unless model_exists?
-
-        content = model_contents
-        class_path = (namespaced? ? class_name.to_s.split("::") : [class_name])
-        indent_depth = class_path.size - 1
-        content = content.split("\n").map { |line| "  " * indent_depth + line }.join("\n") << "\n"
-        inject_into_class(model_path, class_path.last, content)
-      end
-
-      private
-
-      def model_contents
-        "  include Pay::Billable"
-      end
-    end
-  end
-end

data/lib/generators/active_record/merchant_generator.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-require "rails/generators/active_record"
-require "generators/pay/orm_helpers"
-
-module ActiveRecord
-  module Generators
-    class MerchantGenerator < ActiveRecord::Generators::Base
-      include Pay::Generators::OrmHelpers
-      source_root File.expand_path("../templates", __FILE__)
-
-      def copy_pay_merchant_migration
-        if (behavior == :invoke && model_exists?) || (behavior == :revoke && migration_exists?(table_name))
-          migration_template "merchant_migration.rb", "#{migration_path}/add_pay_merchant_to_#{table_name}.rb", migration_version: migration_version
-        else
-          say "#{model_path} does not exist.", :red
-          say "⚠️  Make sure the #{name} model exists before running this generator."
-        end
-      end
-
-      # If the file already contains the contents, the user will receive this warning:
-      #
-      #   File unchanged! The supplied flag value not found!
-      #
-      # This can be ignored as it just means the contents already exist and the file is unchanged.
-      # Thor will be updated to improve this message: https://github.com/rails/thor/issues/706
-      def inject_pay_merchant_content
-        return unless model_exists?
-
-        content = model_contents
-        class_path = (namespaced? ? class_name.to_s.split("::") : [class_name])
-        indent_depth = class_path.size - 1
-        content = content.split("\n").map { |line| "  " * indent_depth + line }.join("\n") << "\n"
-        inject_into_class(model_path, class_path.last, content)
-      end
-
-      private
-
-      def model_contents
-        "  include Pay::Merchant"
-      end
-    end
-  end
-end

data/lib/generators/active_record/templates/billable_migration.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-class AddPayBillableTo<%= table_name.camelize %> < ActiveRecord::Migration<%= migration_version %>
-  def change
-    change_table :<%= table_name %>, bulk: true do |t|
-      t.string :processor
-      t.string :processor_id
-      t.public_send(Pay::Adapter.json_column_type, :pay_data)
-      t.datetime :trial_ends_at
-      t.string :card_type
-      t.string :card_last4
-      t.string :card_exp_month
-      t.string :card_exp_year
-      t.text :extra_billing_info
-    end
-  end
-end

data/lib/generators/active_record/templates/merchant_migration.rb

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-class AddPayMerchantTo<%= table_name.camelize %> < ActiveRecord::Migration<%= migration_version %>
-  def change
-    add_column table_name, :merchant_processor, :string
-
-    # We may already have the pay_data column if a Pay::Billable object is also a Pay::Merchant
-    unless ActiveRecord::Base.connection.column_exists?(table_name, :pay_data)
-      add_column :<%= table_name %>, :pay_data, Pay::Adapter.json_column_type
-    end
-  end
-end

data/lib/generators/pay/billable_generator.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-require "rails/generators/named_base"
-
-module Pay
-  module Generators
-    class BillableGenerator < Rails::Generators::NamedBase
-      include Rails::Generators::ResourceHelpers
-
-      source_root File.expand_path("../templates", __FILE__)
-
-      desc "Generates a migration to add Pay::Billable fields to a model."
-
-      hook_for :orm
-    end
-  end
-end

data/lib/generators/pay/merchant_generator.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-require "rails/generators/named_base"
-
-module Pay
-  module Generators
-    class MerchantGenerator < Rails::Generators::NamedBase
-      include Rails::Generators::ResourceHelpers
-
-      source_root File.expand_path("../templates", __FILE__)
-
-      desc "Generates a migration to add Pay::Merchant fields to a model."
-
-      hook_for :orm
-    end
-  end
-end

data/lib/generators/pay/orm_helpers.rb

@@ -1,39 +0,0 @@
-# frozen_string_literal: true
-
-module Pay
-  module Generators
-    module OrmHelpers
-      private
-
-      def model_exists?
-        File.exist?(File.join(destination_root, model_path))
-      end
-
-      def migration_exists?(table_name)
-        Dir.glob("#{File.join(destination_root, migration_path)}/[0-9]*_*.rb").grep(/\d+_add_devise_to_#{table_name}.rb$/).first
-      end
-
-      def migration_path
-        if Rails.version >= "5.0.3"
-          db_migrate_path
-        else
-          @migration_path ||= File.join("db", "migrate")
-        end
-      end
-
-      def model_path
-        @model_path ||= File.join("app", "models", "#{file_path}.rb")
-      end
-
-      def migration_version
-        if rails5_and_up?
-          "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
-        end
-      end
-
-      def rails5_and_up?
-        Rails::VERSION::MAJOR >= 5
-      end
-    end
-  end
-end