Change email source

I changed email source for sending notifications in this module (great module)
I found it in code where it's set, and i want it to be sent to CCK field set in nodes and not to user who made node and i am tring it like this, changing auto_expire_notify function to load node and then send mail to it, seems its WORKING, so just wanted to share this and ask if this is proper way to do it :-)

function _auto_expire_notify($nid, $mailkey, $subject, $body, $args) {
  if (!empty($subject)) {

//load node and get email field
	$node = node_load($nid);
	$user_email = $node->field_email[0]['email'];			
      
      if ($user_email) {
        $subject = t($subject, $args);
        $body    = t($body, $args);
        $params = array('subject' => $subject, 'body' => $body);
        
        $bcc = trim(variable_get(AUTO_EXPIRE_EMAIL .'bcc', ''));
        
        if ($bcc == '') {
          $sent = drupal_mail('auto_expire', $mailkey, $user_email, language_default(), $params);
        }
        else {
          $params['bcc'] = $bcc;
          $sent = drupal_mail('auto_expire', $mailkey .'_bcc', $user_email, language_default(), $params);
        }