WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#56132 closed defect (bug) (fixed)

Need to add url escaping

Reported by: jakariaistauk Owned by: SergeyBiryukov
Priority: normal Milestone: 6.1
Component: Customize Version: 3.0
Severity: normal Keywords: has-patch
Cc: Focuses: administration, coding-standards

Description

Hi,
I found a issue in 'src/wp-admin/includes/class-custom-image-header.php' file. the issue about url escaping

Attachments (1)

56132.patch​ (971 bytes ) - added by jakariaistauk 4 years ago.

Download all attachments as: .zip

Change History (7)

@jakariaistauk
4 years ago

#1 @rudlinkon
4 years ago

  • Keywords has-patch added; needs-patch removed

#2 @rudlinkon
4 years ago

  • Focuses coding-standards added

Thanks, jakariaistauk for your patch, it looks good to me. I appreciate your contribution and hope you will continue it.

#3 @costdev
4 years ago

  • Focuses administration added
  • Version trunk → 3.0

Related ticket: #56133

#4 @costdev
4 years ago

  • Milestone Awaiting Review → 6.1

Hi @jakariaistauk, welcome to Trac and thanks for the patch! Let's milestone this for 6.1.

#5 @SergeyBiryukov
4 years ago

  • Component Administration → Customize

#6 @SergeyBiryukov
4 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 53642:

Coding Standards: Escape the home URL in the "Header updated. Visit your site" message.

This affects Custom_Image_Header::step_1().

Follow-up to [12890], [45654].

Props jakariaistauk, rudlinkon, hztyfoon, costdev.
Fixes #56132.

Note: See TracTickets for help on using tickets.