Home User profile
tagDiv Member
This user did not write anything. So we are just showing here some random text to make the profile page look nice :)
Hortipoint
tagDiv Member

Hi Vlad,

Thanks! To make this more clear for your developers; I think I’ve found the bug. In the old version the text generation looks like this (td_login.php)


$message = __('Someone has requested a password reset for the following account:') . "\r\n\r\n";
$message .= network_home_url( '/' ) . "\r\n\r\n";
$message .= sprintf(__('Username: %s'), $user_login) . "\r\n\r\n";
$message .= __('If this was a mistake, just ignore this email and nothing will happen.') . "\r\n\r\n";
$message .= __('To reset your password, visit the following address:') . "\r\n\r\n";
$message .= '<' . network_site_url("wp-login.php?action=rp&key=$key&login=" . rawurlencode($user_login), 'login') . ">\r\n";

And in the new version like this


$message = 'Someone has requested a password reset for the following account:' . "\r\n\r\n";
$message .= network_home_url( '/' ) . "\r\n\r\n";
$message .= sprintf('Username: %s', $user_login) . "\r\n\r\n";
$message .= 'If this was a mistake, just ignore this email and nothing will happen.' . "\r\n\r\n";
$message .= 'To reset your password, visit the following address:' . "\r\n\r\n";
$message .= '<' . network_site_url("wp-login.php?action=rp&key=$key&login=" . rawurlencode($user_login), 'login') . ">\r\n";

The translation function __() seems to be simply forgotten during the migration to the td_composer plugin. Furthermore you are not passing the user data to the ‘retrieve_password_message’ hook, so it’s mostly useless. It’s implemented like this (td_login.php);
$message = apply_filters('retrieve_password_message', $message, $key);
But it should be (and it’s an easy fix) to do it like this:

$message = apply_filters( 'retrieve_password_message', $message, $key, $user_login, $user_data );

so you can actually generate a working link in a user function. You need the $user_login for this.

Hortipoint
tagDiv Member

Hi!

I ended up creating my own email using the retrieve_password_message hook. It works fine but it’s to bad this is necessary after a newspaper update. It sounds like a bug to me.

Thanks.

Hortipoint
tagDiv Member

Hi Vlad,

Thank you for your time.

Our WordPress language is definitely Dutch.
Screenshot

The e-mails sent from the /wp-login.php screen are therefore Dutch, and the wp-admin panel is also Dutch, as expected. Only the emails from the newspaper login screen are English.

Viewing 3 posts - 1 through 3 (of 3 total)