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

Comments

Dmitry Pogodaev created an issue. See original summary.

pravin ajaaz’s picture

pravin ajaaz commented
StatusFileSize
new mistype_in-2549339-2.patch1.56 KB

a patch to fix this

pravin ajaaz’s picture

pravin ajaaz commented
Status: Active » Needs review
pravin ajaaz’s picture

pravin ajaaz commented
geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
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
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.