WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 4 months ago

#13245 closed enhancement (fixed)

Add trailing slash to home_url() path

Reported by: zeo Owned by:
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: Template Keywords: 2nd-opinion has-patch
Focuses: Cc:

Description

Add missing trailing slash in home_url() path and some clean up.

Attachments (1)

13245.patch​ (2.9 KB) - added by zeo 7 years ago.

Download all attachments as: .zip

Change History (7)

#1 @zeo
7 years ago

  • Keywords 2nd-opinion has-patch added
  • Owner 2nd-opinion has-patch deleted
  • Status changed from new to assigned

#2 @nacin
7 years ago

The change in default-widgets is invalid -- that's JS that we don't want escaped or filtered quite like that.

Also, get_search_query() in that case should actually be unescaped, then escaped, like: esc_url( home_url( '/?s=' . get_search_query( false ) ) ); (I think).

#3 @sivel
7 years ago

  • Keywords needs-refresh added; has-patch removed

@zeo
7 years ago

#4 @zeo
7 years ago

  • Keywords has-patch added; needs-refresh removed

Oops I think I previously send a wrong patch. Attached is the correct one according to title.

That default-widgets, feed-atom-comments, and author-template is for #9008. Sorry.

#5 @nacin
7 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [14443]) Add trailing slash to some home_url() calls. props zeo, fixes #13245.

#6 @emiluzelac
4 months ago

(update) Sorry, wrong ticket... disregard :)

Last edited 4 months ago by emiluzelac (previous) (diff)
Note: See TracTickets for help on using tickets.