WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#24229 closed defect (bug) (fixed)

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

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

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 13 years ago.
24229.2.diff​ (714 bytes ) - added by DrewAPicture 13 years ago.

Download all attachments as: .zip

Change History (10)

#1 @settle
13 years ago

  • Version → trunk

#2 @ocean90
13 years ago

  • Description modified (diff)
  • Keywords needs-patch added
  • Milestone Awaiting Review → 3.6

Since [23769].

Related: #24213

@DrewAPicture
13 years ago

#3 @DrewAPicture
13 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
13 years ago

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

@DrewAPicture
13 years ago

#5 in reply to: ↑ 4 @DrewAPicture
13 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
13 years ago

  • Keywords has-patch added; needs-patch removed

#7 @SergeyBiryukov
13 years ago

Introduced in [23769].

#8 @SergeyBiryukov
13 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → 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.