[drupal-devel] [feature] aggregator: make filtered HMTL tags configurable

Bèr Kessels drupal-devel at drupal.org
Sun Aug 21 08:59:43 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    aggregator.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Uwe Hermann
 Reported by:  Uwe Hermann
 Updated by:   Bèr Kessels
 Status:       patch (code needs review)

sorry. -1.
IMHO aggregator should be small, clean and lean. No options, features
or whatevers. (node aggregator does this for free, and much more. If
you want advanced aggregation, then please add your development to this
module, or to another -new- advanced aggregation module)

Bèr Kessels

Previous comments:

Sun, 21 Aug 2005 00:13:21 +0000 : Uwe Hermann

Attachment: http://drupal.org/files/issues/aggregator_allowed_html_tags.patch (2.34 KB)

I'm currently trying to get aggregator.module in shape for being used as
a planet-like software for a Planet Drupal [1]. Here's a patch which
allows the site admin to specify which HTML tags are stripped from
feeds (or not). This is hardcoded in aggregator.module right now, the
attached 2-line patch (for HEAD) makes it configurable.

I propose to add at least "<img> <font> <blockquote> <div> <span>
<code>" to the default list of allowed HTML tags, but I'll leave that
for another patch. Any objections?

[1] http://drupal.org/node/28194


Sun, 21 Aug 2005 03:18:30 +0000 : Prometheus6


I know it works because it's identical to what I did.


Sun, 21 Aug 2005 08:22:54 +0000 : Junyor

This has come up before: http://drupal.org/node/14104.  Could you
possibly use input filters instead of a setting for this?

More information about the drupal-devel mailing list