Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#26542 closed defect (bug) (fixed)

Page templates - localization does not work for 2014

Reported by: pavelevap Owned by: matt
Milestone: 3.8 Priority: highest omg bbq
Severity: blocker Version: 3.8
Component: Bundled Theme Keywords: has-patch commit
Focuses: Cc:


I am not sure what can be wrong, but even if I translated Page Template name (Full Width Page), name is still in English in related dropdown.

It works with 2012, but does not work with 2014.

Attachments (2)

26542.patch (917 bytes) - added by SergeyBiryukov 2 years ago.
es_ES.png (530.0 KB) - added by lancewillett 2 years ago.
Theme description working correctly with es_ES.mo

Download all attachments as: .zip

Change History (12)

#1 @SergeyBiryukov
2 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 3.8

Confirmed. Caused by missing Text Domain declaration in style.css.

#2 @pavelevap
2 years ago

Good catch, I thought that I am crazy :-)

#3 @nacin
2 years ago

  • Keywords commit added
  • Severity changed from normal to blocker

This is needed for translatable theme headers and theme language packs.

We should add a unit test to verify all themes have this header set.

#4 @lancewillett
2 years ago

I'll ping Otto to add this to Theme Check plugin blockers.

#5 @nacin
2 years ago

  • Priority changed from normal to highest omg bbq

This is without a doubt a huge blocker.

For anyone that is running non-English and updates WordPress to 3.8, they won't be able to read Twenty Fourteen's description.

#6 follow-up: @knutsp
2 years ago

I can confirm the issue and that the patch fixes it. But for the theme description, as nacin mentions, I can't get it translated to nb_NO no matter what I do (active/inactive, with or without the patch).

2 years ago

Theme description working correctly with es_ES.mo

#7 @lancewillett
2 years ago

With this patch applied I can see the theme description correctly translated in wp-admin/themes.php for currently active theme only, using `es_ES.mo` from GlotPress.

Last edited 2 years ago by lancewillett (previous) (diff)

#8 @matt
2 years ago

We decided in the dev chat this is kosher for 3.8.0.

#9 @matt
2 years ago

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

In 26852:

We forgot the text domain, which made translation break. Fixes #26542.

Props SergeyBiryukov and pavelevap

#10 in reply to: ↑ 6 @ocean90
2 years ago

Replying to knutsp:

There was a string change in [26850].

Note: See TracTickets for help on using tickets.