WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 3 years ago

Last modified 3 years ago

#16945 closed enhancement (duplicate)

Add relative URIs containing a query in esc_url()

Reported by: hakre Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: Formatting Keywords: has-patch
Focuses: Cc:

Description

I'd like to enhance the esc_url function to accept an url starting with ? (query) to not require a protocol.

That's comparable with the # (fragment) case we already have in.

This is useful to create links to the current page that need a query only. Otherwise the php file must be added and for frontend pages it won't work with pretty permalinks.

Attachments (1)

16945.patch (874 bytes) - added by hakre 5 years ago.
First approach

Download all attachments as: .zip

Change History (7)

@hakre
5 years ago

First approach

#1 @hakre
5 years ago

Related: (In [11126]) Don't prepend http to fragments. Props scohoust. fixes #4051

#2 follow-up: @c3mdigital
3 years ago

  • Keywords close 2nd-opinion added

#3 @SergeyBiryukov
3 years ago

  • Component changed from General to Formatting

#4 in reply to: ↑ 2 @hakre
3 years ago

Replying to c3mdigital:

Can you please justify your close suggestion?

#5 @nacin
3 years ago

  • Resolution set to worksforme
  • Status changed from new to closed

This actually changed some time ago.

#6 @SergeyBiryukov
3 years ago

  • Keywords close 2nd-opinion removed
  • Milestone Awaiting Review deleted
  • Resolution changed from worksforme to duplicate

Duplicate of #18260.

Note: See TracTickets for help on using tickets.