Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#50147 closed defect (bug) (fixed)

Add missing inline docs information to esc_url_raw

Reported by: audrasjb's profile audrasjb Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.5 Priority: normal
Severity: normal Version:
Component: Formatting Keywords: has-patch
Focuses: docs Cc:

Description (last modified by audrasjb)

In wp-includes/formatting.php, there should be a reference to wp_allowed_protocols under $protocols parameters, in the related inline documentation of esc_url_raw function.

To be consistent, we should probably copy the documentation available for esc_url.

Attachments (1)

50147.diff (575 bytes) - added by audrasjb 5 years ago.
Formatting: add missing inline documentation to esc_url_raw function

Download all attachments as: .zip

Change History (4)

@audrasjb
5 years ago

Formatting: add missing inline documentation to esc_url_raw function

#1 @audrasjb
5 years ago

  • Description modified (diff)

#2 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 5.5

#3 @SergeyBiryukov
5 years ago

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

In 47783:

Docs: Add a note that the $protocols parameter of esc_url_raw() defaults to return value of wp_allowed_protocols().

This complements already existing note in esc_url().

Props audrasjb.
Fixes #50147.

Note: See TracTickets for help on using tickets.