atom.module breaks with the newest cvs because of the deprecation of check_output. the attached patch fixes this issue plus a minor fix for not including an absolute url when referring to itself and using drupal_set_header() rather than header().

thanks,

alastair

CommentFileSizeAuthor
drupal-atom.module.patch.txt2.43 KBliquidx

Comments

deekayen’s picture

deekayen commented
Status: Active » Needs work

I changed check_output() to check_markup(), but I don't understand what this is for:

+
+    if (node_hook($item, 'view')) {
+      node_invoke($item, 'view', TRUE, FALSE);
+    }
+    else {
+      $item = node_prepare($item, TRUE);
+    }
+
+    // Allow modules to change $node->teaser before viewing.
+    node_invoke_nodeapi($item, 'view', true, false);
+
deekayen’s picture

deekayen commented
Status: Needs work » Fixed

I accidentally committed the code I asked about, and it's not breaking my feed reader for DRUPAL-4-6 or DRUPAL-4-7, so I guess I'll just close this.

dries’s picture

dries commented
Status: Fixed » Closed (fixed)