Context Navigation

← Previous Ticket
Next Ticket →

#15413 closed defect (bug) (fixed)

inconsistent use of home_url() in author-template.php

Reported by:	chineseleper	Owned by:	markjaquith
Priority:	normal	Milestone:	3.1
Component:	Permalinks	Version:	3.1
Severity:	minor	Keywords:	has-patch
Cc:		Focuses:

Description

should be

home_url(trailingslashit($link))

not

home_url().trailingslashit($link)

get_author_posts_url()

Attachments (1)

author-template.patch (526 bytes ) - added by chineseleper 16 years ago.: Patch

Download all attachments as: .zip

Change History (8)

#1 @nacin
16 years ago

Milestone Awaiting Review → 3.1

Best to diff from the root, that way the path in the patch includes wp-includes.

We'd also want to add some whitespace there, i.e. home_url( trailingslashit( $link ) ).

@chineseleper
16 years ago

Attachment author-template.patch added

Patch

#2 @chineseleper
16 years ago

patch updated

#3 @kapeels
16 years ago

Keywords has-patch added

#4 @markjaquith
16 years ago

Owner set to markjaquith
Status new → assigned

Looks like it got fixed, but there is another that I see.

#5 @markjaquith
16 years ago

Resolution → fixed
Status assigned → closed

(In [16425]) The path goes INSIDE home_url(). fixes #15413

#6 @nacin
16 years ago

Fixed it in [16401] when moving to user_trailingslashit.

This ticket was mentioned in Slack in #core-editor by bobbingwide. View the logs.

6 years ago

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#15413 closed defect (bug) (fixed)

inconsistent use of home_url() in author-template.php

Description

Attachments (1)

Change History (8)

#1 @nacin 16 years ago

@chineseleper 16 years ago

#2 @chineseleper 16 years ago

#3 @kapeels 16 years ago

#4 @markjaquith 16 years ago

#5 @markjaquith 16 years ago

#6 @nacin 16 years ago

This ticket was mentioned in ​Slack in #core-editor by bobbingwide. ​View the logs. 6 years ago