Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#47989 closed defect (bug) (fixed)

Must escape `%` in `printf`

Reported by: afragen Owned by: SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: major Version:
Component: Upgrade/Install Keywords: needs-patch
Focuses: Cc:


There's an error in r45926.

after() was changed to use a printf() statement, but the string must escape the % in width="100%" to be width="100%%".

Pinging @SergeyBiryukov

Change History (2)

#1 @SergeyBiryukov
2 years ago

  • Milestone changed from Awaiting Review to 5.3

Great catch, thanks!

#2 @SergeyBiryukov
2 years ago

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

In 46072:

Upgrade/Install: Escape the % in width="100%" in a printf() call in Plugin_Upgrader_Skin::after().

Props afragen.
Fixes #47989.

Note: See TracTickets for help on using tickets.