[buddypress-trac] [BuddyPress] #3219: BP_ENABLE_MULTIBLOG broken

[buddypress-trac at lists.automattic.com]

#3219: BP_ENABLE_MULTIBLOG broken
-----------------------+------------------------------------
  Reporter:  wpmuguru  |      Owner:  boonebgorges
      Type:  defect    |     Status:  assigned
  Priority:  major     |  Milestone:  1.3
 Component:  Core      |    Version:
Resolution:            |   Keywords:  has-patch dev-feedback
-----------------------+------------------------------------
Changes (by boonebgorges):

 * keywords:   => has-patch dev-feedback


Comment:

 3219.1.patch is my first attempt at a fix. Here's my thought process and
 strategy.

 Typically, admin screens are loaded in Network Admin when is_multisite()
 is true. This won't work for BP_ENABLE MULTIBLOG (or for multi-network-BP
 for that matter) because in those cases BP stuff (or at the very least bp-
 pages) needs to be blog-specific in order to work properly.

 3219.1.patch alters this behavior by hooking admin screens/actions to the
 individual blog screen when BP_ENABLE_MULTIBLOG is activated. Also, since
 the admin_hook and do_network_admin values are filtered, it gives things
 like multi-network-BP a chance to change how admin screens are handled.

 This patch has not been super-extensively tested or documented. I'm
 putting it up here so I can get feedback from the other core devs and from
 wpmuguru on whether this is a reasonable approach to the problem. Thanks!

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