[wp-trac] [WordPress Trac] #52886: Update esc_url to allow for specifying an https:// as default protocol

WordPress Trac noreply at wordpress.org
Wed Apr 28 21:53:30 UTC 2021


#52886: Update esc_url to allow for specifying an https:// as default protocol
-------------------------------------------------+-------------------------
 Reporter:  mkaz                                 |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  5.8
Component:  Formatting                           |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-unit-tests needs-    |     Focuses:
  dev-note                                       |
-------------------------------------------------+-------------------------
Changes (by audrasjb):

 * keywords:  has-patch needs-unit-tests => has-patch needs-unit-tests
     needs-dev-note


Comment:

 In `52886.1.diff`:
 - Add @since mention
 - Few coding standards fixes (whitespaces inside docblocks)

 Question: shouldn't we also add the new parameter in the `clean_url`
 filter at the bottom of the function? Not sure it's really needed, but I
 prefer to ask :)

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/52886#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list