Opened 10 years ago
Last modified 5 years ago
#29621 new defect (bug)
Export post title with markup
Reported by: | GaryJ | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | Export | Keywords: | has-patch |
Focuses: | Cc: |
Description
The Theme Unit Test data includes a post Title With Markup. However, the current exporter filters the post titles with the_title_rss
, which has strip_tags
hooked to it, so it always needs fixing up after importing.
The content and excerpt blocks of the export are uniquely filtered with the_content_export
and the_excerpt_export
, and not the_content_rss
and the_excerpt_rss
.
Attached is a patch which changes the exporter from using the_title_rss
to a new the_title_export
filter, and wraps the output in CDATA tags, as per content and export blocks.
Advantages:
- Unique filter for handling post titles during export.
- Fixes stripped tags issue.
- Filter name that is consistent with other filters in that context.
- Use of
wxr_cdata()
for consistency with other blocks of output in that context.
Disadvantage:
- Backwards-incompatibility - applying a filter to
the_title_rss
thinking it would also apply to exports would no longer work.
There are no un-skipped unit tests in place for testing the exporter so I've not included any here.
Attachments (2)
Change History (4)
Note: See
TracTickets for help on using
tickets.
Change exporter from the_title_rss filter to the_title_export