WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#24943 closed defect (bug) (fixed)

Twenty Thirteen - broken localization of custom header images

Reported by: pavelevap's profile pavelevap Owned by: nacin's profile nacin
Milestone: 3.7 Priority: normal
Severity: normal Version: 3.6
Component: Bundled Theme Keywords: has-patch commit
Focuses: Cc:

Description

Function twentythirteen_custom_header_setup() is probably called before twentythirteen_setup() and they are related to the same after_setup_theme action. So, twentythirteen textdomain is loaded after twentythirteen_custom_header_setup() and that is why localization of 3 strings (Star, Diamond and Circle) for custom header does not work.

See attached patch (works for me).

Attachments (1)

custom-header-localization.patch​ (516 bytes) - added by pavelevap 11 years ago.

Download all attachments as: .zip

Change History (7)

@pavelevap
11 years ago

#1 @obenland
11 years ago

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

#2 @SergeyBiryukov
11 years ago

Confirmed the bug and the fix.

#3 @obenland
11 years ago

Another reason I like to aggregate all includes at the end of the functions.php file.

Although setting a priority will always work and would not be depended on the chance of the callback being registered at the right time. :)

#4 @SergeyBiryukov
11 years ago

  • Keywords commit added

#5 @nacin
11 years ago

Priority is much more reliable than include order.

#6 @nacin
11 years ago

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

In 25129:

Twenty Thirteen: Ensure custom headers are set up after the localization is initialized.

props pavelevap.
fixes #24943.

Note: See TracTickets for help on using tickets.