Currently the email_registration module is missing a dependency on the user module.

CommentFileSizeAuthor
#5 add_dependency_to_user-2831246-5.patch350 bytesJeroenT
#3 add_dependency_to_user-2831246-7.x.patch347 bytesJeroenT
#2 add_dependency_to_user-2831246-2.patch343 bytesJeroenT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JeroenT created an issue. See original summary.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Active » Needs review
FileSize
add_dependency_to_user-2831246-2.patch343 bytes

8.x patch.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
FileSize
add_dependency_to_user-2831246-7.x.patch347 bytes

7.x patch.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented 
+++ b/email_registration.info
@@ -2,3 +2,4 @@ name = Email Registration
+dependencies[] = user

I think this should be drupal:user

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
FileSize
add_dependency_to_user-2831246-5.patch350 bytes

Created a new patch.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

thanx

  • andypost committed e22de57 on 8.x-1.x authored by JeroenT 
    Issue #2831246 by JeroenT: Add dependency to user module

  • andypost committed 4100322 on 7.x-1.x authored by JeroenT 
    Issue #2831246 by JeroenT: Add dependency to user module
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Reviewed & tested by the community » Fixed

Thanx, commited to 8.x and 7.x

Status: Fixed » Closed (fixed)

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