Closed (fixed)
Project:
Recent Changes
Version:
6.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
30 Dec 2008 at 21:26 UTC
Updated:
17 Feb 2010 at 22:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
fuzzy_texan commentedIs this module being maintained?
The RSS feed is half of the functionality.
Comment #2
toemaz commentedOn line 296
has to be changed into
Comment #3
toemaz commentedFind it attached. I deployed it on a production website. Lets see how it works out.
Comment #4
fuzzy_texan commentedThanks for the work on the patch toemaz.
I applied the patch using the normal patch command. Still getting the same error.
Also tried manually editing the file as described in #2. No luck there either.
Any suggestions of what I could be doing wrong?
Comment #5
toemaz commentedDid you empty the cache?
Go to: admin/settings/performance
Second, perhaps your RSS reader has a cache layer as well?
Comment #6
fuzzy_texan commentedThanks for the suggestions. It turned out to be just a permissions issue. I hadn't set the new access recent changes permission in 6.x yet. Setting that made the feed work for both the dev version, and the patch you submitted.
So both work. What does your patch do exactly? If it's presenting the feed in a better / more correct way, it could still be worth committing.
Comment #7
toemaz commentedStrange. The dev version didn't work out for me. Well, I guess more people will hit the bug if it's really out there. Lets wait and see.
Comment #8
leop commentedI had the same problem, applied #2, which solved it.
Comment #9
leop commentedA similar error occurs with
theme('username'...calls in this module.theme_usernameyields only relative paths, not absolute paths that are needed for RSS feeds. I am looking for a solution.Comment #10
leop commentedHm, seems #9 shouldn't be a problem, since
xml:baseis correctly outputted in the feed. Still I feel that theming shouldn't be applied to RSS output. Therefore I propose the following change in line 475:into
Comment #11
toemaz commentedBetter creating a new issue for this. Since you tested to patch from #3, I consider it tested by the community.
Comment #12
toemaz commentedCommitted http://drupal.org/cvs?commit=323944