Simplify tnt import method for rubocop

Split out part of the tnt import update_contact method so that rubocop would pass (was saying it was too complex). That was breaking the build.
Cool, just a little thought.

On Wed, Dec 3, 2014 at 11:11 AM, David Raffensperger
<notifications@github.com> wrote:

>> @@ -243,6 +243,26 @@ def create_or_update_mailchimp(mail_chimp_list_id, mail_chimp_key)
>>    end
>>  
>>    def update_contact(contact, row)
>> +    update_contact_basic_fields(contact, row)
>> +
>> +    if (@import.override? || contact.send_newsletter.blank?) && true?(row['SendNewsletter'])
>> +      case row['NewsletterMediaPref']
>> +      when '+E', '+E-P'
>> +        contact.send_newsletter = 'Email'
>> +      when '+P', '+P-E'
>> +        contact.send_newsletter = 'Physical'
>> +      else
>> +        contact.send_newsletter = 'Both'
>> +      end
>> +    end
>> +
>> +    tags = @tags_by_contact_id[row['id']]
>> +    tags.each { |tag| contact.tag_list.add(tag) } if tags
> I'll be working on improving the tnt import as one of my major next steps, so I'll make a note to myself to improve that line hopefully making it a single array amend command like you mentioned.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CruGlobal/mpdx/pull/200/files#r21247018