Make WordPress Core

Opened 10 years ago

Last modified 5 years ago

#36439 reviewing defect (bug)

Wrong language when resetting password

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

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 6 years ago.

Download all attachments as: .zip

Change History (12)

#1 @ocean90
10 years ago

  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status newclosed

Hello @vasikgreif, welcome to our Trac!

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

#2 @johnbillion
6 years ago

  • Keywords needs-patch added
  • Milestone5.5
  • Resolution duplicate
  • Status closedreopened
  • Version 4.4.2

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.


6 years ago

#4 @johnbillion
6 years ago

  • Milestone 5.55.6

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


6 years ago
#5

  • 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
6 years ago

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


6 years ago

#7 @helen
6 years ago

  • Milestone 5.6Future 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
6 years ago

  • Milestone Future Release5.7
  • Owner set to johnbillion
  • Status reopenedreviewing

#9 @johnbillion
6 years ago

  • Milestone 5.7Future Release

#10 @ocean90
5 years ago

#54297 was marked as a duplicate.

#11 @henry.wright
5 years ago

Related ticket: #54298.

Note: See TracTickets for help on using tickets.