Hi,
I recently updated to Drupal 5 and Inline 5, and the inline images are working fine in HTML, but are not visible in my RSS feed. I'm using Google Reader to view the RSS feed. My site is: http://www.curdnerds.com. Can anyone here help me troubleshoot this?
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | inline.zip | 4.37 KB | NoRandom |
| #8 | inline-DRUPAL-5.patch | 649 bytes | sun |
Comments
Comment #1
faunapolis commentedHi, I just noticed that your inline images show in your feed, and they even show when added to the iGoogle page. Can you share your solution? Thanks very much.
Comment #2
jaharmi commentedI am also interested in having this work. If there's a way to do it with the existing Inline module, I'd love to know how. Otherwise, I'm hoping the module can be enhanced to support this in the future.
I've had readers of my site complain about the lack of images in my feed, and I didn't even realize the pictures were missing. All that appears is the [inline:] tag.
Comment #3
TomArah commentedYes please add RSS support - the automatic thumbnailing for teasers is a great feature so it's a pity they don't currently make it through to the feed.
Alternatively - ideally additionally - is there a way to strip the [inline] code from the feed?
Comment #4
sunSorry, I'm not very used to generation of RSS feeds and have to study that topic first.
So if anyone is able to help out here, I'll be happy to help review and commit a patch.
Comment #5
sunI just checked a RSS feed from a site that uses Inline and images are showing up like they should. Are you using private files method? If so, you might try http://drupal.org/node/192586
Comment #6
aries commentedHi,
The images are not present in my rss.xml too. Check it: http://paca.sablonet.hu/rss.xml
Comment #7
aries commentedI've found a fix: you should put the folloving into the inline.module's
inline_nodeapifunction, around the line 235:case 'rss item':
$node->teaser = _inline_substitute_tags($node, 'teaser');
return;
That solves the problem. Sorry for this non-patch solution, but seems to trivial to do a patch for this. :)
Comment #8
sunPlease test attached patch and report back if it works for you.
Comment #9
slavikt commentedI have tested the attached patch and can confirm this is work for me.
Comment #10
sunCommitted, thanks!
Comment #11
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #12
NoRandom commentedI had problems applying the patch and I had to do it manually. If someone has the same problem here is the patched module: