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

[robertDouglass]

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

I have to add that I would rather have this patch than nothing, since as
Uwe correctly points out, the behavior is hardcoded and not suitable for
many common cases.




robertDouglass



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?




------------------------------------------------------------------------

Sun, 21 Aug 2005 08:59:41 +0000 : Bèr Kessels

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)




------------------------------------------------------------------------

Sun, 21 Aug 2005 09:09:28 +0000 : robertDouglass

-1 for this approach and a big +1 for running aggregators through
filters that can be configured.