Make WordPress Core

Opened 9 years ago

Closed 8 years ago

#3733 closed defect (bug) (fixed)

Some string in TinyMCE can't be translated

Reported by: yskins Owned by:
Milestone: 2.3 Priority: low
Severity: normal Version: 2.1
Component: I18N Keywords: i18n
Focuses: Cc:


Line 47 in wp-includes/js/tinymce/tiny_mce_gzip.php has been replaced from

$search2 = "/ : (['\"])(.*)\\1/Uem";


$search2 = "/\\s:\\s(['\"])(.*)\\1(,|\\s*})/Uem";

in http://trac.wordpress.org/changeset/3664#file87 .

So some string in wp-includes\js\tinymce\plugins\wordpress\langs\en.js can't be translated, such as

justifyleft_desc : 'Align left (' + metaKey + '+f)',

In WordPress 2.0.7, I can translate “Align left (” in my po file, but in WordPress 2.1 it does not work.

Change History (8)

#1 @foolswisdom
9 years ago

  • Cc andy nbachiyski@… added

#2 @yskins
9 years ago

And the new regular expression has another mistake. In wp-includes\js\tinymce\plugins\paste\langs\en.js, the last two line are

selectall_desc : 'Select All'

Because of the "m" pattern modifier which means multiline mode, it does not match "(,|
s*})", so it can't be translate too.

Just delete "(,|
s*})", the two bug will be killed.

Sorry for my poor english, hoop you can understand.

#3 @Nazgul
9 years ago

  • Milestone changed from 2.1.1 to 2.1.2

#4 @rob1n
9 years ago

  • Component changed from i18n to TinyMCE

#5 @rob1n
9 years ago

  • Component changed from TinyMCE to i18n

#6 @foolswisdom
9 years ago

  • Milestone changed from 2.1.3 to 2.2

#7 @foolswisdom
9 years ago

  • Milestone changed from 2.2 to 2.3

#8 @ryan
8 years ago

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

The code that does this is gone. Resolving as fixed.

Note: See TracTickets for help on using tickets.