Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#35720 closed defect (bug) (fixed)

Missing periods in translation strings

Reported by: ramiy's profile ramiy Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.5 Priority: lowest
Severity: trivial Version:
Component: Text Changes Keywords: has-patch commit
Focuses: Cc:


In wp-admin/network/sites.php we have this array of strings:

	$manage_actions = array(
		'activateblog'   => __( 'You are about to activate the site %s' ),
		'deactivateblog' => __( 'You are about to deactivate the site %s' ),
		'unarchiveblog'  => __( 'You are about to unarchive the site %s.' ),
		'archiveblog'    => __( 'You are about to archive the site %s.' ),
		'unspamblog'     => __( 'You are about to unspam the site %s.' ),
		'spamblog'       => __( 'You are about to mark the site %s as spam.' ),
		'deleteblog'     => __( 'You are about to delete the site %s.' ),
		'unmatureblog'   => __( 'You are about to mark the site %s as mature.' ),
		'matureblog'     => __( 'You are about to mark the site %s as not mature.' ),

Seems like the first two strings are missing periods at the end.

Attachments (1)

sites.patch (830 bytes) - added by ramiy 8 years ago.

Download all attachments as: .zip

Change History (4)

8 years ago

This ticket was mentioned in Slack in #core-i18n by ramiy. View the logs.

8 years ago

#2 @ocean90
8 years ago

  • Keywords has-patch commit added
  • Milestone changed from Awaiting Review to 4.5
  • Priority changed from normal to lowest
  • Severity changed from normal to trivial

#3 @SergeyBiryukov
8 years ago

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

In 36664:

I18N: Add missing periods to two strings in wp-admin/network/sites.php

Props ramiy.
Fixes #35720.

Note: See TracTickets for help on using tickets.