Password reset invalid mail notify language

@anup.sinha. Thanks working like i needed.

function imberia_user_user_mail_language_alter(AccountInterface $account, &$language) {
  if( Drupal::currentUser()->isAnonymous()){
    $language = Drupal::languageManager()->getCurrentLanguage()->getId();
  }
}

Thank you again. And you are welcome to own social network.Imberia

Hi @Utilvideo,

I am happy to hear that the solution is working as you expected. So can you please now merge it into 9.5 and 10 branches if everything looks good.

Thanks & Regards,
Anup

Changes will be integrate in Drupal Core?

Hi @Utilvideo, I was just thinking that it would be good if we can integrate the patch into Drupal core as the issue exists in Drupal 8, 9 and 10 versions. Please let me know your thoughts on this.

Thanks & Regards,
Anup

I think, this changes must be in core.

Thanks.

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Hi @Utilvideo,

Now I have found out why my earlier patch failed to apply for the automated test run here but working fine in local. I created the patch from 9.5.0 as this was the installed version in my local but here for the automated test run the version is 9.5.x. The dictionary.txt file is different in 9.5.x compare 9.5.0 and that is why the last patch failed to apply here.

So here is my updated patch created for 9.5.x version.

Thanks & Regards,
Anup

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

Did not test.

But this will need an issue summary update for the proposed solution, helps the committers later.
Also with a new api this will need a change record.

Thanks.

Removing issue summary update but still will need a change record.

Change record added for the new hook - hook_user_mail_language_alter().

Removing change record.

1 NIT is the addition of the dictionary.txt should be alphabetic but that's being added to the bottom.

D10 patch would be needed too.

Adding a reroll patch for drupal: 10.1.x, please review.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.