Sending to multiple email addresses not working

Good day.
I noticed since the upgrade to 1.6.3, sending to multiple email addresses at the same time no longer works. The mail addresses were always just separated by a ‘comma’. I am now getting a “The To Email field must contain a valid email address” error message when trying to do it the same way.
Thanks

2 Likes

Hello @Shane_van_Niekerk,

Thank you for your report. This might be a bug. Could I kindly ask you to file a bug report here so we can take a closer look at the issue?

If you’re not comfortable using GitHub, you can also describe the issue to us here, step by step, so we can reproduce it and identify where the problem lies.

2 Likes

I have never used GitHub, so I think it best I just describe the issue here. I was running on 1.6.1 and simply did an upgrade to 1.6.3. Before the upgrade I was able to send an invoice to multiple email address’s (in some cases I used up to 4) simply by comma separating the address’s. Since the upgrade it is no longer possible. I had to send the emails multiple times to get the invoice delivered to all the required recipients.
Thank you.

1 Like

This is the template to describe your issues (with an example):

### Title

Sending emails to multiple email addresses gives error message

### Expected Behavior

Sending emails to multiple email addresses gives error message

### Current Behavior

Sending emails to multiple email addresses gives error message

### Steps to Reproduce

1. Go to Invoice
2. Click menu top right corner to email invoice
3. In "cc", "bcc" or "to" enter multiple email addresses, separated by a comma ","
4. Error message: `the field "cc_mail" must contain a valid email address`

### Possible Solution (Optional)

In mailer_helper.php de email addresses are run through "filter_var" even though they can contain multiple email addresse separated by a comma

### Context (Environment)

...

### Failure Logs (Optional)

...

Focus on the Steps to Reproduce (and screenshots)
The small, small sentences that you give are usually not enough (that’s why my co-worker asked to fill in the issue on github).

Issue found, bug report created: Sending emails to multiple email addresses gives error message · Issue #1307 · InvoicePlane/InvoicePlane · GitHub

Bugfix created, it’s now a pull-request

2 Likes

Thanks. Managed to make the change to my installation.

2 Likes