- Issue created by @seanB
- Merge request !10396Take UID and langcode into account in user_requirements() β (Closed) created by seanB
- First commit to issue fork.
- πͺπΈSpain pcambra Asturies
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.
- π¬π§United Kingdom catch
I think we could add some additional coverage to
UserRequirementsTest
for this - might just be a case of adding some more data. - Status changed to Needs work
2 months ago 4:58am 22 April 2025 - First commit to issue fork.
- πΊπΈUnited States smustgrave
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
- πΊπΈUnited States w01f
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.
- πͺπΈSpain eduardo morales alberti Spain, πͺπΊ
Hi!
Any news on this issue?
We had the same problem on our Drupal with duplicated users -
larowlan β
committed df7e4792 on 11.2.x
Issue #3490639 by dcam, seanb: Checking for duplicate emails in...
-
larowlan β
committed df7e4792 on 11.2.x
-
larowlan β
committed 95d3d95f on 11.x
Issue #3490639 by dcam, seanb: Checking for duplicate emails in...
-
larowlan β
committed 95d3d95f on 11.x
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Committed to 11.x and backported to 11.2.x - thanks folks
- Status changed to Fixed
5 days ago 8:37pm 30 June 2025 - πͺπΈSpain pcambra Asturies
Thanks @larowlan, could we backport this to 10.x too?
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
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.