From bdec61f312db14b7d1b19d24be6874a9c4f1ecfb Mon Sep 17 00:00:00 2001 From: Josh Pigford Date: Tue, 4 Feb 2025 11:02:50 -0600 Subject: [PATCH] Improve HoldingCalculator Security Preloading - Include securities from both trades and current holdings - Ensure unique securities are preloaded - Enhance logging for preloaded securities --- app/models/account/holding_calculator.rb | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/app/models/account/holding_calculator.rb b/app/models/account/holding_calculator.rb index b82ad47a..c9327daa 100644 --- a/app/models/account/holding_calculator.rb +++ b/app/models/account/holding_calculator.rb @@ -117,7 +117,11 @@ class Account::HoldingCalculator end def preload_securities + # Get securities from trades and current holdings securities = trades.map(&:entryable).map(&:security).uniq + securities += account.holdings.where(date: Date.current).map(&:security) + securities.uniq! + Rails.logger.info "[HoldingCalculator] Preloading #{securities.size} securities for account #{account.id}" securities.each do |security|