Hi all! just came to tell you, that line 328 looks like mistype: $default_form instead $default_from.

Good luck, and thanks for great module!

CommentFileSizeAuthor
#2 mistype_in-2549339-2.patch1.56 KBPravin Ajaaz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dmitry Pogodaev created an issue. See original summary.

Pravin Ajaaz’s picture

Pravin Ajaaz CreditAttribution: Pravin Ajaaz commented
FileSize
mistype_in-2549339-2.patch1.56 KB

a patch to fix this

Pravin Ajaaz’s picture

Pravin Ajaaz CreditAttribution: Pravin Ajaaz commented
Status: Active » Needs review
Pravin Ajaaz’s picture

Pravin Ajaaz CreditAttribution: Pravin Ajaaz at Ameex-Drupal Geeks commented
geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
Status: Needs review » Postponed (maintainer needs more info)

In the module it's $default_from, like it is supposed. Your patch would change it to default_form which seems wrong.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles at CARD.com commented
Status: Postponed (maintainer needs more info) » Closed (duplicate)

I think this is a duplicate of #2478935: Undefined variable: default_form in email_confirm_build_mail(). That one solves the problem in a different way that makes more sense.