Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#24229 closed defect (bug) (fixed)

Something is probably missing in post-template.php:1424

Reported by: settle Owned by: SergeyBiryukov
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: I18N Keywords: has-patch
Focuses: Cc:

Description (last modified by ocean90)

It seems %2$s should also be here:

	/* translators: post revision: 1: when, 2: author name */
	$titlef = _x( '%1$s', 'post revision' );

Attachments (2)

24229.diff (478 bytes) - added by DrewAPicture 3 years ago.
24229.2.diff (714 bytes) - added by DrewAPicture 3 years ago.

Download all attachments as: .zip

Change History (10)

#1 @settle
3 years ago

  • Version set to trunk

#2 @ocean90
3 years ago

  • Description modified (diff)
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 3.6

Since [23769].

Related: #24213

3 years ago

#3 @DrewAPicture
3 years ago

If this was just a case of forgetting to remove the second translator note, 24229.diff takes care of that.

I'm confused about whether this ticket calls for adding $name back or fixing the translator comment.

#4 follow-up: @settle
3 years ago

If only %1$s stays - there is no point to "translate" it at all.

3 years ago

#5 in reply to: ↑ 4 @DrewAPicture
3 years ago

Replying to settle:

If only %1$s stays - there is no point to "translate" it at all.

Good call, wasn't thinking clearly. 24229.2.diff

#6 @SergeyBiryukov
3 years ago

  • Keywords has-patch added; needs-patch removed

#7 @SergeyBiryukov
3 years ago

Introduced in [23769].

#8 @SergeyBiryukov
3 years ago

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

In 24150:

Don't translate a single placeholder. Remove unused variables. props settle, DrewAPicture. fixes #24229.

Note: See TracTickets for help on using tickets.