Mapper for CCK Email Field

Hi I've been doing some tests with this module and I tried to import cck email fields.
I copied the feedapi_mapper_link.inc and changed it to feedapi_mapper_email.inc.

Here's the code:

/**
 * On behalf implementation of hook_feedapi_mapper for email.module (CCK).
 * Email field module available at http://drupal.org/project/email
 * based on feedapi_mapper_link and feedapi_mapper_content.inc
 *
 * @param string $op
 * @param Drupal node $node
 * @param string $field_name
 * @param string, number or array of string or number $feed_element
 * @param string or number as id $sub_field
 *
 */
function email_feedapi_mapper($op, $node, $field_name, $feed_element = array(), $sub_field = '') {
  // Test for the node field that we would like to map to.
  if (strpos($field_name, 'field_') === 0) {
    if ($op == 'describe') {
      if (feedapi_mapper_content_is_cck_type($field_name, array('email'))) {
        return t('Maps an e-mail address to this e-mail CCK field.');
      }
      // Describe what we are doing in this mapper. This shows up as help text on the mapping page.
    }
    else if ($op == 'list') {
      if (feedapi_mapper_content_is_cck_type($field_name, array('email'))) {
        return TRUE;
      }
      // Here we are being asked to list sub fields we would like to map to.
      // In this case, we only map to the CCK field or not, so we return just TRUE.
      return FALSE;
    }
    else if ($op == 'map') {
      // Here is where the actual mapping happens.
      // When we are called at this point, $field_name contains the name of the field the user has
      // decided to map to and $field_element is the feed item element the user has decided to map.
      // We just need to put the two things together. The data structure here depends a lot on
      // CCK. We stick the value in $feed_element into $node->$field_name[0]['value'].
      if (!is_array($feed_element)) {
        $field = $node->$field_name;
        $field[0]['email'] = valid_email_address($feed_element) ? $feed_element : NULL;
        $node->$field_name = $field;
        return $node;
      }
    }
  }
}

I just did some quick checks so beware...

cheers