spree-paypal_platform 5.1.0 → 5.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3002d47c7c53f6eb690a9dbd4717e47983e47083feac8a359369378c8435d24f
-  data.tar.gz: 5b4e7e078632cd6dc4e7e2796edc696bc543c4cfd7360b3a721064fc2114b62e
+  metadata.gz: 963337c6ac7e3f41915fa04edad216c40f021ae69957c16c60a6d503596b9d5f
+  data.tar.gz: fab2c096c3f02d758cf1e90d1019e06184601fb74d9176ea0b52e8ce3a6f52ed
 SHA512:
-  metadata.gz: 0c8a5d07fa468eb3a275844b667ac07be8c8d5dd5878518ceb649fe0743a977a3b7b00335cb6cb885bc2d90c0d86695bbbdf3cee9eb34d0303fe2eddaeb9901e
-  data.tar.gz: a45636029c6e2888067299a8e9900f533ca6959eb79d005ca8e44187de359ca42af68d350ccbc39fa890bb285b179694479ca4812bbc795d9b3427fae6be68d8
+  metadata.gz: b003f28202908c38acf93ab895835bae013d893da184c451df9e2b3ca98ee70d8c8c8dcf4f7497abe1dc62cf00760414a607a036282d4af9675111ac8b73ba40
+  data.tar.gz: 605032f390401d9aaf2de1a33d54a6555ddb6b56c274c594836889a36eeb4bcd21d9299a4342398b7b012b53f812806bf73ac7710b24916a8b8d9de4dc82973c

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to this project are documented in this file.
 
+## 5.1.1
+
+One migration: create `spree_paypal_platform_orders`. Removed the leftover
+checkout-table / STI rewrite / rename migrations from 5.1.0.
+
 ## 5.1.0
 
 Full internal rename to `Spree::PaypalPlatform`.

data/README.md

@@ -70,26 +70,6 @@ https://<store>/api/v3/webhooks/payments/<prefixed-payment-method-id>
    `Spree::PaypalPlatform::PaymentSources::ApplePay`.
 4. Safari + HTTPS only.
 
-## Coming from `spree_paypal_checkout`
-
-Existing `spree_payment_methods.type` rows may still store
-`SpreePaypalCheckout::Gateway` or `Spree::PaypalCheckout::Gateway`. The
-install generator copies a rewrite migration that updates those STI strings
-to `Spree::PaypalPlatform::Gateway` and renames
-`spree_paypal_checkout_orders` → `spree_paypal_platform_orders`.
-
-```ruby
-# before
-gem 'spree_paypal_checkout', github: 'aypex-io/spree_paypal_checkout', branch: 'tongkat-fitness'
-
-# after
-gem 'spree-paypal_platform'
-```
-
-Storefront `resolveGatewayId` should map `paypal_platform` /
-`Spree::PaypalPlatform::Gateway`. Keep the old `paypal_checkout` /
-`SpreePaypalCheckout::Gateway` keys until the STI rewrite has run.
-
 ## Developing
 
 ```bash

data/db/migrate/{20250528095719_create_spree_paypal_checkout_orders.rb → 20250528095719_create_spree_paypal_platform_orders.rb}

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-class CreateSpreePaypalCheckoutOrders < ActiveRecord::Migration[7.2]
+class CreateSpreePaypalPlatformOrders < ActiveRecord::Migration[7.2]
   def change
-    create_table :spree_paypal_checkout_orders do |t|
+    create_table :spree_paypal_platform_orders do |t|
       t.references :order, null: false
       t.references :payment_method, null: false
       t.string :paypal_id, null: false
@@ -17,6 +17,7 @@ class CreateSpreePaypalCheckoutOrders < ActiveRecord::Migration[7.2]
       t.timestamps
     end
 
-    add_index :spree_paypal_checkout_orders, %i[order_id paypal_id], unique: true
+    add_index :spree_paypal_platform_orders, %i[order_id paypal_id], unique: true
+    add_index :spree_paypal_platform_orders, :paypal_id, unique: true
   end
 end

data/lib/spree/paypal_platform/version.rb

@@ -3,6 +3,6 @@
 module Spree
   module PaypalPlatform
     # Major version tracks Spree's major version: 5.x supports Spree 5.x.
-    VERSION = '5.1.0'
+    VERSION = '5.1.1'
   end
 end

data/lib/spree/paypal_platform.rb

@@ -23,20 +23,10 @@ module Spree
     ##
     # STI type names that count as this gem's gateway.
     #
-    # Includes historical strings from the official `spree_paypal_checkout`
-    # gem and the short-lived `Spree::PaypalCheckout` name so existing
-    # payment-method rows keep matching before the rewrite migration runs.
-    #
     # @return [Array<String>]
     #
     def self.gateway_type_names
-      (
-        [
-          Gateway.name,
-          'Spree::PaypalCheckout::Gateway',
-          'SpreePaypalCheckout::Gateway'
-        ] + Gateway.descendants.map(&:name)
-      ).uniq
+      ([Gateway.name] + Gateway.descendants.map(&:name)).uniq
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spree-paypal_platform
 version: !ruby/object:Gem::Version
-  version: 5.1.0
+  version: 5.1.1
 platform: ruby
 authors:
 - Aypex
@@ -139,10 +139,7 @@ files:
 - app/views/spree/admin/payments/source_forms/_spree_paypal_platform.html.erb
 - app/views/spree/payment_sources/_paypal_platform.html.erb
 - config/initializers/spree.rb
-- db/migrate/20250528095719_create_spree_paypal_checkout_orders.rb
-- db/migrate/20260817120000_rewrite_spree_paypal_checkout_sti_types.rb
-- db/migrate/20260817130000_add_unique_index_on_paypal_checkout_order_paypal_id.rb
-- db/migrate/20260817140000_rename_paypal_checkout_orders_to_paypal_platform.rb
+- db/migrate/20250528095719_create_spree_paypal_platform_orders.rb
 - lib/generators/spree/paypal_platform/install/install_generator.rb
 - lib/spree-paypal_platform.rb
 - lib/spree/paypal_platform.rb

data/db/migrate/20260817120000_rewrite_spree_paypal_checkout_sti_types.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-class RewriteSpreePaypalCheckoutStiTypes < ActiveRecord::Migration[7.2]
-  # Hosts coming from the official / forked `spree_paypal_platform` gem store
-  # `SpreePaypalCheckout::Gateway` (and `…::PaymentSources::Paypal`) in STI
-  # type columns. The new gem aliases those constants, so checkout keeps
-  # working before this migration runs; rewriting the strings makes the
-  # alias optional for the next deploy.
-  LEGACY_TO_CURRENT = {
-    'SpreePaypalCheckout::Gateway' => 'Spree::PaypalPlatform::Gateway',
-    'SpreePaypalCheckout::PaymentSources::Paypal' => 'Spree::PaypalPlatform::PaymentSources::Paypal',
-    'SpreePaypalCheckout::Order' => 'Spree::PaypalPlatform::Order'
-  }.freeze
-
-  def up
-    rewrite_column(:spree_payment_methods, :type)
-    rewrite_column(:spree_payment_sources, :type) if column_exists?(:spree_payment_sources, :type)
-  end
-
-  def down
-    rewrite_column(:spree_payment_methods, :type, LEGACY_TO_CURRENT.invert)
-    rewrite_column(:spree_payment_sources, :type, LEGACY_TO_CURRENT.invert) if column_exists?(:spree_payment_sources, :type)
-  end
-
-  private
-
-  def rewrite_column(table, column, mapping = LEGACY_TO_CURRENT)
-    mapping.each do |from, to|
-      reversible_update = "UPDATE #{table} SET #{column} = #{quote(to)} WHERE #{column} = #{quote(from)}"
-      execute reversible_update
-    end
-  end
-end

data/db/migrate/20260817130000_add_unique_index_on_paypal_checkout_order_paypal_id.rb

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-class AddUniqueIndexOnPaypalCheckoutOrderPaypalId < ActiveRecord::Migration[7.2]
-  def change
-    add_index :spree_paypal_checkout_orders, :paypal_id, unique: true
-  end
-end

data/db/migrate/20260817140000_rename_paypal_checkout_orders_to_paypal_platform.rb

@@ -1,48 +0,0 @@
-# frozen_string_literal: true
-
-class RenamePaypalCheckoutOrdersToPaypalPlatform < ActiveRecord::Migration[7.2]
-  LEGACY_TO_CURRENT = {
-    'SpreePaypalCheckout::Gateway' => 'Spree::PaypalPlatform::Gateway',
-    'SpreePaypalCheckout::PaymentSources::Paypal' => 'Spree::PaypalPlatform::PaymentSources::Paypal',
-    'SpreePaypalCheckout::Order' => 'Spree::PaypalPlatform::Order',
-    'Spree::PaypalCheckout::Gateway' => 'Spree::PaypalPlatform::Gateway',
-    'Spree::PaypalCheckout::PaymentSources::Paypal' => 'Spree::PaypalPlatform::PaymentSources::Paypal',
-    'Spree::PaypalCheckout::PaymentSources::ApplePay' => 'Spree::PaypalPlatform::PaymentSources::ApplePay',
-    'Spree::PaypalCheckout::PaymentSources::Card' => 'Spree::PaypalPlatform::PaymentSources::Card',
-    'Spree::PaypalCheckout::Order' => 'Spree::PaypalPlatform::Order'
-  }.freeze
-
-  def up
-    if table_exists?(:spree_paypal_checkout_orders) && !table_exists?(:spree_paypal_platform_orders)
-      rename_table :spree_paypal_checkout_orders, :spree_paypal_platform_orders
-    end
-
-    rewrite_column(:spree_payment_methods, :type)
-    rewrite_column(:spree_payment_sources, :type) if column_exists?(:spree_payment_sources, :type)
-  end
-
-  def down
-    if table_exists?(:spree_paypal_platform_orders) && !table_exists?(:spree_paypal_checkout_orders)
-      rename_table :spree_paypal_platform_orders, :spree_paypal_checkout_orders
-    end
-
-    rewrite_column(:spree_payment_methods, :type, invert_mapping)
-    rewrite_column(:spree_payment_sources, :type, invert_mapping) if column_exists?(:spree_payment_sources, :type)
-  end
-
-  private
-
-  def invert_mapping
-    {
-      'Spree::PaypalPlatform::Gateway' => 'SpreePaypalCheckout::Gateway',
-      'Spree::PaypalPlatform::PaymentSources::Paypal' => 'SpreePaypalCheckout::PaymentSources::Paypal',
-      'Spree::PaypalPlatform::Order' => 'SpreePaypalCheckout::Order'
-    }
-  end
-
-  def rewrite_column(table, column, mapping = LEGACY_TO_CURRENT)
-    mapping.each do |from, to|
-      execute "UPDATE #{table} SET #{column} = #{quote(to)} WHERE #{column} = #{quote(from)}"
-    end
-  end
-end