Skip to:
Content

BuddyPress.org

Opened 14 years ago

Closed 14 years ago

#2997 closed defect (bug) (fixed)

wp-includes/registration.php is deprecated in WP 3.1

Reported by: djpaul's profile DJPaul Owned by: nacin's profile nacin
Milestone: 1.5 Priority: normal
Severity: Version:
Component: Core Keywords: has-patch
Cc:

Description

Create a wrapper function to retain compatibility with 3.0.

Attachments (1)

2997.diff​ (4.0 KB) - added by nacin 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 @djpaul
14 years ago

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

(In [3661]) Don't include wp-includes/registration.php if on WordPress 3.1. Fixes #2997

#2 @nacin
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Going to supply an alternative patch for this.

#3 @nacin
14 years ago

  • Owner set to nacin
  • Status changed from reopened to accepted

#4 @nacin
14 years ago

  • Keywords has-patch added

Two problems: One, this doesn't work, due to the lack of version_compare. Two, it's better to do feature detection (which you can later just drop when you drop support for 3.0). Three, after searching for registration.php functions, I found places that registration.php was being included apparently unnecessarily. Patch attached. Needs a test, as I'm doing this without an install of BuddyPress trunk actually active.

@nacin
14 years ago

#5 @boonebgorges
14 years ago

+1 on feature detection over an extra function. Patch is mostly good, though there was one necessary require that got deleted somewhere along the way (fired at account activation).

#6 @boonebgorges
14 years ago

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

(In [3666]) Refactors registration.php requires so they don't require an additional function. Fixes #2997. Props nacin

Note: See TracTickets for help on using tickets.