Skip to:

Opened 14 years ago

Closed 14 years ago

#2373 closed defect (bug) (fixed)

Use wp-admin/includes/ms.php in WP 3.0

Reported by: mattwiebe's profile mattwiebe Owned by:
Milestone: 1.2.4 Priority: critical
Severity: Version:
Component: Core Keywords: has-patch


BP includes the deprecated in WP 3.0 wp-admin/includes/mu.php file for setting spammer status and account deletion. We should use ms.php instead. Currently get a fatal error when trying to delete a user under WP 3.0.

I've attached a patch to load whichever is appropriate.

Attachments (2)

ms_include.diff (1.2 KB) - added by mattwiebe 14 years ago.
ms_include_with_checks.diff (2.0 KB) - added by mattwiebe 14 years ago.

Download all attachments as: .zip

Change History (5)

#1 @wpmuguru
14 years ago

You should check the bp_core_is_multisite function first before hitting the file system.

Also, $wp_version can be used to determine whether it is 3.0 or not.

#2 @mattwiebe
14 years ago

Thanks for the feedback Ron.

Loading of ms.php/mu.php happens after a bp_core_is_multisite() check (the bp_core_delete account() function already checked it before) and I'm using $wp_version to check which file to load. I did away with the file_exists() calls because they were unnecessary once we check for the version.

#3 @apeatling
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [3010]) Fixes #2373 props mattwiebe

Note: See TracTickets for help on using tickets.