Opened 11 years ago
Last modified 6 years ago
#29621 new defect (bug)
Export post title with markup
| Reported by: |
|
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_rssthinking 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