Ticket #3574 (closed defect (bug): fixed)

Opened 5 years ago

Last modified 5 years ago

Character set not specified in TinyMCE files

Reported by: ruckus Owned by: anonymous
Priority: normal Milestone: 2.0.9
Component: Administration Version: 2.0.7
Severity: major Keywords: has-patch
Cc: nbachiyski, ruckus

Description

Using 8-bit characters in TinyMCE translation does not work, because the output from tiny_mce_gzip.php and wp-mce-help.php does not specify any charset in Content-Type. This leaves it up to PHP and the web server to complete the Content-Type charset information, possibly incorrectly.

The charset should not be hard-coded in these scripts, so that plugins such as Character Set Conversions[1] can use the gettext hook to transcode the messages to the character set specified in the blog settings.

[1]: http:/kimmo.suominen.com/sw/charsets/

Attachments

tinymce-charset.diff Download (1.2 KB) - added by ruckus 5 years ago.
Patch against 2.0.7-RC1

Change History

ruckus5 years ago

Patch against 2.0.7-RC1

  • Milestone changed from 2.0.8 to 2.2

Does this apply to 2.1 or 2.2? If so, we should fix it there first.

  • Cc nbachiyski added
  • Keywords has-patch added; bg|has-patch removed

comment:3   ryan5 years ago

  • Status changed from new to closed
  • Resolution set to fixed

(In [4845]) Set charset for MCE. Props nbachiyski. fixes #3574

comment:4   ryan5 years ago

(In [4846]) Set charset for MCE. Props nbachiyski. fixes #3574

comment:5   ryan5 years ago

  • Milestone changed from 2.2 to 2.0.9

Committed for 2.2 and 2.1.1. Let's milestone for 2.0.9.

  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Cc ruckus added

Looks like the change was applied to 2.1 and 2.2, so it applied. I'm not running the trunk (yet), so my patches tend to be against 2.0.x.

comment:9   ryan5 years ago

  • Status changed from reopened to closed
  • Resolution set to fixed

(In [4888]) Set charset for MCE. Props ruckus. fixes #3574

Note: See TracTickets for help on using tickets.