[buddypress-trac] [BuddyPress] #3582: Browser title for a component's directory only displays the word 'Directory'

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Fri Sep 16 19:58:01 UTC 2011


#3582: Browser title for a component's directory only displays the word
'Directory'
------------------------------------+------------------
 Reporter:  masonjames              |       Owner:
     Type:  defect                  |      Status:  new
 Priority:  normal                  |   Milestone:  1.5
Component:  Core                    |     Version:  1.5
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+------------------

Comment (by boonebgorges):

 This is another change that stems from the fact that
 $bp->active_components and $bp->loaded_components were jumbled at some
 point during the dev cycle.

 As for bp_get_options_nav(), this change will have no effect. It turns out
 that $bp->displayed_user is *never* empty, so the "then" condition of the
 ternary operator is never called. This is either due to a logic error of
 mine at one point, or due to more recent changes in BuddyPress. At the
 moment, the logic error doesn't matter, because the way that component ids
 are indexed has also changed since that point - so, in contrast to what
 the comment says, you don't actually ever need the root_slug. We should
 open a ticket to look at this after 1.5, but for now I don't want to touch
 it because it's working. In any case, 3582.01.patch will have no effect on
 it.

 r-a-y: Good call. (Except that a straight numeric check won't work,
 because those 1s are strings.) See 3582.02.patch.

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


More information about the buddypress-trac mailing list