Make WordPress Core

Opened 5 years ago

Last modified 11 months ago

#18743 new enhancement

Persistent redirect_to For Login, Registration, and Lost Password

Reported by: sparkweb Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.3
Component: Login and Registration Keywords: needs-patch
Focuses: Cc:


Currently, if redirect_to is passed in to the login screen and then a user clicks Register or Lost Password, the redirect_to gets lost. I think that the redirect_to should be persistent and honored whether the user logs in, registers or gets a lost password.

I've written a hacky filter to do this, but I think this ought to be included in the core. Here's what I did:

//Keep redirect_to in URL
add_filter('site_url', 'foxyshop_add_registration_redirect', 5);
function foxyshop_add_registration_redirect($path) {
	if ((strpos($path, "action=register") !== false || strpos($path, "action=lostpassword") !== false) && isset($_REQUEST['redirect_to'])) return $path . '&redirect_to='.urlencode($_REQUEST['redirect_to']);
	if (substr($path, strlen($path)-12) == "wp-login.php" && isset($_REQUEST['redirect_to'])) return $path . '?redirect_to='.urlencode($_REQUEST['redirect_to']);
	return $path;

(I know 3.3 is in freeze now, but there wasn't another version option)

Change History (6)

#1 @kawauso
5 years ago

The Version is for the earliest version affected by the issue, rather than the one you want it fixed in.

#2 @SergeyBiryukov
4 years ago

#23523 was marked as a duplicate.

#3 @onokazu
3 years ago

Maybe the following line in wp-login.php (line 709 in WP 3.5.1)

<a href="<?php echo esc_url( site_url( 'wp-login.php?action=register', 'login' ) ); ?>"><?php _e( 'Register' ); ?></a> |

should be

<a href="<?php echo esc_url( site_url( 'wp-login.php?action=register&' . urlencode($redirect_to), 'login' ) ); ?>"><?php _e( 'Register' ); ?></a> |
Last edited 3 years ago by onokazu (previous) (diff)

#4 @jeremyfelt
3 years ago

  • Component changed from Users to Login and Registration

This ticket was mentioned in IRC in #wordpress-dev by rmccue. View the logs.

2 years ago

#6 @chriscct7
11 months ago

  • Keywords needs-patch added

Good First Bug: Attempt to replicate

Note: See TracTickets for help on using tickets.