Problems sending to multiple addresses without using address book.

[stever-corp]

When composing a new message to multiple recipients and typing in the recipients, an error is received when attempting to send the message. A javascript alert comes up and says "Please enter at least one recipient." The error occurs in Firefox 2.0.0.11 Win32 and IE7.

I have tried the following combinations of email addresses:

address1@email.com,address2@email.com
address1@email.com ,address2@email.com
address1@email.com, address2@email.com
address1@email.com , address2@email.com
address1@email.com;address2@email.com
address1@email.com ;address2@email.com
address1@email.com; address2@email.com
address1@email.com ; address2@email.com

[stever-corp]

This also occurs with spaces and new lines separating the addresses.

SVN Info:

Path: .
URL: https://svn.roundcube.net/trunk/roundcubemail
Repository Root: https://svn.roundcube.net
Repository UUID: 208e9e7b-5314-0410-a742-e7e81cd9613c
Revision: 952
Node Kind: directory
Schedule: normal
Last Changed Author: tomekp
Last Changed Rev: 952
Last Changed Date: 2008-01-08 13:40:06 -0500 (Tue, 08 Jan 2008)

[thomasb]

This is done by function rcube_check_email() in program/js/common.js

[robin]

RoundCube does not recognize this as recipient:

robin@…, robin@…

But it recognizes this:

<robin@…>, <robin@…>

And even this (which is weird):

<robin@…>, robin@…

[robin]

Possible fix.

[robin]

#1484710.200802011152.patch contains a possible fix.

Though the expression is now still broken. For example it still allows:

address@…>, address@…

I propose to first split the recipient list using ',' and ';', and then check for valid addresses.

[robin]

#1484710.200802011307.patch first splits using ',' or ';', then checks the seperate addresses for validity. Also fixed the check that allowed 'address@…>' and such.

[till]

@robin Please commit!

[thomasb]

Fixed in r1011