Opened 14 years ago
Closed 14 years ago
#17761 closed defect (bug) (fixed)
wp-activate.php fails to load properly
Reported by: | frumph | Owned by: | ryan |
---|---|---|---|
Milestone: | 3.2 | Priority: | normal |
Severity: | blocker | Version: | 3.2 |
Component: | General | Keywords: | has-patch 2nd-opinion |
Focuses: | Cc: |
Description
wp-activate.php calls get_header(); it comes from the theme, if the theme has functions called from the functions.php it fatal errors out. So noone can activate their registered accounts.
wp-activate.php needs to load up the themes functions.php before calling the get_header(); get_footer();
Attachments (2)
Change History (7)
#1
@
14 years ago
- Milestone changed from Awaiting Review to 3.2
- Severity changed from normal to blocker
#2
@
14 years ago
WP_INSTALLING is defined in five situations:
- wp-activate.php, unsure original reason, but it prevents non-network plugins (plugins on the main site) from loading
- wp-admin/install.php - initial installation
- wp-admin/setup-config.php - config file creation
- wp-admin/upgrade.php - DB upgrades
- wp-includes/ms-functions.php - wpmu_create_blog()
This is a bit of hack, but we could introduce a new WP_ACTIVATING constant that is checked against:
if ( ! defined( 'WP_INSTALLING' ) || defined( 'WP_ACTIVATING' ) ) {
Since WP_INSTALLING is normally not going to be defined, this doesn't add any more overhead to typical pages, only on installing pages, which are already heavy. But it'll prevent an issue on activation.
Oooh, I had said this wasn't a regression over Twitter, but it is. Broken in [17727].