WordPress.org

Make WordPress Core

Opened 15 months ago

Closed 14 months ago

Last modified 14 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 15 months ago.
47227.patch (1.6 KB) - added by ramiy 15 months ago.
47227a.png (13.0 KB) - added by ramiy 15 months ago.
site-health.php
47227b.png (13.0 KB) - added by ramiy 15 months ago.
site-health-info.php
47227.2.patch (875 bytes) - added by ramiy 15 months ago.

Download all attachments as: .zip

Change History (19)

@ramiy
15 months ago

@ramiy
15 months ago

#1 @ramiy
15 months ago

  • Keywords has-screenshots has-patch added

#2 @ramiy
15 months ago

  • Keywords site-health added

@ramiy
15 months ago

site-health.php

@ramiy
15 months ago

site-health-info.php

#3 @ramiy
15 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
15 months ago

  • Milestone changed from Awaiting Review to 5.3

#5 @SergeyBiryukov
15 months ago

  • Milestone changed from 5.3 to 5.2.2

#6 @ocean90
15 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
15 months ago

#7 @ramiy
15 months ago

  • Keywords needs-refresh removed

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


15 months ago

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


15 months ago

#10 @SergeyBiryukov
15 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
15 months ago

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

#12 @audrasjb
14 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
14 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
14 months ago

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