WordPress.org

Make WordPress Core

#23723 closed defect (bug) (fixed)

Don't force '(time_diff) ago' sequence in the post revision information

Reported by: vanillalounge Owned by: ryan
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: I18N Keywords: has-patch commit
Focuses: Cc:

Description

Some languages (Portuguese, for one) express this the other way around, i.e. '(string) (time_diff)'.

Example: '3 days ago' translates to 'há 3 dias' ('ago' is 'há')

http://core.trac.wordpress.org/browser/trunk/wp-includes/post-template.php#L1313

Attachments (1)

23723.patch (2.8 KB) - added by johnbillion 13 months ago.

Download all attachments as: .zip

Change History (10)

comment:1 SergeyBiryukov13 months ago

  • Milestone changed from Awaiting Review to 3.6
  • Version set to trunk

Introduced in [23506].

comment:2 toscho13 months ago

  • Cc info@… added

The whole string should be translatable. The placeholders need numbers, and the string should get a context parameter explaining what they are for.

In German it would be: Vor drei Tagen (date).

comment:3 johnbillion13 months ago

  • Keywords has-patch added

Patch which adds context and a translator comment.

comment:4 SergeyBiryukov13 months ago

  • Keywords commit added

johnbillion13 months ago

comment:6 johnbillion13 months ago

Updated the patch to add in the other missing i18n.

comment:7 SergeyBiryukov13 months ago

Looks good.

The context in line 1493 is probably not necessary, since the string seems self-explanatory and is not used anywhere else. Wouldn't hurt though, I guess.

comment:8 johnbillion13 months ago

Yeah I thought it'd be better to have it than not.

comment:9 ryan13 months ago

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

In 23743:

Fix i18n for revision diff strings displaying human time.

Props johnbillion
fixes #23723

Note: See TracTickets for help on using tickets.