collective / collective.easyform

Forms for Plone

Home Page:https://pypi.org/project/collective.easyform

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

should the Mailer action send email if a "to" field is not set?

yurj opened this issue · comments

commented

to = formataddr(self.get_owner_info(context))

Hi!

this fallback will send the mail to the object owner/portal email address if the to fields is None. This is a problem because a second mailer can be setup just to get an email from a dropdown only if the user select a value from the dropdown. Now, the mail is always sent to the obect owner or portal email address. The original commit 8c438c3 didn't mention why this change has been done.

I think no mail should be sent if no to field is set. If cc is set and no to is set, to field should get the value from cc.

What do you think?