CRM-5622 malformed error message causing fatal error in contact.php

Below, the result of print_r($values) directly ebfore the cough.

Notice the comma in the email address.

If i use the attached 5622.csv file with a simple mapping to first,last,email

It is caught in step 3 as an error and doesn't have the fatal error thrown between step 3 and four. I will try taking this one line as is into the same mapping

print_r($values);

Array
(
[is_error] => 1
[error_message] => No valid email address
)

Array
(
[0] => http://unbreakablecrypto.com:20082/civicrm/contact/view?reset=1&cid=84295
[1] => 3227
[2] => Sunrise Child Care
[3] => P.O. Box 36
[4] => North Thetford
[5] => VT
[6] => Orange
[7] => Geoffrey
[8] => Provider
[9] => Sheila
[10] => Home provider
[11] =>
[12] => FALSE
[13] => registered
[14] =>
[15] =>
[16] => Owner
[17] => 05054
[18] => (802) 333-9802
[19] =>
[20] =>
[21] =>
[22] =>
[23] =>
[24] =>
[25] => (802) 299-7979
[26] => geoffrey,sa@gmail.com
[27] =>
[28] =>
[29] =>
[30] => TRUE
[31] =>
[32] => TRUE
[33] => FALSE
[34] => NEW
[35] =>
[36] => 468
)

Okay, this file successfully breaks it.

okay 5622-a.csv successfully gets past step 3 and breaks on the way to step four.

The key here might be choosing 'fill' as the way to deal wtih duplicates.

Okay. here is the kicker!

So, what is happening is when i choose

mm/dd/yy OR mm-dd-yy (12/25/98 OR 12-25-98) OR (9/1/08 OR 9-1-08)

As the date, it some how doesn't see the malformed email at step 3, and allows the bad emails to attempt to be imported at step 4 which is where ti is breaking

Using
yyyy-mm-dd OR yyyymmdd (1998-12-25 OR 19981225) OR (2008-9-1 OR 20080901)

and

dd/mm/yyyy (25/12/1998) OR (1/9/2008)

Both successfully caught the error using the -a csv file on step 3.

We need to validate all email addresses, currently we are validating only single email address corresponding to $this->_emailIndex.

Tested in r25781.