Attempt to fix IndexError in Opower (#124478)

* Change the order of async_add_external_statistics in Opower * Use consumption_statistic_id instead of cost_statistic_id
home-assistant · Aug 30, 2024 · 910fb09 · 910fb09
1 parent cb742a6
commit 910fb09
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/homeassistant/components/opower/coordinator.py b/homeassistant/components/opower/coordinator.py
@@ -110,7 +110,7 @@ async def _insert_statistics(self) -> None:
             )
 
             last_stat = await get_instance(self.hass).async_add_executor_job(
-                get_last_statistics, self.hass, 1, cost_statistic_id, True, set()
+                get_last_statistics, self.hass, 1, consumption_statistic_id, True, set()
             )
             if not last_stat:
                 _LOGGER.debug("Updating statistic for the first time")
@@ -124,7 +124,7 @@ async def _insert_statistics(self) -> None:
                 cost_reads = await self._async_get_cost_reads(
                     account,
                     self.api.utility.timezone(),
-                    last_stat[cost_statistic_id][0]["start"],
+                    last_stat[consumption_statistic_id][0]["start"],
                 )
                 if not cost_reads:
                     _LOGGER.debug("No recent usage/cost data. Skipping update")
@@ -141,7 +141,7 @@ async def _insert_statistics(self) -> None:
                 )
                 cost_sum = cast(float, stats[cost_statistic_id][0]["sum"])
                 consumption_sum = cast(float, stats[consumption_statistic_id][0]["sum"])
-                last_stats_time = stats[cost_statistic_id][0]["start"]
+                last_stats_time = stats[consumption_statistic_id][0]["start"]
 
             cost_statistics = []
             consumption_statistics = []
@@ -187,7 +187,17 @@ async def _insert_statistics(self) -> None:
                 else UnitOfVolume.CENTUM_CUBIC_FEET,
             )
 
+            _LOGGER.debug(
+                "Adding %s statistics for %s",
+                len(cost_statistics),
+                cost_statistic_id,
+            )
             async_add_external_statistics(self.hass, cost_metadata, cost_statistics)
+            _LOGGER.debug(
+                "Adding %s statistics for %s",
+                len(consumption_statistics),
+                consumption_statistic_id,
+            )
             async_add_external_statistics(
                 self.hass, consumption_metadata, consumption_statistics
             )