Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#15131 closed defect (bug) (fixed)

wp_admin_bar_me_separator using wrong profile link

Reported by: willnorris's profile willnorris Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch dev-feedback close
Focuses: Cc:


the wp_admin_bar_me_separator function is constructing the link to the user's profile instead of using admin_url(), which can generate the wrong URL when WordPress is not installed at the document root.

patch attached.

Attachments (1)

admin-bar.diff (763 bytes) - added by willnorris 12 years ago.

Download all attachments as: .zip

Change History (6)

#1 @ryan
12 years ago

I think we need something like get_profile_link() that will link to the profile.php in the users primary blog admin if the user has a blog and to the personal dashboard if the user does not have a blog.

#2 @ryan
12 years ago

  • Milestone changed from Awaiting Review to 3.1

#3 @jane
12 years ago

  • Keywords has-patch dev-feedback added

@ryan: Would Will's patch be a decent band-aid, given that freeze is coming up, or will you do what you propose above?

#4 @filosofo
12 years ago

  • Keywords close added

The particular change in willnorris's patch came in with r16038.
I suggest we close and open a separate ticket to do something like get_profile_link.

#5 @ryan
12 years ago

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

[16085] handles my suggestion.

Note: See TracTickets for help on using tickets.