Ticket #1785 (closed enhancement: fixed)

Opened 6 years ago

Last modified 5 years ago

Atom-based comment feed

Reported by: kurtmckee Owned by: rob1n
Priority: normal Milestone: 2.2
Component: General Version: 2.1
Severity: normal Keywords: has-patch needs-testing dev-feedback
Cc: rboren

Description

I've created an Atom 0.3 comment feed. Holizz (holizz.com) updated the code to be Atom 1.0 compliant. It is coded against WordPress 1.5.2.

Attachments

wp-commentsatom.php Download (3.7 KB) - added by kurtmckee 6 years ago.
wp-commentsatom.2.php Download (3.8 KB) - added by kurtmckee 6 years ago.
Fixed incorrect xml:base and id output
1785.diff Download (5.2 KB) - added by rob1n 5 years ago.

Change History

Fixed incorrect xml:base and id output

  • Keywords bg|has-patch added

comment:2   matt6 years ago

  • Priority changed from normal to low
  • Milestone changed from 1.6 to 2.1

comment:3   _ck_6 years ago

  • Priority changed from low to normal

Note for the wp-commentsatom.php to work correctly with rewritten urls in Wordpress 2.x this function has to be updated to the following in wp-includes/functions.php

function do_feed_atom($for_comments) {
if ( $for_comments ) {
load_template(ABSPATH . 'wp-commentsatom.php');
} else {
load_template(ABSPATH . 'wp-atom.php');
}
}

Wordpress is WAY overdue to have working ATOM 1.0 feeds and it's super easy to add into the next alpha release with all the groundword already done.

comment:4   matt5 years ago

  • Milestone changed from 2.1 to 2.2
  • Keywords dev-feedback added; atom comments feed bg|has-patch removed
  • Owner changed from anonymous to rob1n
  • Status changed from new to assigned

Is this something that the devs want in the core?

Also, the patch is outdated. I can work on a new one.

Waiting on feedback.

(Out-of-Trac feedback)

comment:8   ryan5 years ago

Add the template as wp-includes/feed-atom-comments.php.

Update do_feed_atom() to handle withcomments.

Update do_feed() to not assume comment feeds are always rss2.

  • Keywords dev-feedback removed
  • Version changed from 1.6 to 2.1

I'm on it. :)

rob1n5 years ago

  • Cc rboren added
  • Keywords has-patch needs-testing dev-feedback added

Added preliminary patch that NEEDS looking over. I *believe* I did everything right, but then again...

Okay, I found one error already: I use bloginfo_rss('content_type') instead of bloginfo_rss('html_type'). I'll fix this in the morning.

  • Status changed from assigned to closed
  • Resolution set to fixed

(In [4928]) Atom comments feed. First pass. Props kurtmckee and rob1n. fixes #1785

Note: See TracTickets for help on using tickets.