Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Post patches to the issue to update module with last core changes
Comment | File | Size | Author |
---|---|---|---|
#8 | update_email-2360037-8.patch | 4.71 KB | gambry |
|
Comments
Comment #1
berenddeboer CreditAttribution: berenddeboer commentedComment #3
andypostI fixed the patch and incorporate changes from duplicates
#2382653: Drupal 8, beta 3 update - registration errors
#2382701: Drupal 8, beta 3 update - User login error
Let's left this open to continue update while core changes
Comment #4
andypostFix tests
Comment #7
andypostTest fixed, now we need to extend tests to make sure that all forms altered without errors
Comment #8
gambryIn this patch:
- Code re-factoring to better follow drupal standards
- Removing deprecated calls
- Removing useless db_update on email_registration_user_insert()
- Adding test for email_registration_unique_username()
Comment #9
estoyausenteThe patch apply correctly and the code is much better than before. We need a little push over this module, it is simple and the use case is very common.
Can be this patch committed?
Comment #10
andypostThanx, looks great, will commit asap
Minor nit
s/Make/Makes
Comment #12
andypostThanx, fixed on commit and pushed
Beta version coming