Fix logging and remove cruft.

[kiosk.git] / seattletimes_rss_renderer.py

diff --git a/seattletimes_rss_renderer.py b/seattletimes_rss_renderer.py
index 7b39bca4a974ac3e82ee2b9528abec7d4eb6989a..653c74ac4c7e984f153cf189ca82b07623a1c4e0 100644 (file)
--- a/seattletimes_rss_renderer.py
+++ b/seattletimes_rss_renderer.py
@@ -7,7 +7,7 @@ import xml
 import generic_news_rss_renderer as gnrss
 
 
-logger = logging.getLogger(__file__)
+logger = logging.getLogger(__name__)
 
 
 class seattletimes_rss_renderer(gnrss.generic_news_rss_renderer):
@@ -38,9 +38,7 @@ class seattletimes_rss_renderer(gnrss.generic_news_rss_renderer):
         feed_uris: List[str],
         page_title: str,
     ):
-        super().__init__(
-            name_to_timeout_dict, feed_site, feed_uris, page_title
-        )
+        super().__init__(name_to_timeout_dict, feed_site, feed_uris, page_title)
 
     def get_headlines_page_prefix(self) -> str:
         return "seattletimes-nonnews"
@@ -58,25 +56,21 @@ class seattletimes_rss_renderer(gnrss.generic_news_rss_renderer):
             logger.debug(f'{title}: item.tag ({item}) isn\'t "item"?!')
             return False
         if self.is_item_older_than_n_days(item, 14):
-            logger.info(f'{title}: too old to be interesting.')
+            logger.info(f"{title}: too old to be interesting.")
             return False
 
         details = {}
         for detail in list(item):
-            logger.debug(
-                f'detail {detail.tag} => {detail.attrib} ({detail.text})'
-            )
+            logger.debug(f"detail {detail.tag} => {detail.attrib} ({detail.text})")
             if detail.text is not None:
                 details[detail.tag] = detail.text
-        if 'category' not in details:
-            logger.debug(f'{title}: no category in details?')
+        if "category" not in details:
+            logger.debug(f"{title}: no category in details?")
             return False
         interesting = False
         for x in seattletimes_rss_renderer.interesting_categories:
             if x in details["category"]:
-                logger.debug(
-                    f'{x} looks like a good category.'
-                )
+                logger.debug(f"{x} looks like a good category.")
                 interesting = True
                 break
         return interesting
@@ -85,17 +79,15 @@ class seattletimes_rss_renderer(gnrss.generic_news_rss_renderer):
         self, title: str, description: str, item: xml.etree.ElementTree.Element
     ) -> bool:
         if self.is_item_older_than_n_days(item, 14):
-            logger.info(
-                f'{title}: is too old to be interesting.'
-            )
+            logger.info(f"{title}: is too old to be interesting.")
             return False
         return len(description) >= 65
 
 
 # Test
-#x = seattletimes_rss_renderer({"Test", 123},
+# x = seattletimes_rss_renderer({"Test", 123},
 #                              "www.seattletimes.com",
 #                              [ "/outdoors/feed/", '/live/feed/' ],
 #                              "nonnews")
-#x.periodic_render("Fetch News")
-#x.periodic_render("Shuffle News")
+# x.periodic_render("Fetch News")
+# x.periodic_render("Shuffle News")