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) can we not use the default filter system here? 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? ------------------------------------------------------------------------ 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. ------------------------------------------------------------------------ Sun, 21 Aug 2005 09:10:44 +0000 : robertDouglass 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.