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

[Junyor]

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:   Junyor
 Status:       patch (code needs review)

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




Junyor



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.