Checking for duplicate emails in user_requirements() does not take langcode into account

I just found this issue myself, I think it doesn't really matter to group by uid, but we definitely need to group by language.

I think we could add some additional coverage to UserRequirementsTest for this - might just be a case of adding some more data.

I added a test.

Test coverage added

1) Drupal\Tests\user\Kernel\UserRequirementsTest::testTranslatedUserEmail
Failed asserting that an array does not have the key 'conflicting emails'.
/builds/issue/drupal-3490639/core/modules/user/tests/src/Kernel/UserRequirementsTest.php:80
FAILURES!
Tests: 3, Assertions: 20, Failures: 1.

Fix itself seems pretty simple enough

LGTM

Confirmed changes in fork branch to the two core user files resolves the issue on a new Drupal 11.6 install with multiple languages.

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

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

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

Rebased, restoring RTBC

Hi!
Any news on this issue?
We had the same problem on our Drupal with duplicated users

Committed to 11.x and backported to 11.2.x - thanks folks

larowlan → closed merge request !10396

Thanks @larowlan, could we backport this to 10.x too?

Hi @pcambra - I don't think it qualifies for allowed changes for a maintenance minor release → - but thanks for asking. If you feel otherwise, please comment with reasons and/or reach out on slack and we can discuss with release managers.

Fixing credits

Automatically closed - issue fixed for 2 weeks with no activity.