[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