From 618d597f6df2742c048760b1e19ddad5f93ce5c0 Mon Sep 17 00:00:00 2001 From: Gaetan Craig-Riou Date: Wed, 9 Jul 2025 13:33:29 +1000 Subject: [PATCH] ScopeVariantToHub require an Enterprise object The hub parameter is used to check if inventory is enabled, so it breaks if we just pass an id --- lib/reporting/reports/orders_and_fulfillment/base.rb | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/lib/reporting/reports/orders_and_fulfillment/base.rb b/lib/reporting/reports/orders_and_fulfillment/base.rb index b0bcff9511..209d8739df 100644 --- a/lib/reporting/reports/orders_and_fulfillment/base.rb +++ b/lib/reporting/reports/orders_and_fulfillment/base.rb @@ -81,16 +81,14 @@ module Reporting def variant_scoper_for(distributor_id) @variant_scopers_by_distributor_id ||= {} variant_overrides = {} - if OpenFoodNetwork::FeatureToggle.enabled?(:inventory, - Enterprise.find_by(id: distributor_id)) + distributor = Enterprise.find_by(id: distributor_id) + + if OpenFoodNetwork::FeatureToggle.enabled?(:inventory, distributor) variant_overrides = report_variant_overrides[distributor_id] end @variant_scopers_by_distributor_id[distributor_id] ||= - OpenFoodNetwork::ScopeVariantToHub.new( - distributor_id, - variant_overrides, - ) + OpenFoodNetwork::ScopeVariantToHub.new(distributor, variant_overrides) end def not_all_have_unit?(line_items)