ba_spree_loyalty_points 1.0.0 → 1.0.1

data/app/views/spree/checkout/payment/_loyaltypoints.html.erb

@@ -1,8 +1,8 @@
-<%- lp_balance = spree_current_user.loyalty_points_balance %>
-<%- equivalent_currency_balance = spree_current_user.loyalty_points_equivalent_currency %>
+<%- lp_balance = spree_current_user&.loyalty_points_balance %>
+<%- equivalent_currency_balance = spree_current_user&.loyalty_points_equivalent_currency %>
 Loyalty Points Balance: <%= lp_balance %><%= "($" + equivalent_currency_balance.to_s + ")" %>
 <%- lp_needed = @order.loyalty_points_for(@order.total, 'redeem') %>
 <br/>Loyalty Points Needed: <%= lp_needed %>
-<% if lp_needed > lp_balance %>
+<% if lp_balance.nil? || lp_needed > lp_balance %>
   <br/><b>Insufficient Balance!</b>
 <% end %>

data/app/views/spree/loyalty_points/_transaction_table.html.erb

@@ -1,4 +1,4 @@
-<%= paginate @loyalty_points_transactions %>
+<%= paginate @loyalty_points_transactions if @loyalty_points_transactions.respond_to?(:total_pages) %>
 
 <% if @loyalty_points_transactions.present? %>
   <table class="loyalty-points-summary table table-striped table-hover">
@@ -41,4 +41,4 @@
   <p><%= Spree.t(:you_have_not_earned_any_loyalty_points_yet) %></p>
 <% end %>
 
-<%= paginate @loyalty_points_transactions %>
+<%= paginate @loyalty_points_transactions if @loyalty_points_transactions.respond_to?(:total_pages) %>

data/db/migrate/20140116090042_add_loyalty_points_balance_to_spree_user.rb

@@ -1,4 +1,4 @@
-class AddLoyaltyPointsBalanceToSpreeUser < ActiveRecord::Migration[7.1]
+class AddLoyaltyPointsBalanceToSpreeUser < ActiveRecord::Migration[8.0]
   def change
     users_table_name = Spree.user_class.present? ? Spree.user_class.table_name : :spree_users
     add_column users_table_name, :loyalty_points_balance, :integer, default: 0, null: false

data/db/migrate/20140116110437_create_loyalty_points_transactions.rb

@@ -1,4 +1,4 @@
-class CreateLoyaltyPointsTransactions < ActiveRecord::Migration[7.1]
+class CreateLoyaltyPointsTransactions < ActiveRecord::Migration[8.0]
   def change
     create_table :spree_loyalty_points_transactions do |t|
       t.integer :loyalty_points

data/db/migrate/20140117062720_add_timestamps_to_loyalty_points_transaction.rb

@@ -1,4 +1,4 @@
-class AddTimestampsToLoyaltyPointsTransaction < ActiveRecord::Migration[7.1]
+class AddTimestampsToLoyaltyPointsTransaction < ActiveRecord::Migration[8.0]
   def change
     change_table :spree_loyalty_points_transactions do |t|
       t.timestamps

data/db/migrate/20140117065314_add_fields_to_spree_loyalty_points_transaction.rb

@@ -1,4 +1,4 @@
-class AddFieldsToSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[7.1]
+class AddFieldsToSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[8.0]
   def change
     add_column :spree_loyalty_points_transactions, :source_type, :string
     rename_column :spree_loyalty_points_transactions, :order_id, :source_id

data/db/migrate/20140122084005_add_comment_to_spree_loyalty_points_transaction.rb

@@ -1,4 +1,4 @@
-class AddCommentToSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[7.1]
+class AddCommentToSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[8.0]
   def change
     add_column :spree_loyalty_points_transactions, :comment, :string
   end

data/db/migrate/20140123092709_add_paid_at_to_spree_order.rb

@@ -1,4 +1,4 @@
-class AddPaidAtToSpreeOrder < ActiveRecord::Migration[7.1]
+class AddPaidAtToSpreeOrder < ActiveRecord::Migration[8.0]
   def change
     add_column :spree_orders, :paid_at, :datetime
   end

data/db/migrate/20140123120355_add_loyalty_points_to_spree_return_authorization.rb

@@ -1,4 +1,4 @@
-class AddLoyaltyPointsToSpreeReturnAuthorization < ActiveRecord::Migration[7.1]
+class AddLoyaltyPointsToSpreeReturnAuthorization < ActiveRecord::Migration[8.0]
   def change
     add_column :spree_return_authorizations, :loyalty_points, :integer, default: 0, null: false
     add_column :spree_return_authorizations, :loyalty_points_transaction_type, :string

data/db/migrate/20140124121728_add_loyalty_points_awarded_to_spree_order.rb

@@ -1,4 +1,4 @@
-class AddLoyaltyPointsAwardedToSpreeOrder < ActiveRecord::Migration[7.1]
+class AddLoyaltyPointsAwardedToSpreeOrder < ActiveRecord::Migration[8.0]
   def change
     add_column :spree_orders, :loyalty_points_awarded, :boolean, default: false, null: false
   end

data/db/migrate/20140130131957_rename_fields_in_spree_loyalty_points_transaction.rb

@@ -1,4 +1,4 @@
-class RenameFieldsInSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[7.1]
+class RenameFieldsInSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[8.0]
   def change
     rename_column :spree_loyalty_points_transactions, :updated_balance, :balance
     rename_column :spree_loyalty_points_transactions, :transaction_type, :type

data/db/migrate/20140131072702_remove_loyalty_points_awarded_from_spree_order.rb

@@ -1,4 +1,4 @@
-class RemoveLoyaltyPointsAwardedFromSpreeOrder < ActiveRecord::Migration[7.1]
+class RemoveLoyaltyPointsAwardedFromSpreeOrder < ActiveRecord::Migration[8.0]
   def change
     remove_column :spree_orders, :loyalty_points_awarded, :boolean, default: false, null: false
   end

data/db/migrate/20140203063433_add_indexing_to_spree_loyalty_points_transaction.rb

@@ -1,4 +1,4 @@
-class AddIndexingToSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[7.1]
+class AddIndexingToSpreeLoyaltyPointsTransaction < ActiveRecord::Migration[8.0]
   def change
     add_index :spree_loyalty_points_transactions, [:source_type, :source_id], name: 'by_source'
     add_index :spree_loyalty_points_transactions, :user_id

data/db/migrate/20140205111553_add_transaction_id_to_spree_loyalty_points_transactions.rb

@@ -1,4 +1,4 @@
-class AddTransactionIdToSpreeLoyaltyPointsTransactions < ActiveRecord::Migration[7.1]
+class AddTransactionIdToSpreeLoyaltyPointsTransactions < ActiveRecord::Migration[8.0]
   def change
     add_column :spree_loyalty_points_transactions, :transaction_id, :string
   end

data/db/migrate/20140207055836_add_lock_version_to_spree_user.rb

@@ -1,4 +1,4 @@
-class AddLockVersionToSpreeUser < ActiveRecord::Migration[7.1]
+class AddLockVersionToSpreeUser < ActiveRecord::Migration[8.0]
   def change
     users_table_name = Spree.user_class.present? ? Spree.user_class.table_name : :spree_users
     add_column users_table_name, :lock_version, :integer, default: 0, null: false unless column_exists?(users_table_name, :lock_version)

data/db/migrate/20251210070500_remove_lock_version_from_spree_user.rb

@@ -0,0 +1,11 @@
+class RemoveLockVersionFromSpreeUser < ActiveRecord::Migration[8.0]
+  def change
+    # Remove lock_version column that was added by 20140207055836_add_lock_version_to_spree_user.rb
+    # This column is not actually used by the loyalty points engine and causes
+    # ActiveRecord::StaleObjectError when User is touched by related models
+    users_table_name = Spree.user_class.present? ? Spree.user_class.table_name : :spree_users
+    if table_exists?(users_table_name) && column_exists?(users_table_name, :lock_version)
+      remove_column users_table_name, :lock_version, :integer, default: 0, null: false
+    end
+  end
+end

data/lib/ba_spree_loyalty_points.rb

@@ -1,2 +1,3 @@
 require 'spree_core'
+require 'deface'
 require 'spree_loyalty_points/engine'

data/lib/spree_loyalty_points/engine.rb

@@ -15,6 +15,7 @@ module SpreeLoyaltyPoints
       Dir.glob(File.join(File.dirname(__FILE__), '../../app/**/*_decorator*.rb')) do |c|
         Rails.configuration.cache_classes ? require(c) : load(c)
       end
+      Rails.application.config.paths["app/views"].unshift([Spree::Storefront::Engine.root, 'app/views/themes/default'].join('/'))
 
       Spree.user_class.class_eval do
         validates :loyalty_points_balance, numericality: { only_integer: true, greater_than_or_equal_to: 0 }
@@ -26,7 +27,7 @@ module SpreeLoyaltyPoints
         end
 
         def loyalty_points_balance_sufficient?
-          loyalty_points_balance >= Spree::Store.default.loyalty_points_redeeming_balance
+          loyalty_points_balance >= Spree::Store.default.preferred_loyalty_points_redeeming_balance
         end
 
         def has_sufficient_loyalty_points?(order)
@@ -34,7 +35,7 @@ module SpreeLoyaltyPoints
         end
 
         def loyalty_points_equivalent_currency
-          loyalty_points_balance * Spree::Store.default.loyalty_points_conversion_rate
+          loyalty_points_balance * Spree::Store.default.preferred_loyalty_points_conversion_rate
         end
 
       end

data/lib/spree_loyalty_points/version.rb

@@ -1,3 +1,3 @@
 module SpreeLoyaltyPoints
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ba_spree_loyalty_points
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - be agile Co., Ltd.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-02-18 00:00:00.000000000 Z
+date: 2026-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree_core
@@ -16,42 +16,56 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.10.1
+        version: 5.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 4.10.1
+        version: 5.3.0
 - !ruby/object:Gem::Dependency
-  name: spree_backend
+  name: spree_admin
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: 4.8.0
+        version: 5.3.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: 4.8.0
+        version: 5.3.6
 - !ruby/object:Gem::Dependency
-  name: spree_frontend
+  name: spree_storefront
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 5.3.6
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 5.3.6
+- !ruby/object:Gem::Dependency
+  name: deface
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.8.0
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.8.0
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: capybara
   requirement: !ruby/object:Gem::Requirement
@@ -170,14 +184,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.0.0
+        version: 5.1.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.0.0
+        version: 5.1.1
 - !ruby/object:Gem::Dependency
   name: selenium-webdriver
   requirement: !ruby/object:Gem::Requirement
@@ -220,6 +234,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.3.13
+- !ruby/object:Gem::Dependency
+  name: rails-controller-testing
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.0.1
+- !ruby/object:Gem::Dependency
+  name: gem_override_marker
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: To award loyalty points and add loyalty points payment method to spree.  Compatible
   with Ruby 3.1, Rails 7.1 and Spree 4.10.
 email: develop@be-agile.jp
@@ -238,6 +280,7 @@ files:
 - app/controllers/spree/admin/return_authorizations_controller_decorator.rb
 - app/controllers/spree/checkout_controller_decorator.rb
 - app/controllers/spree/loyalty_points_controller.rb
+- app/controllers/spree_loyalty_points/checkout_controller_decorator.rb
 - app/models/concerns/spree/loyalty_points.rb
 - app/models/concerns/spree/order/loyalty_points.rb
 - app/models/concerns/spree/payment/loyalty_points.rb
@@ -251,9 +294,14 @@ files:
 - app/models/spree/payment_method_decorator.rb
 - app/models/spree/return_authorization_decorator.rb
 - app/models/spree/store_decorator.rb
+- app/models/spree_loyalty_points/order_decorator.rb
+- app/models/spree_loyalty_points/payment_decorator.rb
+- app/models/spree_loyalty_points/payment_method_decorator.rb
+- app/models/spree_loyalty_points/store_decorator.rb
 - app/overrides/add_condition_to_amount_on_return_authorization_page.rb
 - app/overrides/add_loyalty_points_balance_to_account_page.rb
 - app/overrides/add_loyalty_points_debiting_to_return_authorization_page.rb
+- app/overrides/add_loyalty_points_to_account_nav.rb
 - app/overrides/add_loyalty_points_to_admin_user_show_page.rb
 - app/overrides/add_loyalty_points_to_cart_page.rb
 - app/overrides/add_loyalty_points_to_order_checkout_page.rb
@@ -286,6 +334,7 @@ files:
 - db/migrate/20140203063433_add_indexing_to_spree_loyalty_points_transaction.rb
 - db/migrate/20140205111553_add_transaction_id_to_spree_loyalty_points_transactions.rb
 - db/migrate/20140207055836_add_lock_version_to_spree_user.rb
+- db/migrate/20251210070500_remove_lock_version_from_spree_user.rb
 - lib/ba_spree_loyalty_points.rb
 - lib/generators/spree_loyalty_points/install/install_generator.rb
 - lib/spree_loyalty_points/engine.rb
@@ -295,7 +344,6 @@ files:
 homepage: https://github.com/be-agile/ba_spree_loyalty_points
 licenses:
 - AGPL-3.0-or-later
-- BSD-3-Clause
 metadata: {}
 post_install_message:
 rdoc_options: []
@@ -313,7 +361,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements:
 - none
-rubygems_version: 3.2.33
+rubygems_version: 3.4.9
 signing_key:
 specification_version: 4
 summary: Add loyalty points to spree (be agile fork)