WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#15131 closed defect (bug) (fixed)

wp_admin_bar_me_separator using wrong profile link

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

Description

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 4 years ago.

Download all attachments as: .zip

Change History (6)

@willnorris — 4 years ago

comment:1 @ryan — 4 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.

comment:2 @ryan — 4 years ago

  • Milestone changed from Awaiting Review to 3.1

comment:3 @jane — 4 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?

comment:4 @filosofo — 4 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.

comment:5 @ryan — 4 years ago

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

[16085] handles my suggestion.

Note: See TracTickets for help on using tickets.