Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#3490 closed defect (bug) (fixed)

wp-login.php?action=register should redirect you to /register

Reported by: intermike's profile InterMike 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)

3490.01.patch (712 bytes) - added by r-a-y 13 years ago.

Download all attachments as: .zip

Change History (7)

@r-a-y
13 years ago

#1 @r-a-y
13 years ago

  • Component changed from Core to Members
  • Keywords has-patch added
  • Version set to 1.5

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 :)

#2 @DJPaul
13 years ago

  • Milestone changed from Awaiting Review to 1.5
  • Severity changed from major to normal

This is a regression from 1.2

#3 @djpaul
13 years ago

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

(In [4999]) Fix wp-login register redirect. Fixes #3490

#4 @r-a-y
13 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!

Last edited 13 years ago by r-a-y (previous) (diff)

#5 @djpaul
13 years ago

(In [5013]) Use BP template function instead of global when redirecting to register page. See #3490, props r-a-y

#6 @sucipto82
12 years ago

deleted as spam - MOD

Last edited 12 years ago by boonebgorges (previous) (diff)
Note: See TracTickets for help on using tickets.