Pass security name to security info provider APIs

Pass the security name to the security info provider APIs to improve
logging messages.

src/investir/findata/dataprovider.py

@@ -14,11 +14,11 @@ class DataProviderError(Exception):
 
 class SecurityInfoProvider(Protocol):
     def get_info(
-        self, isin: ISIN, refresh_date: datetime | None = None
+        self, isin: ISIN, name: str = "", refresh_date: datetime | None = None
     ) -> SecurityInfo:
         pass
 
-    def get_price(self, isin: ISIN) -> Money:
+    def get_price(self, isin: ISIN, name: str = "") -> Money:
         pass
 
 

src/investir/findata/financialdata.py

@@ -25,20 +25,20 @@ def __init__(
         self._live_rates_provider = live_rates_provider
 
     def get_security_info(
-        self, isin: ISIN, refresh_date: datetime | None = None
+        self, isin: ISIN, name: str = "", refresh_date: datetime | None = None
     ) -> SecurityInfo:
         if self._security_info_provider is not None:
             try:
-                return self._security_info_provider.get_info(isin, refresh_date)
+                return self._security_info_provider.get_info(isin, name, refresh_date)
             except DataProviderError as ex:
                 logger.warning(str(ex))
 
-        return SecurityInfo()
+        return SecurityInfo(name, [])
 
-    def get_security_price(self, isin) -> Money | None:
+    def get_security_price(self, isin, name: str = "") -> Money | None:
         if self._security_info_provider is not None:
             try:
-                return self._security_info_provider.get_price(isin)
+                return self._security_info_provider.get_price(isin, name)
             except DataProviderError as ex:
                 logger.warning(str(ex))
 

src/investir/findata/yahoofinanceprovider.py

@@ -24,15 +24,15 @@ def __init__(self, cache_file: Path | None = None) -> None:
         self._prices: dict[ISIN, Money] = {}
 
     def get_info(
-        self, isin: ISIN, refresh_date: datetime | None = None
+        self, isin: ISIN, name: str = "", refresh_date: datetime | None = None
     ) -> SecurityInfo:
         self._load_cache()
 
         if info := self._infos.get(isin):
             if refresh_date is None or info.last_updated >= refresh_date:
                 return info
 
-        logger.info("Fetching information for %s", isin)
+        logger.info("Fetching information for %s - %s", isin, name)
 
         try:
             yf_data = yfinance.Ticker(isin)
@@ -51,10 +51,12 @@ def get_info(
 
         return self._infos[isin]
 
-    def get_price(self, isin: ISIN) -> Money:
+    def get_price(self, isin: ISIN, name: str = "") -> Money:
         if cached_price := self._prices.get(isin):
             return cached_price
 
+        logger.info("Fetching last price for %s - %s", isin, name)
+
         try:
             yf_data = yfinance.Ticker(isin)
             price = Decimal(yf_data.info["currentPrice"])

src/investir/taxcalculator.py

@@ -358,7 +358,7 @@ def _validate_orders(self) -> None:
     def _normalise_orders(self, orders: Sequence[Order]) -> Sequence[Order]:
         return [
             o.adjust_quantity(
-                self._findata.get_security_info(o.isin, o.timestamp).splits
+                self._findata.get_security_info(o.isin, o.name, o.timestamp).splits
             )
             for o in orders
         ]
@@ -502,7 +502,8 @@ def _process_section104_disposals(self, isin: ISIN) -> None:
     def _get_holding_value(
         self, isin: ISIN, holding: Section104Holding
     ) -> Decimal | None:
-        if (price := self._findata.get_security_price(isin)) and (
+        security_name = self._tr_hist.get_security_name(isin) or ""
+        if (price := self._findata.get_security_price(isin, security_name)) and (
             price_base_currency := self._findata.convert_money(price, BASE_CURRENCY)
         ):
             return holding.quantity * price_base_currency.amount