WordPress.org

Make WordPress Core

Opened 13 months ago

Closed 12 months ago

Last modified 11 months ago

#47227 closed defect (bug) (fixed)

i18n: Merge similar translation strings - site health tabs

Reported by: ramiy Owned by: SergeyBiryukov
Milestone: 5.2.2 Priority: normal
Severity: normal Version:
Component: Site Health Keywords: has-screenshots has-patch site-health fixed-major
Focuses: administration Cc:

Description

Site Health has two files (site-health.php and site-health-info.php). Both use tab navigation. One file use simple _e() functions, the other _ex(). This creates two translations strings in translate.wordpress.org .

The attached patch merges the two translation strings by using context function in both files.

In addition, it remove the translator comments. Not required as the string has context.

Attachments (5)

47227.png (6.6 KB) - added by ramiy 13 months ago.
47227.patch (1.6 KB) - added by ramiy 13 months ago.
47227a.png (13.0 KB) - added by ramiy 12 months ago.
site-health.php
47227b.png (13.0 KB) - added by ramiy 12 months ago.
site-health-info.php
47227.2.patch (875 bytes) - added by ramiy 12 months ago.

Download all attachments as: .zip

Change History (19)

@ramiy
13 months ago

@ramiy
13 months ago

#1 @ramiy
13 months ago

  • Keywords has-screenshots has-patch added

#2 @ramiy
13 months ago

  • Keywords site-health added

@ramiy
12 months ago

site-health.php

@ramiy
12 months ago

site-health-info.php

#3 @ramiy
12 months ago

In Hebrew we have two different strings in those two pages. Because one uses context function for the site-health page and the other doesn't, and it uses the general "status" which is used as post-status.

#4 @desrosj
12 months ago

  • Milestone changed from Awaiting Review to 5.3

#5 @SergeyBiryukov
12 months ago

  • Milestone changed from 5.3 to 5.2.2

#6 @ocean90
12 months ago

  • Keywords needs-refresh added

Please do not remove the translator comments. They are fine as is even if a context is provided.

@ramiy
12 months ago

#7 @ramiy
12 months ago

  • Keywords needs-refresh removed

This ticket was mentioned in Slack in #core-php by marioernestoms. View the logs.


12 months ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


12 months ago

#10 @SergeyBiryukov
12 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 45488:

I18N: Merge duplicate "Status" and "Info" strings on Site Health screens.

Props ramiy.
Fixes #47227.

#11 @SergeyBiryukov
12 months ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

#12 @audrasjb
12 months ago

Hi @SergeyBiryukov do you think we can have a backport for this ticket today so we can ship that on 5.2.2 RC1?
Thank you! The RC processing is going to start at 19:00 UTC.

#13 @SergeyBiryukov
12 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 45512:

I18N: Merge duplicate "Status" and "Info" strings on Site Health screens.

Props ramiy.
Merges [45488] to the 5.2 branch.
Fixes #47227.

#14 @spacedmonkey
11 months ago

  • Component changed from Administration to Site Health
Note: See TracTickets for help on using tickets.