Porting seems streightforward... For the basics it just needs the .info file (attached). Put it in an own folder with both files (.module & .info). (i.e. /modules/commentrss/)

I guess this is just a provisional fix, none of the .module code has been changed.

Cheers,
Phil

CommentFileSizeAuthor
#1 commentrss-50.patch.txt2.95 KBEgon Bianchet
commentrss.info97 bytesfiliptc

Comments

Egon Bianchet’s picture

Egon Bianchet commented
Status: Needs work » Needs review
StatusFileSize
new commentrss-50.patch.txt2.95 KB

Some changes to t() placeholders

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs review » Postponed (maintainer needs more info)

Thanks for the patches to both of you. I have committed them, although needed to fix some stuff along the way:

filip_tc:
- improper usage of version field
- module name is "Comment RSS"

Egon Bianchet:
- multiple check_plain()-ing of some values
- still missing check_plain() on site name

Plus:
- the node type API was changed, so needed a different function

I have branched the new code for Drupal 5.0 and added a daily snapshot release (http://drupal.org/node/103979). Please check that out, and test at least one of every possible comment rss type (whole site, node, term, vocabulary, node type). If everything works, I am willing to tag and release a stable tarball.

Thanks for your contributions!

add1sun’s picture

add1sun commented
Status: Postponed (maintainer needs more info) » Fixed

Since the dev version for 5 exists and this has been sitting for well over a year, I think it is safe to close this now.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.