Set isPermaLink="no" for guids in rss feeds.
authorJoey Hess <joey@kitenet.net>
Tue, 14 Sep 2010 16:40:40 +0000 (12:40 -0400)
committerJoey Hess <joey@kitenet.net>
Tue, 14 Sep 2010 16:40:40 +0000 (12:40 -0400)
The rss spec says that unless the attribute is set, guid elements *are*
permalinks. The problem with that is that if [[meta permalink=]] is used,
as is done with aggregated posts, that goes into the link element, and
apparently some rss readers prefer the not-really-permalink in the guid
element when linking to the post.

Without meta permalink, the link and guid elements have the same content,
so it should be ok, in that case too for the guid to not be a permalink.

(Checked and this does not flood aggregators.)

debian/changelog
templates/rssitem.tmpl

index 3428aa6..8b127b5 100644 (file)
@@ -17,6 +17,7 @@ ikiwiki (3.20100832) UNRELEASED; urgency=low
     information earlier than the delete hook.
   * actiontabs: Improve tab padding.
   * blueview: Fix display of links to translated pages in the page header.
+  * Set isPermaLink="no" for guids in rss feeds. 
 
  -- Joey Hess <joeyh@debian.org>  Tue, 07 Sep 2010 12:08:05 -0400
 
index a64a4eb..272b340 100644 (file)
@@ -8,7 +8,7 @@
 <TMPL_IF GUID>
        <guid isPermaLink="no"><TMPL_VAR GUID></guid>
 <TMPL_ELSE>
-       <guid><TMPL_VAR URL></guid>
+       <guid isPermaLink="no"><TMPL_VAR URL></guid>
 </TMPL_IF>
        <link><TMPL_VAR PERMALINK></link>
 <TMPL_IF CATEGORIES>