[buddypress-trac] [BuddyPress] #5026: bp_get_loggedin_user_nav() - marked as @deprecated, but used in themes

[buddypress-trac]

#5026: bp_get_loggedin_user_nav() - marked as @deprecated, but used in themes
-------------------------+-----------------------
 Reporter:  tivnet       |       Owner:
     Type:  enhancement  |      Status:  reopened
 Priority:  lowest       |   Milestone:
Component:  Core         |     Version:  1.7
 Severity:  trivial      |  Resolution:
 Keywords:               |
-------------------------+-----------------------
Changes (by tivnet):

 * cc: gregory@… (added)
 * status:  closed => reopened
 * resolution:  invalid =>


Comment:

 @boonebgorges:

 Just to get your attention. Probably, related also to #4624

 `bp_nav_menu`, while very nice indeed, '''cannot be displayed site-wide'''
 without some fixes. Specifically, outside the `/members/`, all top links
 are missing the root URL, because when `bp_displayed_user_domain()` is
 empty, this part works quite ugly:

 `$link = bp_loggedin_user_domain() ? str_replace(
 bp_loggedin_user_domain(), bp_displayed_user_domain(), $nav['link'] ) :
 trailingslashit( bp_displayed_user_domain() . $nav['link'] );`

 In addition, the `if ( ! $nav['show_for_displayed_user']...` part hides
 "Mesages" and "Settings", when outside the `/members/`.

 I am not back the original subject, "deprecated
 `bp_get_loggedin_user_nav()`". Just wanted to say that `bp_nav_menu()` is
 not a 100% suitable substitution.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5026#comment:4>
BuddyPress <http://buddypress.org/>
BuddyPress