--- contact/contact.module	2007-01-10 16:17:51.000000000 +0100
+++ contact.module	2007-05-28 22:10:08.000000000 +0200
@@ -35,7 +35,7 @@ function contact_help($section) {
  * Implementation of hook_perm
  */
 function contact_perm() {
-  return array('access site-wide contact form');
+  return array('access site-wide contact form', 'access personal contact form');
 }
 /**
  * Implementation of hook_menu().
@@ -101,7 +101,7 @@ function contact_menu($may_cache) {
           'title' => t('Contact'),
           'callback' => 'contact_user_page',
           'type' => MENU_LOCAL_TASK,
-          'access' => $user->uid,
+          user_access('access personal contact form'),
           'weight' => 2,
         );
       }
@@ -303,7 +303,7 @@ function contact_user_page() {
   global $user;
 
   if ($account = user_load(array('uid' => arg(1)))) {
-    if (!valid_email_address($user->mail)) {
+    if ($user->uid && !valid_email_address($user->mail)) {
       $output = t('You need to provide a valid e-mail address to contact other users. Please update your <a href="@url">user information</a> and try again.', array('@url' => url("user/$user->uid/edit")));
     }
     else if (!flood_is_allowed('contact', variable_get('contact_hourly_threshold', 3))) {
@@ -324,10 +324,28 @@ function contact_user_page() {
 function contact_mail_user($recipient) {
   global $user;
   $form['#token'] = $user->name . $user->mail;
-  $form['from'] = array('#type' => 'item',
-    '#title' => t('From'),
-    '#value' => check_plain($user->name) .' &lt;'. check_plain($user->mail) .'&gt;',
-  );
+  if ($user->uid) {
+    // User is logged in so we'll use the account e-mail address.
+    $form['from'] = array('#type' => 'item',
+      '#title' => t('From'),
+      '#value' => check_plain($user->name) .' &lt;'. check_plain($user->mail) .'&gt;',
+    );
+  }
+  else
+  {
+    // User is not logged in so we must ask his e-mail address and validate it.
+    $form['name'] = array('#type' => 'textfield',
+      '#title' => t('Your name'),
+      '#required' => TRUE,
+      '#maxlength' => 255,
+    );
+    $form['mail'] = array('#type' => 'textfield',
+      '#title' => t('Your e-mail address'),
+      '#required' => TRUE,
+      '#maxlength' => 255,
+      '#validate' => array('contact_email_validate' => array()),
+    );
+  }
   $form['to'] = array('#type' => 'item',
     '#title' => t('To'),
     '#value' => check_plain($recipient->name),
@@ -342,9 +360,13 @@ function contact_mail_user($recipient) {
     '#rows' => 15,
     '#required' => TRUE,
   );
-  $form['copy'] = array('#type' => 'checkbox',
-    '#title' => t('Send yourself a copy.'),
-  );
+  // We do not allow anonymous users to send themselves a copy
+  // because it can be abused to spam people.
+  if ($user->uid) {
+    $form['copy'] = array('#type' => 'checkbox',
+      '#title' => t('Send yourself a copy.'),
+    );
+  }
   $form['submit'] = array('#type' => 'submit',
     '#value' => t('Send e-mail'),
   );
@@ -352,6 +374,16 @@ function contact_mail_user($recipient) {
 }
 
 /**
+ * Validate the personal contact page form submission.
+ */
+function contact_email_validate($form) {
+  $address = $form['#value'];
+  if (!valid_email_address($address)) {
+    form_error($form, t('%address is an invalid e-mail address.', array('%address' => $address)));
+  }
+}
+
+/**
  * Process the personal contact page form submission.
  */
 function contact_mail_user_submit($form_id, $form_values) {
@@ -360,7 +392,12 @@ function contact_mail_user_submit($form_
   $account = user_load(array('uid' => arg(1), 'status' => 1));
   // Compose the body:
   $message[] = "$account->name,";
-  $message[] = t("!name (!name-url) has sent you a message via your contact form (!form-url) at !site.", array('!name' => $user->name, '!name-url' => url("user/$user->uid", NULL, NULL, TRUE), '!form-url' => url($_GET['q'], NULL, NULL, TRUE), '!site' => variable_get('site_name', 'Drupal')));
+  if (!$user->uid) {
+    $message[] = t("!name (!name-url) has sent you a message via your contact form (!form-url) at !site.", array('!name' => $form_values['name'], '!name-url' => $form_values['mail'], '!form-url' => url($_GET['q'], NULL, NULL, TRUE), '!site' => variable_get('site_name', 'Drupal')));
+  }
+  else {
+    $message[] = t("!name (!name-url) has sent you a message via your contact form (!form-url) at !site.", array('!name' => $user->name, '!name-url' => url("user/$user->uid", NULL, NULL, TRUE), '!form-url' => url($_GET['q'], NULL, NULL, TRUE), '!site' => variable_get('site_name', 'Drupal')));
+  }
   $message[] = t("If you don't want to receive such e-mails, you can change your settings at !url.", array('!url' => url("user/$account->uid", NULL, NULL, TRUE)));
   $message[] = t('Message:');
   $message[] = $form_values['message'];
@@ -372,8 +409,12 @@ function contact_mail_user_submit($form_
 
   // Prepare all fields:
   $to = $account->mail;
-  $from = $user->mail;
-
+  if ($user->uid) {
+    $from = $user->mail;
+  }
+  else {
+    $from = $form_values['mail'];
+  }
   // Format the subject:
   $subject = '['. variable_get('site_name', 'Drupal') .'] '. $form_values['subject'];
 
@@ -390,7 +431,12 @@ function contact_mail_user_submit($form_
 
   // Log the operation:
   flood_register_event('contact');
-  watchdog('mail', t('%name-from sent %name-to an e-mail.', array('%name-from' => $user->name, '%name-to' => $account->name)));
+  if ($user->uid) {
+    watchdog('mail', t('%name-from sent %name-to an e-mail.', array('%name-from' => $user->name, '%name-to' => $account->name)));
+  }
+  else {
+    watchdog('mail', t('%name-from sent %name-to an e-mail.', array('%name-from' => $form_values['name'] . ', ' . $form_values['mail'] . ',', '%name-to' => $account->name)));
+  }
 
   // Set a status message:
   drupal_set_message(t('The message has been sent.'));