rss_feed.py: sort items by modification time

[webber.git] / plugins / rss_feed.py

diff --git a/plugins/rss_feed.py b/plugins/rss_feed.py
index 79fdae9fd5de6b22fddb31336c1c3d1f41afeea3..1dbc11e25b874ffad8157b9cf3d561d7a2e22605 100644 (file)
--- a/plugins/rss_feed.py
+++ b/plugins/rss_feed.py
@@ -1,6 +1,6 @@
 # -*- coding: iso-8859-1 -*-
 from webber import *
-import os, datetime
+import os, datetime, re
 try:
        import PyRSS2Gen
 except ImportError:
@@ -52,13 +52,20 @@ def sitemap_scan(params):
                link = full_url,
                guid = PyRSS2Gen.Guid("%s %s" % (full_url, file["mtime"]), isPermaLink=0),
                description = change,
-               pubDate = datetime.datetime.fromtimestamp(file["mtime"], utc),
+               pubDate = file["mtime"]
        )
        items.append(item)
 
 
 @set_hook("finish")
 def finish(params):
+       # Sort items by pubDate, which still holds the mtime
+       items.sort(key=lambda i: i.pubDate, reverse=True)
+
+       # convert mtime to real pupDate
+       for i in items:
+               i.pubDate = datetime.datetime.fromtimestamp(i.pubDate, utc)
+
        rss = PyRSS2Gen.RSS2(
                title = cfg.subtitle,
                link = "http://%s" % cfg.main_url,
@@ -76,8 +83,10 @@ def finish(params):
                pass
        f = open(os.path.join(cfg.out_dir, cfg.rss_file), "w")
        # Ugly XML beautification
-       s = rss.to_xml().replace("<", "\n<").replace("\n\n", "\n")[1:]
+       s = rss.to_xml()
+       s = re.sub("<(?!/)", "\n<", s)
+       s = s.replace("\n\n", "\n")
        # Step two of self-reference
        s = s.replace('<channel>', '<channel>\n<atom:link href="http://%s/%s" rel="self" type="application/rss+xml" />' % (cfg.main_url, cfg.rss_file))
-       f.write(s)
+       f.write(s[1:])
        f.write("\n")