[buddypress-trac] [BuddyPress Trac] #6370: Blogs: Improvements to bp_blogs_record_existing_blogs()

buddypress-trac noreply at wordpress.org
Sat Oct 10 15:37:30 UTC 2015


#6370: Blogs: Improvements to bp_blogs_record_existing_blogs()
--------------------------------------------------+------------------
 Reporter:  r-a-y                                 |       Owner:
     Type:  enhancement                           |      Status:  new
 Priority:  normal                                |   Milestone:  2.4
Component:  Component - Blogs                     |     Version:  2.1
 Severity:  normal                                |  Resolution:
 Keywords:  has-patch dev-feedback needs-testing  |
--------------------------------------------------+------------------

Comment (by DJPaul):

 I don't have a multisite to test this. Code review of the patch:

 * bp-core-admin-tools.php: use get_current_screen() to get the screen ID
 rather than look in the global?
 * Same file: I'd suggest the message more like "blog recording in
 progress, but something went wrong, so click some button here to start it
 again".
 * Same file: `$blogs_offset` only used once, remove the variable.

 Nothing else major.

 This needs to land in trunk very, very soon I think to make 2.4. ;)

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


More information about the buddypress-trac mailing list