WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 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:

Description

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

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

to

$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)

comment:1 foolswisdom7 years ago

  • Cc andy nbachiyski@… added

comment:2 yskins7 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.

comment:3 Nazgul7 years ago

  • Milestone changed from 2.1.1 to 2.1.2

comment:4 rob1n7 years ago

  • Component changed from i18n to TinyMCE

comment:5 rob1n7 years ago

  • Component changed from TinyMCE to i18n

comment:6 foolswisdom7 years ago

  • Milestone changed from 2.1.3 to 2.2

comment:7 foolswisdom7 years ago

  • Milestone changed from 2.2 to 2.3

comment:8 ryan7 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.