Ticket #3456 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

Add do_action('commentsrss2_item') to wp-commentsrss2.php

Reported by: seek3r Assigned to: markjaquith
Priority: normal Milestone: 2.1
Component: General Version: 2.1
Severity: normal Keywords:
Cc:

Description

This would allow for comment enclosures. I use this for my support of audio comments in wordpress, and then the comment feed has the enclosure and becomes its own little podcast.

So right above the closing item tag

<?php do_action('commentsrss2_item'); ?>

</item>

a 2 second fix from anyone with svn access, Im even attaching the latest version of the file with that one line added

Attachments

wp-commentsrss2.php (3.7 kB) - added by seek3r on 12/08/06 09:48:51.
wp-commentsrss2.php with requested line added
wp-commentsrss2.patch (377 bytes) - added by Viper007Bond on 12/08/06 12:04:04.
Patch of mentioned changed

Change History

12/08/06 09:48:51 changed by seek3r

  • attachment wp-commentsrss2.php added.

wp-commentsrss2.php with requested line added

12/08/06 12:04:04 changed by Viper007Bond

  • attachment wp-commentsrss2.patch added.

Patch of mentioned changed

12/08/06 18:32:39 changed by foolswisdom

  • version set to 2.1.

12/09/06 06:36:20 changed by markjaquith

  • owner changed from anonymous to markjaquith.
  • status changed from new to assigned.

+1, but I'm no RSS expert... objections? Better way to implement? Redundant?

12/09/06 09:39:15 changed by Viper007Bond

Looks fine to me. It'll allow one to add additional data for posts.

However, I think it'd be best if the ID of the post was passed via the hook so you don't have to do global and such.

12/10/06 00:11:29 changed by markjaquith

Good call, Viper007Bond. I'm also going to pass the comment_post_ID as a 3rd param, in case someone wants to have it without having to grab the whole comment object.

12/10/06 00:13:01 changed by markjaquith

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [4635]) Add commentrss2_item action hook. Props seek3r and Viper007Bond. fixes #3456