Make WordPress Core

Opened 8 years ago

Last modified 3 years ago

#36439 reviewing defect (bug)

Wrong language when resetting password

Reported by: vasikgreif's profile vasikgreif Owned by: johnbillion's profile johnbillion
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Login and Registration Keywords: has-patch
Focuses: multisite Cc:


When using WPML (or other multilanguage plugin) along with multisite, the reset password email comes in wrong language, as the reset password form is sent to network_site_url() instead of site_url(). I think a proper way should be:

<form name="lostpasswordform" id="lostpasswordform" action="<?php echo esc_url( site_url( 'wp-login.php?action=lostpassword', 'login_post' ) ); ?>" method="post">

Attachments (1)

36439.diff (774 bytes) - added by adriantirusli 4 years ago.

Download all attachments as: .zip

Change History (12)

#1 @ocean90
8 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Hello @vasikgreif, welcome to our Trac!

Thanks for your report, we're already tracking this issue in #21352.

#2 @johnbillion
4 years ago

  • Keywords needs-patch added
  • Milestone set to 5.5
  • Resolution duplicate deleted
  • Status changed from closed to reopened
  • Version 4.4.2 deleted

I think this issue can be addressed separately from #21352, which is focused on the URL used for the password reset, and keep this ticket focused on the language used in the email.

I believe this can be fixed along with the work on #43700 which enables users to switch language on the login screens.

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.

4 years ago

#4 @johnbillion
4 years ago

  • Milestone changed from 5.5 to 5.6

This ticket was mentioned in PR #517 on WordPress/wordpress-develop by adriantirusli.

4 years ago

  • Keywords has-patch added; needs-patch removed

In this update I changed the login URL to fix reset password email comes in wrong language.

Trac ticket: #36439

4 years ago

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.

4 years ago

#7 @helen
4 years ago

  • Milestone changed from 5.6 to Future Release

Patch/PR here looks more related to #21352 so should probably be moved over there. As for the language of the email itself issue, I'm going to punt from 5.6 seeing as #43700 isn't milestoned in and we're into beta now. If somebody wants to pick this up, we can always reconsider.

#8 @johnbillion
3 years ago

  • Milestone changed from Future Release to 5.7
  • Owner set to johnbillion
  • Status changed from reopened to reviewing

#9 @johnbillion
3 years ago

  • Milestone changed from 5.7 to Future Release

#10 @ocean90
3 years ago

#54297 was marked as a duplicate.

#11 @henry.wright
3 years ago

Related ticket: #54298.

Note: See TracTickets for help on using tickets.