WordPress.org

Make WordPress Core

Opened 5 years ago

Last modified 7 weeks ago

#36439 reviewing defect (bug)

Wrong language when resetting password

Reported by: vasikgreif Owned by: johnbillion
Milestone: 5.7 Priority: normal
Severity: normal Version:
Component: Login and Registration Keywords: has-patch
Focuses: multisite Cc:

Description

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 5 months ago.

Download all attachments as: .zip

Change History (9)

#1 @ocean90
5 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
9 months 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.


7 months ago

#4 @johnbillion
7 months ago

  • Milestone changed from 5.5 to 5.6

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


5 months 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

@adriantirusli
5 months ago

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


3 months ago

#7 @helen
3 months 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
7 weeks ago

  • Milestone changed from Future Release to 5.7
  • Owner set to johnbillion
  • Status changed from reopened to reviewing
Note: See TracTickets for help on using tickets.