constant_contact_user() function should only act on accounts if the $edit['cc_newsletter'] variable is present

The section of code in constant_contact_user() will unsubscribe users if user_save() is called as an api function (in our case, via the migrate module)

				if(isset($edit['cc_newsletter']) && $edit['cc_newsletter']):
					$status = $cc->update_contact($contact['id'], $edit['mail'], array_keys($lists), $fields);
				elseif (isset($edit['cc_newsletter'])) :
					$status = $cc->update_contact($contact['id'], $edit['mail'], array(), $fields);
				endif;

furthermore, even without that little gotcha, it slows the import process down by several orders of magnitude by unnecessarily contacting constant contact. The attached patch does 2 things. First, it aborts the function if $edit['cc_newsletter'] is not set (which is an indication that user_save() was called from a source other than the user edit or user registration form, and secondly, it updates the code above to explicitly only act if the cc_newsletter variable is present. I'm not familiar enough with the code here:

			else:
				if(isset($account->cc_newsletter) && $account->cc_newsletter):
					$status = $cc->update_contact($contact['id'], $account->mail, array_keys($lists), $fields);
				else:
					$status = $cc->update_contact($contact['id'], $account->mail, array(), $fields);
				endif;
			endif;

to know if and how it is called. It may need a similar elseif (isset($account->cc_newsletter)) to avoid inadvertent unsubscribe requests being sent to constant contact.