Ticket #5604 (closed defect (bug): duplicate)
the_excerpt should strip style tags
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | General | Version: | 2.5 |
| Severity: | normal | Keywords: | the_excerpt, flickr, style, has_patch, needs_testing |
| Cc: |
Description
the_excerpt prints <style> tags, rather than omitting them, causing unattractive excerpts.
example: http://archgfx.net/wordpress/?cat=1
the problem usually comes up in the context of posting from flickr.
Attachments
Change History
comment:1
sunburntkamel — 4 years ago
sunburntkamel — 4 years ago
-
attachment
formatting.php.5604.patch
added
patch wp_trim_excerpt to strip style and script tags completely
sunburntkamel — 4 years ago
-
attachment
formatting.php.5604.diff
added
patch wp_trim_excerpt to strip style and script tags completely
comment:2
sunburntkamel — 4 years ago
- Keywords has_patch added
- Component changed from Template to General
ignore the first patch, my tabs weren't set correctly.
comment:3
sunburntkamel — 4 years ago
- Keywords the_excerpt, flickr, style, added; the_excerpt flickr style removed
I think filtering out everything between style tags (and CDATA) is a good idea, but it should probably be done for the content_save_pre filter, as this can also be a problem when someone posts by email, for example.
comment:6
sunburntkamel — 4 years ago
I'm not sure that style/script tags should be removed from the posts entirely, as that might cause posts to not display properly.

it seems like wp_trim_excerpt might need something like this: http://us.php.net/manual/en/function.strip-tags.php#71962 but i'm really not qualified to make a patch.