WordPress.org

Make WordPress Core

Opened 10 years ago

Last modified 2 years ago

#7813 new defect (bug)

export function does not preserve encoding

Reported by: tott Owned by: tott
Milestone: Future Release Priority: normal
Severity: minor Version: 2.7
Component: Export Keywords: needs-testing has-patch
Focuses: Cc:

Description

when exporting strings are always converted to utf-8 while header and encoding is set to encoding used in blog.

this causes trouble when importing later.

Attachments (1)

export.diff (782 bytes) - added by tott 10 years ago.
possible patch for encoding problem. needs testing

Download all attachments as: .zip

Change History (11)

@tott
10 years ago

possible patch for encoding problem. needs testing

#1 @lloydbudd
10 years ago

  • Cc lloydbudd added
  • Milestone set to 2.7
  • Version set to 2.7

#2 @lloydbudd
10 years ago

  • Keywords has-patch added

#3 follow-up: @westi
10 years ago

  • Keywords needs-patch needs-testing added; has-patch removed
  • Milestone changed from 2.7 to 2.8

Moving to 2.8 for now.

Needs a new patch and some testing before commit:

We can't rely on mb_ functions existing so we need a fallback for that.

#4 @Denis-de-Bernardy
9 years ago

  • Milestone changed from 2.8 to Future Release

#5 @Denis-de-Bernardy
9 years ago

  • Component changed from i18n to Export

#6 @Denis-de-Bernardy
9 years ago

  • Milestone changed from Future Release to 2.9
  • Severity changed from normal to minor

#7 @ryan
8 years ago

  • Milestone changed from 2.9 to Future Release

#8 @iseulde
5 years ago

  • Keywords export encoding i18n removed

#9 in reply to: ↑ 3 @GaryJ
4 years ago

Replying to westi:

We can't rely on mb_ functions existing so we need a fallback for that.

Several years on, and the only real fallback is to see if the function_exists before calling it. Still, that would hopefully catch a fair proportion of affected cases, wouldn't it?

#10 @chriscct7
2 years ago

  • Keywords has-patch added; needs-patch removed
Note: See TracTickets for help on using tickets.