WordPress.org
Get WordPress

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#56132 closed defect (bug) (fixed)

Need to add url escaping

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

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 2 years ago.

Download all attachments as: .zip

Change History (7)

@jakariaistauk
2 years ago

#1 @rudlinkon
2 years ago

  • Keywords has-patch added; needs-patch removed

#2 @rudlinkon
2 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
2 years ago

  • Focuses administration added
  • Version changed from trunk to 3.0

Related ticket: #56133

#4 @costdev
2 years ago

  • Milestone changed from Awaiting Review to 6.1

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

#5 @SergeyBiryukov
2 years ago

  • Component changed from Administration to Customize

#6 @SergeyBiryukov
2 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to 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.