X-Git-Url: https://wannabe.guru.org/gitweb/?a=blobdiff_plain;f=wsj_rss_renderer.py;h=15eee9f88e1a1e6b633d8817d2adcea1e72d31d7;hb=addd4980077f6e3857c5c035b49784dc3ceca49a;hp=a8ccf29c871e54dd4c09a915e0796e539710ed4e;hpb=0e2601682b4629eb425f2d20a3c4ca3d592d3cbe;p=kiosk.git

diff --git a/wsj_rss_renderer.py b/wsj_rss_renderer.py
index a8ccf29..15eee9f 100644
--- a/wsj_rss_renderer.py
+++ b/wsj_rss_renderer.py
@@ -1,54 +1,61 @@
-import generic_news_rss_renderer
-
-class wsj_rss_renderer(generic_news_rss_renderer.generic_news_rss_renderer):
-    def __init__(self, name_to_timeout_dict, feed_site, feed_uris, page_title):
-        super(wsj_rss_renderer, self).__init__(
-            name_to_timeout_dict,
-            feed_site,
-            feed_uris,
-            page_title)
-        self.debug = 1
-
-    def debug_prefix(self):
-        return "wsj(%s)" % (self.page_title)
-
-    def get_headlines_page_prefix(self):
-        return "wsj-%s" % (self.page_title)
-
-    def get_details_page_prefix(self):
-        return "wsj-details-%s" % (self.page_title)
-
-    def find_image(self, item):
-        image = item.findtext('image')
-        if image is not None:
-            url = image.get('url')
-            return url
+#!/usr/bin/env python3
+
+import xml
+from typing import Dict, List, Optional
+
+import generic_news_rss_renderer as gnrssr
+
+
+class wsj_rss_renderer(gnrssr.generic_news_rss_renderer):
+    def __init__(
+        self,
+        name_to_timeout_dict: Dict[str, int],
+        feed_site: str,
+        feed_uris: List[str],
+        page_title: str,
+    ):
+        super().__init__(
+            name_to_timeout_dict, feed_site, feed_uris, page_title
+        )
+        self.debug = True
+
+    def debug_prefix(self) -> str:
+        return f"wsj({self.page_title})"
+
+    def get_headlines_page_prefix(self) -> str:
+        return f"wsj-{self.page_title}"
+
+    def get_details_page_prefix(self) -> str:
+        return f"wsj-details-{self.page_title}"
+
+    def find_image(self, item: xml.etree.ElementTree.Element) -> Optional[str]:
         return None
 
-    def should_use_https(self):
+    def should_use_https(self) -> bool:
         return True
 
-    def item_is_interesting_for_headlines(self, title, description, item):
+    def item_is_interesting_for_headlines(
+        self, title: str, description: str, item: xml.etree.ElementTree.Element
+    ) -> bool:
         if self.is_item_older_than_n_days(item, 7):
-            self.debug_print("%s: is too old!" % title)
             return False
-        return ("WSJ.com" not in title and
-                "WSJ.com" not in description)
+        return "WSJ.com" not in title and "WSJ.com" not in description
 
-    def item_is_interesting_for_article(self, title, description, item):
+    def item_is_interesting_for_article(
+        self, title: str, description: str, item: xml.etree.ElementTree.Element
+    ) -> bool:
         if self.is_item_older_than_n_days(item, 7):
-            self.debug_print("%s: is too old!" % title)
             return False
-        return ("WSJ.com" not in title and
-                "WSJ.com" not in description)
+        return "WSJ.com" not in title and "WSJ.com" not in description
+
 
 # Test
-#x = wsj_rss_renderer(
+# x = wsj_rss_renderer(
 #    {"Fetch News" : 1,
 #     "Shuffle News" : 1},
 #    "feeds.a.dj.com",
 #    [ "/rss/RSSWorldNews.xml" ],
 #    "Test" )
-#if x.fetch_news() == 0:
-#    print "Error fetching news, no items fetched."
-#x.shuffle_news()
+# if x.fetch_news() == 0:
+#    print("Error fetching news, no items fetched.")
+# x.shuffle_news()