[buddypress-trac] [BuddyPress Trac] #5762: WP Multi Network plugin & BP 2.0.1 signup

buddypress-trac noreply at wordpress.org
Mon Oct 19 00:00:14 UTC 2015


#5762: WP Multi Network plugin & BP 2.0.1 signup
------------------------------+-----------------------
 Reporter:  summoner          |       Owner:
     Type:  defect (bug)      |      Status:  reopened
 Priority:  normal            |   Milestone:
Component:  Component - Core  |     Version:
 Severity:  normal            |  Resolution:
 Keywords:                    |
------------------------------+-----------------------

Comment (by needle):

 Replying to [comment:4 r-a-y]:
 > Misread your post!

 To be fair, I may have misread the original ticket as well!

 I'm not entirely sure the issue I identified is actually the cause of the
 behaviour described by @summoner. On re-reading the ticket, it's asking
 for separated signups per network, yet BP Multi Network specifically
 states that it doesn't separate signups between networks.

 Whilst the SQL errors as a result of targeting a non-existent 'signups'
 table are real, they don't actually prevent signups on the non-main
 network. They do, however, break all the checks to the signups table that
 BuddyPress makes on the non-main network - for example the 'Manage
 Signups' page is non-functional. I wonder if the OP could have been
 getting confused by the broken behaviour?

 > Does `bp_core_get_table_prefix()` even need to be added to `buddypress
 ()->members-table_name_signups`?

 I wondered about this too, but it seems to get more complex and involved
 the more I think about it. Ultimately, I don't think it's worth BuddyPress
 doing any heavy-lifting here at all, given that the current code works
 just fine in all situations except a BP Multi Network context...

 > Regardless, I think we could add a filter for devs that need this.

 Thanks - it seems better to let BP Multi Network do the filtering of
 `$wpdb->base_prefix` to point to the existing 'signups' table. I've
 [https://wordpress.org/support/topic/filtering-table-prefix-for-signups-
 table-produces-sql-errors opened a thread] at the plugin's WP repo.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5762#comment:5>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list