WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#38791 closed defect (bug) (fixed)

REST API: Strings with placeholders are missing translators comments

Reported by: dimadin Owned by: rmccue
Priority: normal Milestone: 4.7
Component: I18N Version: 4.7
Severity: normal Keywords: has-patch
Cc: Focuses: rest-api

Description

There are new strings with a placeholder but that lack translators comment. In the patch I marked those that I saw.

Attachments (1)

38791.1.diff​ (8.2 KB ) - added by dimadin 10 years ago.

Download all attachments as: .zip

Change History (10)

@dimadin
10 years ago

#1 @rmccue
10 years ago

  • Keywords has-patch added
  • Milestone Awaiting Review → 4.7
  • Owner set to rmccue
  • Status new → reviewing

#2 @rmccue
10 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 39238:

REST API: Add translator comments to text with placeholders.

Props dimadin.
Fixes #38791.

#3 @rmccue
10 years ago

In 39239:

REST API: Move translator comments to preceding line.

Inline translator comments break POT file generation.

Props dd32.
See #38791.

#4 @SergeyBiryukov
10 years ago

In 39245:

REST API: After [39238] and [39239], move the remaining translator comments to preceding line.

See #38791.

#5 @ramiy
10 years ago

The messages should comply the rules in #34521.

#6 @SergeyBiryukov
10 years ago

In 39251:

REST API: Unify permission error messages.

Props ramiy.
See #38791, #34521.

#7 @SergeyBiryukov
10 years ago

In 39252:

REST API: Remove two duplicate strings, use the ones we already have.

See #38791.

#8 @SergeyBiryukov
10 years ago

In 39264:

REST API: Uppercase 'ID' in endpoint descriptions and error messages for consistency with other strings.

See #38791.

#9 @SergeyBiryukov
10 years ago

In 39266:

REST API: After [39252] and [39264], uppercase some more 'ID' references in translatable strings.

See #38791.

Note: See TracTickets for help on using tickets.