Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#40939 closed defect (bug) (fixed)

Incorrect placeholder in translator comment in WP_Community_Events->get_events

Reported by: tfirdaus Owned by: SergeyBiryukov
Priority: normal Milestone: 4.9
Component: I18N Version: 4.8
Severity: minor Keywords: has-patch
Cc: Focuses: administration

Description

Shouldn’t be the type specifier in the translator comment %d instead of %s?

Attachments (2)

Screen Shot 2017-06-07 at 5.38.57 PM.png​ (111.4 KB ) - added by tfirdaus 9 years ago.
Preview in GlotPress
40939.diff​ (724 bytes ) - added by Soean 9 years ago.
40939.diff Replaces the %s with %d in the translator comment.

Download all attachments as: .zip

Change History (10)

@tfirdaus
9 years ago

Preview in GlotPress

This ticket was mentioned in ​Slack in #polyglots by tfirdaus. ​View the logs.


9 years ago

@Soean
9 years ago

40939.diff Replaces the %s with %d in the translator comment.

#2 @Soean
9 years ago

  • Keywords has-patch added
  • Severity normal → minor
  • Summary wp-admin/includes/class-wp-communit-events.php:111 Inconsistent type specifier → Incorrect placeholder in translator comment in WP_Community_Events->get_events

#3 @iandunn
9 years ago

40939.diff​ looks good to me.

#4 @SergeyBiryukov
9 years ago

  • Milestone Awaiting Review → 4.8.1

#5 @SergeyBiryukov
9 years ago

  • Owner set to SergeyBiryukov
  • Status new → accepted

#6 @swissspidy
9 years ago

#40944 was marked as a duplicate.

#7 @SergeyBiryukov
9 years ago

  • Milestone 4.8.1 → 4.9

#8 @SergeyBiryukov
9 years ago

  • Resolution → fixed
  • Status accepted → closed

In 40909:

I18N: Correct translator comment for response error message in WP_Community_Events::get_events().

Props Soean, tfirdaus.
Fixes #40939.

Note: See TracTickets for help on using tickets.