Making WordPress.org

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#3542 closed defect (bug) (fixed)

Incorrectly escaped percent sign in "about security"

Reported by: tobifjellner's profile tobifjellner Owned by: dd32's profile dd32
Milestone: Priority: normal
Component: General Keywords:
Cc:

Description

Location: https://meta.trac.wordpress.org/browser/sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-main/page-about-security.php around line 77

printf(
/* translators: 1: WordPress Market share - 30. Note the following % sign is escaped as %%. 2: Footnote 3: Market Penetration - 60.  Note the following % sign is escaped as %%. */
__( 'WordPress is a free and open source content management system (CMS). It is the most widely-used CMS software in the world and it powers more than %1$s%% of the top 10 million websites%2$s, giving it an estimated 58% market share of all sites
using a CMS.', 'wporg' ),
WP_MARKET_SHARE,
'<sup id="ref1"><a href="#footnote1">1</a></a></sup>',
60
);

The problem here is with "58%", and it's twofold:

  1. "58" would better be a parameter.
  2. The percent sign needs to be escaped into %%

Change History (2)

#1 @dd32
7 years ago

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

In 6997:

Main: About: Correct an incorrect placeholder in a string.

H/t tobifjellner.
Fixes #3542.

#2 @ocean90
7 years ago

  • Component changed from International Sites (Rosetta) to General
Note: See TracTickets for help on using tickets.