Twenty Ten i18n - translated strings not visible

[pavelevap]

New ticket for partial problem in http://core.trac.wordpress.org/ticket/14063

There are several strings in .pot file which were translated, but translations not displayed, for example "The 2010 default theme for WordPress." and "the WordPress team".

You can see following discussion here: ​http://wppolyglots.wordpress.com/2010/11/27/there-are-some-strings-in-twenty-ten-1-3/

(In [17072]) Handle 'This gallery contains X photos' properly in Twenty Ten. fixes #15858.

[17072] was meant for #15859.

Should we reuse _get_plugin_data_markup_translate() or create a separate function?

Bump. It was problem for Twenty Ten and now it is also problem for Twenty Eleven. Many translators translate long (important and descriptive) strings, but they do not work. Any chance to solve this for 3.2?

Also "Showcase Template" and "Sidebar Template" are not translatable (and not in .pot file). And it is not consistent because "Default Template" is translated in core. So users can see translation of "Default Template" and then not translated template names from Twenty Eleven.

In get_page_templates(), we would possibly run the string through the theme textdomain.

We don't currently have get_theme_data looking for a textdomain header -- I imagine this means we don't support translated headers for themes the way we do for plugins.

These both sound like 3.3 opportunities.

OK, I expected that it is probably too late for 3.2, but this problem is very serious from the point of language packs. Users without English knowledge will not understand for example very informative description about Showcase template and other possibilities of Twenty Eleven.

Related: #19597

15858.diff​ is the beginnings of a fix for #15858 and #6007. I started by adding TextDomain and DomainPath, then fixed up _get_plugin_data_markup_translate() and added a corresponding function for themes. Page templates were translated.

Then it went downhill. (Or down a rabbit hole, you might say.)

1. get_file_description() is passed an absolute path to a file. Figuring out what theme it belongs to (to ascertain the textdomain) is harder than one might expect.

2. get_themes() should burn in hell and will likely give me night terrors for the rest of the week. Seriously. I struggled with this mess two years ago during GSoC, and returning to it now makes me want to stab my eyes with a pencil.

Other than that, things are peachy.

15858.2.diff​ isn't an improvement. Decided to take a swing at translating the data of get_themes(). You'd think this would work (given that it is not cached), but haha, you'd be wrong. A few places, mainly the theme editor, rely on $theme_data['Name'] to uniquely identify a theme.

The patch does do something else: It takes the list of tags on themes.php and translates them, which is nice.

In [20029]:

Introduce WP_Theme, wp_get_themes(), and wp_get_theme() to replace get_themes(), get_theme(), get_theme_data(), current_theme_info(), and others.

• Getters and Helpers: Introduces a series of methods to allow for easy generation of headers for display, and other theme metadata, including page templates.
• Screenshots: Handles support for multiple screenshots. (see # Additional screenshots must be PNG and start with screenshot-2.png, and be sequential to be counted. see #19816.
• Error Handling: Broken themes have a WP_Error object attached to them.
• Caching: Introduces a wp_cache_themes_persistently filter (also in [20020]) to enable persistent caching of all filesystem and sanitization operations normally handled by WP_Theme (and formerly get_file_data() and get_themes()). Themes are cached individually and across five different cache keys for different data pieces.
• Compatibility: A WP_Theme object is backwards compatible with a theme's array formerly returned by get_themes() and get_theme(), and an stdClass object formerly returned by current_theme_info().
• i18n/L10n: Theme headers are now localizable with proper Text Domain and Domain Path headers, like plugins. (Language packs may remove the requirement for headers.) For page templates, see #6007 (not fixed yet, but will be easy now). For headers, fixes #15858.
• PHP and CSS files: New methods that fetch a list of theme files (for the theme editor) only on demand, rather than only loading them into memory. fixes #11214.

Functions deprecated:

• get_themes(), get_allowed_themes() and get_broken_themes() -- use wp_get_themes()
• get_theme() and current_theme_info() -- use wp_get_theme()
• get_site_allowed_themes() -- use WP_Theme::get_allowed_on_network()
• wpmu_get_blog_allowedthemes() -- use WP_theme::get_allowed_on_site()

see also [20016], [20018], [20019], [20020], [20021], [20022], [20025], [20026], [20027]. also fixes #19244.

see #20103.

Simply awesome!

I tried to add headers and localization works well. Nice. Do not forget to add headers to all default themes (if needed with Language packs).