Duplicate array key `Code` in `_WP_Editors::get_translation()`

[justlevine]

This is a bug introduced in #38061, where the new Code key added to _WP_Editors::get_translation() on Line 1399 overwrites the existing Code registered on Line 1168.

 Line    wp-includes/class-wp-editor.php                                 
 ------- ---------------------------------------------------------------- 
  :1168   Array has 2 duplicate keys with value 'Code' ('Code', 'Code').  
          🪪  array.duplicateKey       

---

Caught as part of #61175 (​PHPStan baseline)

[sabernhardt]

The simplest fix might be to restore 'Text' as the array key (but leaving the translation).

• base.js : text( window.tinymce.translate( 'Text' ) )
• class-wp-editor.php : 'Text' => _x( 'Code', 'Name for the Code editor tab (formerly Text)' ),

One way to test the script is with a Text widget (using the Classic Widgets plugin).

[sabernhardt]

restoring 'Text' array key

[joedolson]

I think we should follow the same model used elsewhere in the array for distinguishing between common keys, e.g. Edit vs. Edit|button. Perhaps Code|editor?

[joedolson]

Alternate with Code|tab

[joedolson]

Added an alternate that uses Code|tab, which better describes the use case.

[audrasjb]

I think the alternate looks good to me, but I'm wondering wether we should fix this in 6.8 or in 6.8.1. What is the potential issue for end users: user facing strings conflicts?

[swissspidy]

From what I can see, most if not all locales translate the two strings the same anyway, so it doesn't really matter in that regard. This can be fixed in 6.8.1 or even 6.9

[audrasjb]

Let's move this ticket to milestone 6.8.1 then.

[sabernhardt]

Before : TinyMCE Code button with untranslated "Code" tooltip

[sabernhardt]

I found the first 'Code' string in the TinyMCE Code button. Core does not activate that in a Classic block or with Classic Editor, but it can be added with a plugin such as Advanced Editor Tools.

• If the newer string is not translated in the user's language, the tooltip shows "Code" (in English).
• Now the tooltip does not give the keyboard shortcut in any language.

​5 out of 38 translations do not match the previous "Code" string perfectly, but the differences seem minor (to someone who cannot read them).

The "Text" could have one small advantage: if someone replaced the old string using the wp_mce_translation filter, the replacement would continue to work. However, I did not find any ​directory plugins that edited the "Text" tab, and most plugins in the search results just add strings for their own elements (very few replace a string).

[sabernhardt]

The "Text" could have one small advantage...

If a site replaced "Text" with the wp_mce_translation filter, updating to 6.8 would already change the key and string. That decreases any value that restoring "Text" might have had.

[jorbin]

https://core.trac.wordpress.org/attachment/ticket/63269/63269.2.diff looks good to me for 6.8.1

[jorbin]

In 60182:

Editor: Use different keys in array of translatable strings.

[59696] changed the 'Text' tab of the classic editor to 'Code' but Code was already used as a key in the array of translatable text. Since arrays keys need to be unique, this meant that it is possible for the wrong translation to appear in a locale. Using different keys fixes that.

Props joedolson, sabernhardt, justlevine, swissspidy, audrasjb.
Fixes #63269. See #38061.

[jorbin]

Reopening for 2nd committer review and backport consideration

[sabernhardt]

in trunk, the two different Vietnamese strings display correctly: "Mã" for the Code tab, and "Mã code" plus the keyboard shortcut for the TinyMCE Code button tooltip

[joedolson]

Looks good for backport.

[joedolson]

In 60187:

Editor: Use different keys in array of translatable strings.

[59696] changed the 'Text' tab of the classic editor to 'Code' but Code was already used as a key in the array of translatable text. Since arrays keys need to be unique, this meant that it is possible for the wrong translation to appear in a locale. Using different keys fixes that.

Reviewed by joedolson.
Merges [60182] to the 6.8 branch.

Props joedolson, sabernhardt, justlevine, swissspidy, audrasjb.
Fixes #63269. See #38061.