By webslinger on

Hello!

I've noticed that images do not appear on my "News Aggregator" page, even though I know that many of the feeds I'm displaying send image tags. I've checked my filters, and the IMG tag is allowed. Is there something I can do to allow this?

Thanks,
Dieter

Comments

mantis8’s picture

mantis8 commented

I'm looking to find a way to add an image to my rss feed also. I'm just trying to add it at the top of the page. Any help is appreciated.

Thanks in advance.

jvincher’s picture

jvincher commented

I'm experiencing the same issue as webslinger in 4.5.2. Anyone have any suggestions?

jasonwhat’s picture

jasonwhat commented

Any help or plans with this? There is an old discussion here http://civicspacelabs.org/home/node/1816#comment-1063 on this topic. I'd like to see this option as well.

boris mann’s picture

boris mann commented

Currently, many HTML tags, including < img >, are filtered out. Richard has a patch which lets you set which tags are filtered or not, and then you can enable images to display.

I can't find the patch right now, will come back later with an update.

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented

The patch is at http://drupal.org/node/14104 It's in the wrong 'issue' (which was a feature request to get the aggregator hooked into input formats), really, but it's there. Note that I'm 100% sure this patch works when you manually click "update items" next to an individual feed, but I think it could use some more extensive testing when used with cron. Also, the patch I submitted is against the 4.5 version of the module.

(Username formerly my full name, Richard Eriksson.)

jasonwhat’s picture

jasonwhat commented

If you don't want to patch this (I'm not sure if the referenced patch works in 4.6 or CVS anyways) you can allow images in all feeds by adding "img" as allowable html in your aggregator. Here is how the fixed version looks:

// TODO: Make handling of aggregated HTML more flexible/configurable.
      $value = decode_entities(trim($value));
      $value = strip_tags($value, '<a> <img> <b> <br> <dd> <dl> <dt> <em> <i> <li> <ol> <p> <strong> <u> <ul>');

The only difference from the original is adding in <img>. If there are other tags you want to allow, this is where you'd do it.

aliak’s picture

aliak commented

thanks jasonwhat, I had this problem also since upgrading to 4.6.3 and it's working well now after implementing your change. much appreciated!

zach harkey’s picture

zach harkey commented

What are the chances of this becoming a preference in the aggregator settings page? (Just like it is in the default Filtered HTML input format)

It would be very useful to have the ability to specify which tags your feed should filter without having to hack a core module.

-zach
------------------------
harkey design

: z

kvarnelis’s picture

kvarnelis commented

i'll second that!

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented

This is currently a feature in the latest version of Drupal (4.6.3). Click administer » settings » aggregator and you should see a text box to configure HTML settings. Just add <img> to the list and the aggregator will show images.

(Username formerly my full name, Richard Eriksson.)

kvarnelis’s picture

kvarnelis commented

excellent!

jivyb’s picture

jivyb commented

on the admin>>settings>>aggregator page, I don't see a textbox. Am I missing it somehow? These are the options I get:

>Items shown in sources and categories pages:none3 items5 items10 items15 items20 items25 items

The number of items which will be shown with each feed or category in the feed and category summary pages.

>Discard news items older than:
1 hour3 hours6 hours9 hours12 hours1 day2 days3 days1 week2 weeks4 weeks8 weeks16 weeks

>Category selection type:
checkboxes
multiple selector

The type of category selection widget which is shown on categorization pages. Checkboxes are easier to use; a multiple selector is good for working with large numbers of categories.

sillygwailo’s picture

sillygwailo
Location Toronto, ON
commented

Ah, it appears I misspoke. It won't be available until 4.7. Bryght sites (disclosure: I work for Bryght) that are running on 4.6 have this feature, but the feature did not make it into the official 4.6 release.

(Username formerly my full name, Richard Eriksson.)

jivyb’s picture

jivyb commented

hi. I can't get images to display in the aggregator for 4.6.5. I changed the code in 4.6.3 as described above, but the line for it isn't there in 4.6.5.

Any suggestions?

sepeck’s picture

sepeck commented

http://www.blkmtn.org/node/269

-sp
---------
Test site, always start with a test site.
Drupal Best Practices Guide -|- Black Mountain

-Steven Peck
---------
Test site, always start with a test site.
Drupal Best Practices Guide