#3490 closed defect (bug) (fixed)
wp-login.php?action=register should redirect you to /register
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 1.5 | Priority: | normal |
Severity: | normal | Version: | 1.5 |
Component: | Members | Keywords: | has-patch |
Cc: |
Description
Title says it all :\
Found this out on my site, unfortunately :( Hope my members don't discover it!
Attachments (1)
Change History (7)
#1
@
14 years ago
- Component changed from Core to Members
- Keywords has-patch added
- Version set to 1.5
#2
@
14 years ago
- Milestone changed from Awaiting Review to 1.5
- Severity changed from major to normal
This is a regression from 1.2
#4
@
14 years ago
You might want to consider my patch and removing the older code; my patch hooks directly into WP's registration page instead of having to declare a variable to look for the $_GET[action] and looking at $_SERVER[SCRIPT_NAME] for wp-signup.php.
If you don't want to consider my patch, you can at least use BP's native signup permalink function:
bp_core_redirect( trailingslashit( bp_get_root_domain() . '/' . $bp->pages->register->slug ) );
to:
bp_core_redirect( bp_get_signup_page() );
---
EDIT: I see that the current code is used for multisite installs as well (wp-signup.php); my code is only useful for single WP. There doesn't appear to be a better hook in wp-signup.php for multisite. In wp-signup.php, 'before_signup_form' is available, but get_header() is rendered first.
Okay, I'll recommend my latter option instead!
Here's a quick patch. I mean 'quick' because I was going so fast that I abruptly cut off the PHPDoc for bp_core_redirect()!
I'll let you guys finish the rest :)