raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-03-01 02:03:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Eager-load the adjustment's adjustable

Browse Source 
Reduces the total migration time by ~15%. Requires including some more models, as order objects are now being eager-loaded as part of the polymorphic associations.
This commit is contained in:
Matt-Yorkley
2021-06-04 14:17:49 +01:00
parent 7ecd67a3fe
commit 7bcb0bcaa8
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
db/migrate/20210406161242_migrate_enterprise_fee_tax_amounts.rb
View File

@@ -2,6 +2,8 @@
# associations. The only workaround seems to be to load the class explicitly, which essentially
# skips the whole point of good_migrations... :/
require 'enterprise_fee'
require 'concerns/balance'
require 'spree/order'
class MigrateEnterpriseFeeTaxAmounts < ActiveRecord::Migration[5.0]
class Spree::Adjustment < ApplicationRecord
@@ -39,7 +41,9 @@ class MigrateEnterpriseFeeTaxAmounts < ActiveRecord::Migration[5.0]
end
def migrate_enterprise_fee_taxes!
Spree::Adjustment.enterprise_fee.where('included_tax <> 0').includes(:originator).find_each do |fee|
Spree::Adjustment.enterprise_fee.where('included_tax <> 0').
includes(:originator, :adjustable).find_each do |fee|
tax_category = tax_category_for(fee)
tax_rate = tax_rate_for(tax_category)