Improve HoldingCalculator Security Preloading

- Include securities from both trades and current holdings
- Ensure unique securities are preloaded
- Enhance logging for preloaded securities
This commit is contained in:
Josh Pigford
2025-02-04 11:02:50 -06:00
parent 9c846e7de4
commit bdec61f312

View File

@@ -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|