Context Navigation

← Previous Ticket
Next Ticket →

#13245 closed enhancement (fixed)

Add trailing slash to home_url() path

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

Description

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

Attachments (1)

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

Download all attachments as: .zip

Change History (7)

#1 @zeo
16 years ago

Keywords 2nd-opinion has-patch added
Owner 2nd-opinion has-patch removed
Status new → assigned

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

Keywords needs-refresh added; has-patch removed

@zeo
16 years ago

Attachment 13245.patch added

#4 @zeo
16 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
16 years ago

Resolution → fixed
Status assigned → closed

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

#6 @emiluzelac
9 years ago

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

Last edited 9 years ago by emiluzelac (previous) (diff)

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#13245 closed enhancement (fixed)

Add trailing slash to home_url() path

Description

Attachments (1)

Change History (7)

#1 @zeo 16 years ago

#2 @nacin 16 years ago

#3 @sivel 16 years ago

@zeo 16 years ago

#4 @zeo 16 years ago

#5 @nacin 16 years ago

#6 @emiluzelac 9 years ago