search

veda-consulting-company / uk.co.vedaconsulting.mailchimp

Other
22 stars 43 forks source link

This extension breaks the Merge facility when a contact has no email address. #83

Closed artfulrobot closed 10 years ago

artfulrobot commented 10 years ago

The Merge tool silently fails to complete because this extension calls its deletion function with an empty value for email id (when a contact does not have an email).

This causes an invalid SQL command to be executed, which triggers an execption, which is silently handled resulting in CiviCRM reporting a successful merge, but it's actually failed at the delete hook; the record that should have been deleted is actually not.

artfulrobot commented 10 years ago

Merge request submitted with patch for this (just this) https://github.com/veda-consulting/uk.co.vedaconsulting.mailchimp/pull/84