From 8830949edd4bc92ba89da9f847b36b5d96d0a22d Mon Sep 17 00:00:00 2001 From: d0nni3q84 <62199227+d0nni3q84@users.noreply.github.com> Date: Wed, 8 Jun 2022 11:57:34 +0000 Subject: [PATCH] Revert "Feedreader: Properly support Atom feeds that use only the `updated` date format and resolve #73207." This reverts commit 4dbd11ee04b4e8f935a22dfb51405b7bdaaba676. --- .../components/feedreader/__init__.py | 18 +++--------------- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/homeassistant/components/feedreader/__init__.py b/homeassistant/components/feedreader/__init__.py index 2de742d5aa2..11a3d4b0498 100644 --- a/homeassistant/components/feedreader/__init__.py +++ b/homeassistant/components/feedreader/__init__.py @@ -70,7 +70,6 @@ class FeedManager: self._last_entry_timestamp = None self._last_update_successful = False self._has_published_parsed = False - self._has_updated_parsed = False self._event_type = EVENT_FEEDREADER self._feed_id = url hass.bus.listen_once(EVENT_HOMEASSISTANT_START, lambda _: self._update()) @@ -123,7 +122,7 @@ class FeedManager: ) self._filter_entries() self._publish_new_entries() - if self._has_published_parsed or self._has_updated_parsed: + if self._has_published_parsed: self._storage.put_timestamp( self._feed_id, self._last_entry_timestamp ) @@ -144,7 +143,7 @@ class FeedManager: def _update_and_fire_entry(self, entry): """Update last_entry_timestamp and fire entry.""" - # Check if the entry has a published or updated date. + # Check if the entry has a published date. if "published_parsed" in entry and entry.published_parsed: # We are lucky, `published_parsed` data available, let's make use of # it to publish only new available entries since the last run @@ -152,17 +151,9 @@ class FeedManager: self._last_entry_timestamp = max( entry.published_parsed, self._last_entry_timestamp ) - elif "updated_parsed" in entry and entry.updated_parsed: - # We are lucky, `updated_parsed` data available, let's make use of - # it to publish only new available entries since the last run - self._has_updated_parsed = True - self._last_entry_timestamp = max( - entry.updated_parsed, self._last_entry_timestamp - ) else: self._has_published_parsed = False - self._has_updated_parsed = False - _LOGGER.debug("No published_parsed or updated_parsed info available for entry %s", entry) + _LOGGER.debug("No published_parsed info available for entry %s", entry) entry.update({"feed_url": self._url}) self._hass.bus.fire(self._event_type, entry) @@ -179,9 +170,6 @@ class FeedManager: if self._firstrun or ( "published_parsed" in entry and entry.published_parsed > self._last_entry_timestamp - ) or ( - "updated_parsed" in entry - and entry.updated_parsed > self._last_entry_timestamp ): self._update_and_fire_entry(entry) new_entries = True