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

[Bèr Kessels]

Issue status update for 
http://drupal.org/node/29275
Post a follow up: 
http://drupal.org/project/comments/add/29275

 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

+1


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?