I was wondering if support could be added to integrate the contact importer directly into forms, rather than requiring them to be clicked on as a pop-up? If you look at forms that have contact importer enabled, it doesn't really stand out to users that you can import your contacts from other sites. Wouldn't it be better if the importer itself could be embedded in the forms, rather than just a link to a pop-up window? That way, users can see the icons from gmail, facebook, etc, and it is more obvious that they can be imported. Alternatively, showing the icons of the contacts that can be imported, and then opening the inviter form in a pop-up when they are clicked on, would be great.

Comments

NathanM’s picture

NathanM commented

This appears as though it is being integrated into the 7.x version. Any plans to port it to 6.2?

dalin’s picture

dalin
he/him
Primary language English
Location 🇨🇦, 🇲🇽, 🌍
commented

There is no development in progress on this (as far as I know). Patches are welcome.

mattcasey’s picture

mattcasey commented

I added this code to line 234 of 6.x-dev. It lets you skip the first step if the 2nd argument in the URL includes a provider's name (eg. http://mysite.com/contact_importer/gmail). I am using this with the form integration module, which seems to handle the redirect correctly once the contact_importer is finished.

  if(empty($form_state['storage']['step']) && arg(1)) {
  
    // Remove any emails stored from a previous attempt.
    contact_importer_set_contacts(NULL);

    $form_state['storage']['providers'] = _contact_importer_providers();
	if(array_key_exists(arg(1), $form_state['storage']['providers'])) {
      $form_state['storage']['provider'] = $form_state['storage']['providers'][arg(1)];
	  $form_state['storage']['provider']['provider_name'] = arg(1);
      $form_state['storage']['step'] = 'credentials';
	}
  }

The following output can be added to a field's description using hook_form_alter() in template.php:

    $output = '';
    $enabled_providers = _contact_importer_providers_enabled();
    foreach ($enabled_providers as $provider_name => $provider) {
      $favicon = !empty($provider['favicon']) ? $provider['favicon'] : '';
      $output .= l($favicon, 'contact_importer/' . $provider_name, array('attributes' => array('class' => 'contact-importer-link'), 'html' => TRUE));
    }
    return $output; // HTML output of favicons for each enabled provider to skip the first step in the form

I am using Popups API, and so far it shows me a popup to enter email and password, but it redirects me to a page to select email addresses, then redirects me back with the textarea filled in.

areynolds’s picture

areynolds commented

Thanks for posting this code. I did a slight variation to avoid hacking the module. Trying to maintain the holiday mood by getting into the Drupal spirit for January :)

function yourmodule_form_contact_importer_form_alter(&$form, &$form_state){
  if($form_state['storage']['step'] == 'select_provider' && arg(1)) {
    //@SEE: http://drupal.org/node/906450
    // Remove any emails stored from a previous attempt.
    contact_importer_set_contacts(NULL);
    $form_state['storage']['providers'] = _contact_importer_providers();
    if(array_key_exists(arg(1), $form_state['storage']['providers'])) {
      $form_state['storage']['provider'] = $form_state['storage']['providers'][arg(1)];
      $form_state['storage']['provider']['provider_name'] = arg(1);
      $form_state['storage']['step'] = 'credentials';
    }
    //Had trouble with this; wasn't passing $form_state correctly?
    //$form = drupal_get_form('contact_importer_form', TRUE);
    //$form = drupal_get_form('contact_importer_form', $form_state, TRUE); 
    $form = contact_importer_form($form_state, TRUE);
  }
}

You'll end up calling the form twice, but I don't think the performance implications are serious. Might be worth it just to keep things a bit more modular.