Ticket #3733 (closed defect: fixed)

Opened 2 years ago

Last modified 1 year ago

Some string in TinyMCE can't be translated

Reported by: yskins Assigned to: anonymous
Priority: low Milestone: 2.3
Component: i18n Version: 2.1
Severity: normal Keywords: i18n
Cc: andy, nbachiyski@developer.bg

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

02/02/07 17:11:24 changed by foolswisdom

  • cc set to andy, nbachiyski@developer.bg.

02/04/07 03:17:19 changed by yskins

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.

02/21/07 16:07:46 changed by Nazgul

  • milestone changed from 2.1.1 to 2.1.2.

02/22/07 07:03:14 changed by rob1n

  • component changed from i18n to TinyMCE.

02/22/07 07:03:38 changed by rob1n

  • component changed from TinyMCE to i18n.

03/28/07 02:27:49 changed by foolswisdom

  • milestone changed from 2.1.3 to 2.2.

04/10/07 17:59:56 changed by foolswisdom

  • milestone changed from 2.2 to 2.3.

09/05/07 22:37:44 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

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