summaryrefslogtreecommitdiff
path: root/_darcs
diff options
context:
space:
mode:
authorEvan Prodromou <evan@prodromou.name>2008-12-23 12:33:30 -0500
committerEvan Prodromou <evan@prodromou.name>2008-12-23 12:33:30 -0500
commitba31059fef3f609673178c7a041c397540da1789 (patch)
treec53a5598f95b8adfaaf4dcd5708071225e78483c /_darcs
parent2d0aec255f3cc49e246d972a3482f444c5a44980 (diff)
reformat and document lib/mail.php for phpcs conformance
darcs-hash:20081223173330-84dde-8cbbfd3953e8a67d5e58679431f93651ddf3122a.gz
Diffstat (limited to '_darcs')
-rw-r--r--_darcs/inventory4
-rw-r--r--_darcs/patches/20081223173330-84dde-8cbbfd3953e8a67d5e58679431f93651ddf3122a.gzbin0 -> 5889 bytes
-rw-r--r--_darcs/pristine/lib/mail.php704
-rw-r--r--_darcs/tentative_pristine1015
4 files changed, 1187 insertions, 536 deletions
diff --git a/_darcs/inventory b/_darcs/inventory
index 083e53fd0..167d793d7 100644
--- a/_darcs/inventory
+++ b/_darcs/inventory
@@ -43,4 +43,6 @@ Evan Prodromou <evan@prodromou.name>**20081222201304
complete compliance with the code standards. Untested; maybe there are
some bad method names now.
-] \ No newline at end of file
+]
+[reformat and document lib/mail.php for phpcs conformance
+Evan Prodromou <evan@prodromou.name>**20081223173330] \ No newline at end of file
diff --git a/_darcs/patches/20081223173330-84dde-8cbbfd3953e8a67d5e58679431f93651ddf3122a.gz b/_darcs/patches/20081223173330-84dde-8cbbfd3953e8a67d5e58679431f93651ddf3122a.gz
new file mode 100644
index 000000000..30dd86a19
--- /dev/null
+++ b/_darcs/patches/20081223173330-84dde-8cbbfd3953e8a67d5e58679431f93651ddf3122a.gz
Binary files differ
diff --git a/_darcs/pristine/lib/mail.php b/_darcs/pristine/lib/mail.php
index c90140d3c..5638ae9bf 100644
--- a/_darcs/pristine/lib/mail.php
+++ b/_darcs/pristine/lib/mail.php
@@ -1,9 +1,12 @@
<?php
-/*
- * Laconica - a distributed open-source microblogging tool
- * Copyright (C) 2008, Controlez-Vous, Inc.
+/**
+ * Laconica, the distributed open-source microblogging tool
*
- * This program is free software: you can redistribute it and/or modify
+ * utilities for sending email
+ *
+ * PHP version 5
+ *
+ * LICENCE: This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as published by
* the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
@@ -15,304 +18,557 @@
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
+ *
+ * @category Mail
+ * @package Laconica
+ * @author Evan Prodromou <evan@controlyourself.ca>
+ * @author Zach Copley <zach@controlyourself.ca>
+ * @author Robin Millette <millette@controlyourself.ca>
+ * @author Sarven Capadisli <csarven@controlyourself.ca>
+ * @copyright 2008 Control Yourself, Inc.
+ * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
+ * @link http://laconi.ca/
*/
-if (!defined('LACONICA')) { exit(1); }
+if (!defined('LACONICA')) {
+ exit(1);
+}
-require_once('Mail.php');
+require_once 'Mail.php';
-function mail_backend() {
- static $backend = NULL;
+/**
+ * return the configured mail backend
+ *
+ * Uses the $config array to make a mail backend. Cached so it is safe to call
+ * more than once.
+ *
+ * @return Mail backend
+ */
- if (!$backend) {
- global $config;
- $backend = Mail::factory($config['mail']['backend'],
- ($config['mail']['params']) ? $config['mail']['params'] : array());
- if (PEAR::isError($backend)) {
- common_server_error($backend->getMessage(), 500);
- }
- }
- return $backend;
+function mail_backend()
+{
+ static $backend = null;
+
+ if (!$backend) {
+ global $config;
+ $backend = Mail::factory($config['mail']['backend'],
+ ($config['mail']['params']) ?
+ $config['mail']['params'] :
+ array());
+ if (PEAR::isError($backend)) {
+ common_server_error($backend->getMessage(), 500);
+ }
+ }
+ return $backend;
}
-# XXX: use Mail_Queue... maybe
+/**
+ * send an email to one or more recipients
+ *
+ * @param array $recipients array of strings with email addresses of recipients
+ * @param array $headers array mapping strings to strings for email headers
+ * @param string $body body of the email
+ *
+ * @return boolean success flag
+ */
-function mail_send($recipients, $headers, $body) {
- $backend = mail_backend();
+function mail_send($recipients, $headers, $body)
+{
+ // XXX: use Mail_Queue... maybe
+ $backend = mail_backend();
if (!isset($headers['Content-Type'])) {
$headers['Content-Type'] = 'text/plain; charset=UTF-8';
}
- assert($backend); # throws an error if it's bad
- $sent = $backend->send($recipients, $headers, $body);
- if (PEAR::isError($sent)) {
- common_log(LOG_ERR, 'Email error: ' . $sent->getMessage());
- return false;
- }
- return true;
+ assert($backend); // throws an error if it's bad
+ $sent = $backend->send($recipients, $headers, $body);
+ if (PEAR::isError($sent)) {
+ common_log(LOG_ERR, 'Email error: ' . $sent->getMessage());
+ return false;
+ }
+ return true;
}
-function mail_domain() {
- $maildomain = common_config('mail', 'domain');
- if (!$maildomain) {
- $maildomain = common_config('site', 'server');
- }
- return $maildomain;
-}
+/**
+ * returns the configured mail domain
+ *
+ * Defaults to the server name.
+ *
+ * @return string mail domain, suitable for making email addresses.
+ */
-function mail_notify_from() {
- $notifyfrom = common_config('mail', 'notifyfrom');
- if (!$notifyfrom) {
- $domain = mail_domain();
- $notifyfrom = common_config('site', 'name') .' <noreply@'.$domain.'>';
- }
- return $notifyfrom;
+function mail_domain()
+{
+ $maildomain = common_config('mail', 'domain');
+ if (!$maildomain) {
+ $maildomain = common_config('site', 'server');
+ }
+ return $maildomain;
}
-function mail_to_user(&$user, $subject, $body, $address=NULL) {
- if (!$address) {
- $address = $user->email;
- }
+/**
+ * returns a good address for sending email from this server
+ *
+ * Uses either the configured value or a faked-up value made
+ * from the mail domain.
+ *
+ * @return string notify from address
+ */
+
+function mail_notify_from()
+{
+ $notifyfrom = common_config('mail', 'notifyfrom');
- $recipients = $address;
- $profile = $user->getProfile();
+ if (!$notifyfrom) {
- $headers['From'] = mail_notify_from();
- $headers['To'] = $profile->getBestName() . ' <' . $address . '>';
- $headers['Subject'] = $subject;
+ $domain = mail_domain();
- return mail_send($recipients, $headers, $body);
+ $notifyfrom = common_config('site', 'name') .' <noreply@'.$domain.'>';
+ }
+
+ return $notifyfrom;
}
-# For confirming a Jabber address
+/**
+ * sends email to a user
+ *
+ * @param User &$user user to send email to
+ * @param string $subject subject of the email
+ * @param string $body body of the email
+ * @param string $address optional specification of email address
+ *
+ * @return boolean success flag
+ */
-function mail_confirm_address($user, $code, $nickname, $address) {
+function mail_to_user(&$user, $subject, $body, $address=null)
+{
+ if (!$address) {
+ $address = $user->email;
+ }
- $subject = _('Email address confirmation');
+ $recipients = $address;
+ $profile = $user->getProfile();
- $body = sprintf(_("Hey, %s.\n\nSomeone just entered this email address on %s.\n\n" .
- "If it was you, and you want to confirm your entry, use the URL below:\n\n\t%s\n\n" .
- "If not, just ignore this message.\n\nThanks for your time, \n%s\n")
- , $nickname, common_config('site', 'name')
- , common_local_url('confirmaddress', array('code' => $code)), common_config('site', 'name'));
- return mail_to_user($user, $subject, $body, $address);
+ $headers['From'] = mail_notify_from();
+ $headers['To'] = $profile->getBestName() . ' <' . $address . '>';
+ $headers['Subject'] = $subject;
+
+ return mail_send($recipients, $headers, $body);
}
-function mail_subscribe_notify($listenee, $listener) {
- $other = $listener->getProfile();
- mail_subscribe_notify_profile($listenee, $other);
+/**
+ * Send an email to confirm a user's control of an email address
+ *
+ * @param User $user User claiming the email address
+ * @param string $code Confirmation code
+ * @param string $nickname Nickname of user
+ * @param string $address email address to confirm
+ *
+ * @see common_confirmation_code()
+ *
+ * @return success flag
+ */
+
+function mail_confirm_address($user, $code, $nickname, $address)
+{
+ $subject = _('Email address confirmation');
+
+ $body = sprintf(_("Hey, %s.\n\n".
+ "Someone just entered this email address on %s.\n\n" .
+ "If it was you, and you want to confirm your entry, ".
+ "use the URL below:\n\n\t%s\n\n" .
+ "If not, just ignore this message.\n\n".
+ "Thanks for your time, \n%s\n"),
+ $nickname, common_config('site', 'name'),
+ common_local_url('confirmaddress', array('code' => $code)),
+ common_config('site', 'name'));
+ return mail_to_user($user, $subject, $body, $address);
+}
+
+/**
+ * notify a user of subscription by another user
+ *
+ * This is just a wrapper around the profile-based version.
+ *
+ * @param User $listenee user who is being subscribed to
+ * @param User $listener user who is subscribing
+ *
+ * @see mail_subscribe_notify_profile()
+ *
+ * @return void
+ */
+
+function mail_subscribe_notify($listenee, $listener)
+{
+ $other = $listener->getProfile();
+ mail_subscribe_notify_profile($listenee, $other);
}
-function mail_subscribe_notify_profile($listenee, $other) {
- if ($listenee->email && $listenee->emailnotifysub) {
- // use the recipients localization
+/**
+ * notify a user of subscription by a profile (remote or local)
+ *
+ * This function checks to see if the listenee has an email
+ * address and wants subscription notices.
+ *
+ * @param User $listenee user who's being subscribed to
+ * @param Profile $other profile of person who's listening
+ *
+ * @return void
+ */
+
+function mail_subscribe_notify_profile($listenee, $other)
+{
+ if ($listenee->email && $listenee->emailnotifysub) {
+
+ // use the recipient's localization
common_init_locale($listenee->language);
- $profile = $listenee->getProfile();
- $name = $profile->getBestName();
- $long_name = ($other->fullname) ? ($other->fullname . ' (' . $other->nickname . ')') : $other->nickname;
- $recipients = $listenee->email;
- $headers['From'] = mail_notify_from();
- $headers['To'] = $name . ' <' . $listenee->email . '>';
- $headers['Subject'] = sprintf(_('%1$s is now listening to your notices on %2$s.'),
+
+ $profile = $listenee->getProfile();
+
+ $name = $profile->getBestName();
+
+ $long_name = ($other->fullname) ?
+ ($other->fullname . ' (' . $other->nickname . ')') : $other->nickname;
+
+ $recipients = $listenee->email;
+
+ $headers['From'] = mail_notify_from();
+ $headers['To'] = $name . ' <' . $listenee->email . '>';
+ $headers['Subject'] = sprintf(_('%1$s is now listening to '.
+ 'your notices on %2$s.'),
$other->getBestName(),
- common_config('site', 'name'));
- $body = sprintf(_('%1$s is now listening to your notices on %2$s.'."\n\n".
- "\t".'%3$s'."\n\n".
- '%4$s'.
- '%5$s'.
- '%6$s'.
- "\n".'Faithfully yours,'."\n".'%7$s.'."\n\n".
- "----\n".
- "Change your email address or notification options at %8$s"),
- $long_name,
- common_config('site', 'name'),
- $other->profileurl,
- ($other->location) ? sprintf(_("Location: %s\n"), $other->location) : '',
- ($other->homepage) ? sprintf(_("Homepage: %s\n"), $other->homepage) : '',
- ($other->bio) ? sprintf(_("Bio: %s\n\n"), $other->bio) : '',
- common_config('site', 'name'),
- common_local_url('emailsettings'));
+ common_config('site', 'name'));
+
+ $body = sprintf(_('%1$s is now listening to your notices on %2$s.'."\n\n".
+ "\t".'%3$s'."\n\n".
+ '%4$s'.
+ '%5$s'.
+ '%6$s'.
+ "\n".'Faithfully yours,'."\n".'%7$s.'."\n\n".
+ "----\n".
+ "Change your email address or ".
+ "notification options at %8$s\n"),
+ $long_name,
+ common_config('site', 'name'),
+ $other->profileurl,
+ ($other->location) ?
+ sprintf(_("Location: %s\n"), $other->location) : '',
+ ($other->homepage) ?
+ sprintf(_("Homepage: %s\n"), $other->homepage) : '',
+ ($other->bio) ?
+ sprintf(_("Bio: %s\n\n"), $other->bio) : '',
+ common_config('site', 'name'),
+ common_local_url('emailsettings'));
+
// reset localization
common_init_locale();
- mail_send($recipients, $headers, $body);
- }
+ mail_send($recipients, $headers, $body);
+ }
}
-function mail_new_incoming_notify($user) {
+/**
+ * notify a user of their new incoming email address
+ *
+ * User's email and incoming fields should already be updated.
+ *
+ * @param User $user user with the new address
+ *
+ * @return void
+ */
+
+function mail_new_incoming_notify($user)
+{
+ $profile = $user->getProfile();
- $profile = $user->getProfile();
- $name = $profile->getBestName();
+ $name = $profile->getBestName();
- $headers['From'] = $user->incomingemail;
- $headers['To'] = $name . ' <' . $user->email . '>';
- $headers['Subject'] = sprintf(_('New email address for posting to %s'),
- common_config('site', 'name'));
+ $headers['From'] = $user->incomingemail;
+ $headers['To'] = $name . ' <' . $user->email . '>';
+ $headers['Subject'] = sprintf(_('New email address for posting to %s'),
+ common_config('site', 'name'));
- $body = sprintf(_("You have a new posting address on %1\$s.\n\n".
- "Send email to %2\$s to post new messages.\n\n".
- "More email instructions at %3\$s.\n\n".
- "Faithfully yours,\n%4\$s"),
- common_config('site', 'name'),
- $user->incomingemail,
- common_local_url('doc', array('title' => 'email')),
- common_config('site', 'name'));
+ $body = sprintf(_("You have a new posting address on %1\$s.\n\n".
+ "Send email to %2\$s to post new messages.\n\n".
+ "More email instructions at %3\$s.\n\n".
+ "Faithfully yours,\n%4\$s"),
+ common_config('site', 'name'),
+ $user->incomingemail,
+ common_local_url('doc', array('title' => 'email')),
+ common_config('site', 'name'));
- mail_send($user->email, $headers, $body);
+ mail_send($user->email, $headers, $body);
}
-function mail_new_incoming_address() {
- $prefix = common_confirmation_code(64);
- $suffix = mail_domain();
- return $prefix . '@' . $suffix;
+/**
+ * generate a new address for incoming messages
+ *
+ * @todo check the database for uniqueness
+ *
+ * @return string new email address for incoming messages
+ */
+
+function mail_new_incoming_address()
+{
+ $prefix = common_confirmation_code(64);
+ $suffix = mail_domain();
+ return $prefix . '@' . $suffix;
}
-function mail_broadcast_notice_sms($notice) {
+/**
+ * broadcast a notice to all subscribers with SMS notification on
+ *
+ * This function sends SMS messages to all users who have sms addresses;
+ * have sms notification on; and have sms enabled for this particular
+ * subscription.
+ *
+ * @param Notice $notice The notice to broadcast
+ *
+ * @return success flag
+ */
- # Now, get users subscribed to this profile
+function mail_broadcast_notice_sms($notice)
+{
+ // Now, get users subscribed to this profile
- $user = new User();
+ $user = new User();
- $user->query('SELECT nickname, smsemail, incomingemail ' .
- 'FROM user JOIN subscription ' .
- 'ON user.id = subscription.subscriber ' .
- 'WHERE subscription.subscribed = ' . $notice->profile_id . ' ' .
- 'AND user.smsemail IS NOT NULL ' .
- 'AND user.smsnotify = 1 ' .
+ $user->query('SELECT nickname, smsemail, incomingemail ' .
+ 'FROM user JOIN subscription ' .
+ 'ON user.id = subscription.subscriber ' .
+ 'WHERE subscription.subscribed = ' . $notice->profile_id . ' ' .
+ 'AND user.smsemail IS NOT null ' .
+ 'AND user.smsnotify = 1 ' .
'AND subscription.sms = 1 ');
- while ($user->fetch()) {
- common_log(LOG_INFO,
- 'Sending notice ' . $notice->id . ' to ' . $user->smsemail,
- __FILE__);
- $success = mail_send_sms_notice_address($notice, $user->smsemail, $user->incomingemail);
- if (!$success) {
- # XXX: Not sure, but I think that's the right thing to do
- common_log(LOG_WARNING,
- 'Sending notice ' . $notice->id . ' to ' . $user->smsemail . ' FAILED, cancelling.',
- __FILE__);
- return false;
- }
- }
-
- $user->free();
- unset($user);
-
- return true;
+ while ($user->fetch()) {
+ common_log(LOG_INFO,
+ 'Sending notice ' . $notice->id . ' to ' . $user->smsemail,
+ __FILE__);
+ $success = mail_send_sms_notice_address($notice,
+ $user->smsemail,
+ $user->incomingemail);
+ if (!$success) {
+ // XXX: Not sure, but I think that's the right thing to do
+ common_log(LOG_WARNING,
+ 'Sending notice ' . $notice->id . ' to ' .
+ $user->smsemail . ' FAILED, cancelling.',
+ __FILE__);
+ return false;
+ }
+ }
+
+ $user->free();
+ unset($user);
+
+ return true;
}
-function mail_send_sms_notice($notice, $user) {
- return mail_send_sms_notice_address($notice, $user->smsemail, $user->incomingemail);
+/**
+ * send a notice to a user via SMS
+ *
+ * A convenience wrapper around mail_send_sms_notice_address()
+ *
+ * @param Notice $notice notice to send
+ * @param User $user user to receive notice
+ *
+ * @see mail_send_sms_notice_address()
+ *
+ * @return boolean success flag
+ */
+
+function mail_send_sms_notice($notice, $user)
+{
+ return mail_send_sms_notice_address($notice,
+ $user->smsemail,
+ $user->incomingemail);
}
-function mail_send_sms_notice_address($notice, $smsemail, $incomingemail) {
+/**
+ * send a notice to an SMS email address from a given address
+ *
+ * We use the user's incoming email address as the "From" address to make
+ * replying to notices easier.
+ *
+ * @param Notice $notice notice to send
+ * @param string $smsemail email address to send to
+ * @param string $incomingemail email address to set as 'from'
+ *
+ * @return boolean success flag
+ */
- $to = $nickname . ' <' . $smsemail . '>';
- $other = $notice->getProfile();
+function mail_send_sms_notice_address($notice, $smsemail, $incomingemail)
+{
+ $to = $nickname . ' <' . $smsemail . '>';
- common_log(LOG_INFO, "Sending notice " . $notice->id . " to " . $smsemail, __FILE__);
+ $other = $notice->getProfile();
- $headers = array();
- $headers['From'] = (isset($incomingemail)) ? $incomingemail : mail_notify_from();
- $headers['To'] = $to;
- $headers['Subject'] = sprintf(_('%s status'),
- $other->getBestName());
- $body = $notice->content;
+ common_log(LOG_INFO, 'Sending notice ' . $notice->id .
+ ' to ' . $smsemail, __FILE__);
- return mail_send($smsemail, $headers, $body);
+ $headers = array();
+
+ $headers['From'] = ($incomingemail) ? $incomingemail : mail_notify_from();
+ $headers['To'] = $to;
+ $headers['Subject'] = sprintf(_('%s status'),
+ $other->getBestName());
+
+ $body = $notice->content;
+
+ return mail_send($smsemail, $headers, $body);
}
-function mail_confirm_sms($code, $nickname, $address) {
+/**
+ * send a message to confirm a claim for an SMS number
+ *
+ * @param string $code confirmation code
+ * @param string $nickname nickname of user claiming number
+ * @param string $address email address to send the confirmation to
+ *
+ * @see common_confirmation_code()
+ *
+ * @return void
+ */
- $recipients = $address;
+function mail_confirm_sms($code, $nickname, $address)
+{
+ $recipients = $address;
- $headers['From'] = mail_notify_from();
- $headers['To'] = $nickname . ' <' . $address . '>';
- $headers['Subject'] = _('SMS confirmation');
+ $headers['From'] = mail_notify_from();
+ $headers['To'] = $nickname . ' <' . $address . '>';
+ $headers['Subject'] = _('SMS confirmation');
- $body = "$nickname: confirm you own this phone number with this code:";
- $body .= "\n\n";
- $body .= $code;
- $body .= "\n\n";
+ // FIXME: I18N
- mail_send($recipients, $headers, $body);
+ $body = "$nickname: confirm you own this phone number with this code:";
+ $body .= "\n\n";
+ $body .= $code;
+ $body .= "\n\n";
+
+ mail_send($recipients, $headers, $body);
}
-function mail_notify_nudge($from, $to) {
+/**
+ * send a mail message to notify a user of a 'nudge'
+ *
+ * @param User $from user nudging
+ * @param User $to user being nudged
+ *
+ * @return boolean success flag
+ */
+
+function mail_notify_nudge($from, $to)
+{
common_init_locale($to->language);
- $subject = sprintf(_('You\'ve been nudged by %s'), $from->nickname);
-
- $from_profile = $from->getProfile();
-
- $body = sprintf(_("%1\$s (%2\$s) is wondering what you are up to these days and is inviting you to post some news.\n\n".
- "So let's hear from you :)\n\n".
- "%3\$s\n\n".
- "Don't reply to this email; it won't get to them.\n\n".
- "With kind regards,\n".
- "%4\$s\n"),
- $from_profile->getBestName(),
- $from->nickname,
- common_local_url('all', array('nickname' => $to->nickname)),
- common_config('site', 'name'));
+ $subject = sprintf(_('You\'ve been nudged by %s'), $from->nickname);
+
+ $from_profile = $from->getProfile();
+
+ $body = sprintf(_("%1\$s (%2\$s) is wondering what you are up to ".
+ "these days and is inviting you to post some news.\n\n".
+ "So let's hear from you :)\n\n".
+ "%3\$s\n\n".
+ "Don't reply to this email; it won't get to them.\n\n".
+ "With kind regards,\n".
+ "%4\$s\n"),
+ $from_profile->getBestName(),
+ $from->nickname,
+ common_local_url('all', array('nickname' => $to->nickname)),
+ common_config('site', 'name'));
common_init_locale();
- return mail_to_user($to, $subject, $body);
+ return mail_to_user($to, $subject, $body);
}
-function mail_notify_message($message, $from=NULL, $to=NULL) {
+/**
+ * send a message to notify a user of a direct message (DM)
+ *
+ * This function checks to see if the recipient wants notification
+ * of DMs and has a configured email address.
+ *
+ * @param Message $message message to notify about
+ * @param User $from user sending message; default to sender
+ * @param User $to user receiving message; default to recipient
+ *
+ * @return boolean success code
+ */
- if (is_null($from)) {
- $from = User::staticGet('id', $message->from_profile);
- }
+function mail_notify_message($message, $from=null, $to=null)
+{
+ if (is_null($from)) {
+ $from = User::staticGet('id', $message->from_profile);
+ }
- if (is_null($to)) {
- $to = User::staticGet('id', $message->to_profile);
- }
+ if (is_null($to)) {
+ $to = User::staticGet('id', $message->to_profile);
+ }
- if (is_null($to->email) || !$to->emailnotifymsg) {
- return true;
- }
+ if (is_null($to->email) || !$to->emailnotifymsg) {
+ return true;
+ }
common_init_locale($to->language);
- $subject = sprintf(_('New private message from %s'), $from->nickname);
-
- $from_profile = $from->getProfile();
-
- $body = sprintf(_("%1\$s (%2\$s) sent you a private message:\n\n".
- "------------------------------------------------------\n".
- "%3\$s\n".
- "------------------------------------------------------\n\n".
- "You can reply to their message here:\n\n".
- "%4\$s\n\n".
- "Don't reply to this email; it won't get to them.\n\n".
- "With kind regards,\n".
- "%5\$s\n"),
- $from_profile->getBestName(),
- $from->nickname,
- $message->content,
- common_local_url('newmessage', array('to' => $from->id)),
- common_config('site', 'name'));
+ $subject = sprintf(_('New private message from %s'), $from->nickname);
+
+ $from_profile = $from->getProfile();
+
+ $body = sprintf(_("%1\$s (%2\$s) sent you a private message:\n\n".
+ "------------------------------------------------------\n".
+ "%3\$s\n".
+ "------------------------------------------------------\n\n".
+ "You can reply to their message here:\n\n".
+ "%4\$s\n\n".
+ "Don't reply to this email; it won't get to them.\n\n".
+ "With kind regards,\n".
+ "%5\$s\n"),
+ $from_profile->getBestName(),
+ $from->nickname,
+ $message->content,
+ common_local_url('newmessage', array('to' => $from->id)),
+ common_config('site', 'name'));
common_init_locale();
- return mail_to_user($to, $subject, $body);
+ return mail_to_user($to, $subject, $body);
}
-function mail_notify_fave($other, $user, $notice) {
+/**
+ * notify a user that one of their notices has been chosen as a 'fave'
+ *
+ * Doesn't check that the user has an email address nor if they
+ * want to receive notification of faves. Maybe this happens higher
+ * up the stack...?
+ *
+ * @param User $other The user whose notice was faved
+ * @param User $user The user who faved the notice
+ * @param Notice $notice The notice that was faved
+ *
+ * @return void
+ */
+
+function mail_notify_fave($other, $user, $notice)
+{
+ $profile = $user->getProfile();
+
+ $bestname = $profile->getBestName();
- $profile = $user->getProfile();
- $bestname = $profile->getBestName();
common_init_locale($other->language);
- $subject = sprintf(_('%s added your notice as a favorite'), $bestname);
- $body = sprintf(_("%1\$s just added your notice from %2\$s as one of their favorites.\n\n" .
- "In case you forgot, you can see the text of your notice here:\n\n" .
- "%3\$s\n\n" .
- "You can see the list of %1\$s's favorites here:\n\n" .
- "%4\$s\n\n" .
- "Faithfully yours,\n" .
- "%5\$s\n"),
- $bestname,
- common_exact_date($notice->created),
- common_local_url('shownotice', array('notice' => $notice->id)),
- common_local_url('showfavorites', array('nickname' => $user->nickname)),
- common_config('site', 'name'));
+
+ $subject = sprintf(_('%s added your notice as a favorite'), $bestname);
+
+ $body = sprintf(_("%1\$s just added your notice from %2\$s".
+ " as one of their favorites.\n\n" .
+ "In case you forgot, you can see the text".
+ " of your notice here:\n\n" .
+ "%3\$s\n\n" .
+ "You can see the list of %1\$s's favorites here:\n\n" .
+ "%4\$s\n\n" .
+ "Faithfully yours,\n" .
+ "%5\$s\n"),
+ $bestname,
+ common_exact_date($notice->created),
+ common_local_url('shownotice',
+ array('notice' => $notice->id)),
+ common_local_url('showfavorites',
+ array('nickname' => $user->nickname)),
+ common_config('site', 'name'));
common_init_locale();
- mail_to_user($other, $subject, $body);
+ mail_to_user($other, $subject, $body);
}
diff --git a/_darcs/tentative_pristine b/_darcs/tentative_pristine
index 5a45154e6..8ce5499a9 100644
--- a/_darcs/tentative_pristine
+++ b/_darcs/tentative_pristine
@@ -1,365 +1,758 @@
-hunk ./actions/inbox.php 2
+hunk ./lib/mail.php 2
-/*
- * Laconica - a distributed open-source microblogging tool
- * Copyright (C) 2008, Controlez-Vous, Inc.
+/**
+ * Laconica, the distributed open-source microblogging tool
-hunk ./actions/inbox.php 5
+hunk ./lib/mail.php 5
- * This program is free software: you can redistribute it and/or modify
-+ * action handler for message inbox
++ * utilities for sending email
+ *
+ * PHP version 5
+ *
+ * LICENCE: This program is free software: you can redistribute it and/or modify
-hunk ./actions/inbox.php 21
+hunk ./lib/mail.php 21
+ *
-+ * @category Message
++ * @category Mail
+ * @package Laconica
+ * @author Evan Prodromou <evan@controlyourself.ca>
++ * @author Zach Copley <zach@controlyourself.ca>
++ * @author Robin Millette <millette@controlyourself.ca>
++ * @author Sarven Capadisli <csarven@controlyourself.ca>
+ * @copyright 2008 Control Yourself, Inc.
+ * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
+ * @link http://laconi.ca/
-+ */
-+
+hunk ./lib/mail.php 33
+-if (!defined('LACONICA')) { exit(1); }
+if (!defined('LACONICA')) {
+ exit(1);
+}
-+
-+require_once INSTALLDIR.'/lib/mailbox.php';
+hunk ./lib/mail.php 37
+-require_once('Mail.php');
++require_once 'Mail.php';
+
+/**
-+ * action handler for message inbox
-+ *
-+ * @category Message
-+ * @package Laconica
-+ * @author Evan Prodromou <evan@controlyourself.ca>
-+ * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
-+ * @link http://laconi.ca/
-+ * @see MailboxAction
-hunk ./actions/inbox.php 47
--if (!defined('LACONICA')) { exit(1); }
-+class InboxAction extends MailboxAction
++ * return the configured mail backend
++ *
++ * Uses the $config array to make a mail backend. Cached so it is safe to call
++ * more than once.
++ *
++ * @return Mail backend
++ */
+hunk ./lib/mail.php 48
+-function mail_backend() {
+- static $backend = NULL;
++function mail_backend()
+{
-+ /**
-+ * returns the title of the page
-+ *
-+ * @param User $user current user
-+ * @param int $page current page
-+ *
-+ * @return string localised title of the page
-+ *
-+ * @see MailboxAction::getTitle()
-+ */
-+
-+ function getTitle($user, $page)
-+ {
-+ if ($page > 1) {
-+ $title = sprintf(_("Inbox for %s - page %d"), $user->nickname, $page);
-+ } else {
-+ $title = sprintf(_("Inbox for %s"), $user->nickname);
-+ }
-+ return $title;
-+ }
-+
-+ /**
-+ * retrieve the messages for this user and this page
-+ *
-+ * Does a query for the right messages
-+ *
-+ * @param User $user The current user
-+ * @param int $page The page the user is on
-+ *
-+ * @return Message data object with stream for messages
-+ *
-+ * @see MailboxAction::getMessages()
-+ */
-+
-+ function getMessages($user, $page)
-+ {
-+ $message = new Message();
-+
-+ $message->to_profile = $user->id;
-+
-+ $message->orderBy('created DESC, id DESC');
-+ $message->limit((($page-1)*MESSAGES_PER_PAGE), MESSAGES_PER_PAGE + 1);
-+
-+ if ($message->find()) {
-+ return $message;
-+ } else {
-+ return null;
++ static $backend = null;
+hunk ./lib/mail.php 52
+- if (!$backend) {
+- global $config;
+- $backend = Mail::factory($config['mail']['backend'],
+- ($config['mail']['params']) ? $config['mail']['params'] : array());
+- if (PEAR::isError($backend)) {
+- common_server_error($backend->getMessage(), 500);
+- }
+- }
+- return $backend;
++ if (!$backend) {
++ global $config;
++ $backend = Mail::factory($config['mail']['backend'],
++ ($config['mail']['params']) ?
++ $config['mail']['params'] :
++ array());
++ if (PEAR::isError($backend)) {
++ common_server_error($backend->getMessage(), 500);
+ }
+ }
-+
-+ /**
-+ * returns the profile we want to show with the message
-+ *
-+ * For inboxes, we show the sender.
-+ *
-+ * @param Message $message The message to get the profile for
-+ *
-+ * @return Profile The profile of the message sender
-+ *
-+ * @see MailboxAction::getMessageProfile()
-+ */
-hunk ./actions/inbox.php 111
--require_once(INSTALLDIR.'/lib/mailbox.php');
-+ function getMessageProfile($message)
-+ {
-+ return $message->getFrom();
-+ }
-hunk ./actions/inbox.php 116
--class InboxAction extends MailboxAction {
--
-- function get_title($user, $page) {
-- if ($page > 1) {
-- $title = sprintf(_("Inbox for %s - page %d"), $user->nickname, $page);
-- } else {
-- $title = sprintf(_("Inbox for %s"), $user->nickname);
-- }
-- return $title;
++ return $backend;
+hunk ./lib/mail.php 65
+-# XXX: use Mail_Queue... maybe
++/**
++ * send an email to one or more recipients
++ *
++ * @param array $recipients array of strings with email addresses of recipients
++ * @param array $headers array mapping strings to strings for email headers
++ * @param string $body body of the email
++ *
++ * @return boolean success flag
++ */
+hunk ./lib/mail.php 75
+-function mail_send($recipients, $headers, $body) {
+- $backend = mail_backend();
++function mail_send($recipients, $headers, $body)
++{
++ // XXX: use Mail_Queue... maybe
++ $backend = mail_backend();
+hunk ./lib/mail.php 82
+- assert($backend); # throws an error if it's bad
+- $sent = $backend->send($recipients, $headers, $body);
+- if (PEAR::isError($sent)) {
+- common_log(LOG_ERR, 'Email error: ' . $sent->getMessage());
+- return false;
- }
--
-- function get_messages($user, $page) {
-- $message = new Message();
-- $message->to_profile = $user->id;
-- $message->orderBy('created DESC, id DESC');
-- $message->limit((($page-1)*MESSAGES_PER_PAGE), MESSAGES_PER_PAGE + 1);
-+ /**
-+ * instructions for using this page
-+ *
-+ * @return string localised instructions for using the page
-+ */
-hunk ./actions/inbox.php 122
-- if ($message->find()) {
-- return $message;
-- } else {
-- return NULL;
-- }
+- return true;
++ assert($backend); // throws an error if it's bad
++ $sent = $backend->send($recipients, $headers, $body);
++ if (PEAR::isError($sent)) {
++ common_log(LOG_ERR, 'Email error: ' . $sent->getMessage());
++ return false;
++ }
++ return true;
+hunk ./lib/mail.php 91
+-function mail_domain() {
+- $maildomain = common_config('mail', 'domain');
+- if (!$maildomain) {
+- $maildomain = common_config('site', 'server');
- }
--
-- function get_message_profile($message) {
-- return $message->getFrom();
+- return $maildomain;
+-}
++/**
++ * returns the configured mail domain
++ *
++ * Defaults to the server name.
++ *
++ * @return string mail domain, suitable for making email addresses.
++ */
+hunk ./lib/mail.php 99
+-function mail_notify_from() {
+- $notifyfrom = common_config('mail', 'notifyfrom');
+- if (!$notifyfrom) {
+- $domain = mail_domain();
+- $notifyfrom = common_config('site', 'name') .' <noreply@'.$domain.'>';
- }
--
-- function get_instructions() {
-- return _('This is your inbox, which lists your incoming private messages.');
+- return $notifyfrom;
++function mail_domain()
++{
++ $maildomain = common_config('mail', 'domain');
++ if (!$maildomain) {
++ $maildomain = common_config('site', 'server');
++ }
++ return $maildomain;
+hunk ./lib/mail.php 108
+-function mail_to_user(&$user, $subject, $body, $address=NULL) {
+- if (!$address) {
+- $address = $user->email;
- }
-+ function getInstructions()
-+ {
-+ return _('This is your inbox, which lists your incoming private messages.');
++/**
++ * returns a good address for sending email from this server
++ *
++ * Uses either the configured value or a faked-up value made
++ * from the mail domain.
++ *
++ * @return string notify from address
++ */
+hunk ./lib/mail.php 117
+- $recipients = $address;
+- $profile = $user->getProfile();
++function mail_notify_from()
++{
++ $notifyfrom = common_config('mail', 'notifyfrom');
+hunk ./lib/mail.php 121
+- $headers['From'] = mail_notify_from();
+- $headers['To'] = $profile->getBestName() . ' <' . $address . '>';
+- $headers['Subject'] = $subject;
++ if (!$notifyfrom) {
+hunk ./lib/mail.php 123
+- return mail_send($recipients, $headers, $body);
++ $domain = mail_domain();
++
++ $notifyfrom = common_config('site', 'name') .' <noreply@'.$domain.'>';
+ }
-hunk ./actions/outbox.php 2
--/*
-- * Laconica - a distributed open-source microblogging tool
-- * Copyright (C) 2008, Controlez-Vous, Inc.
++
++ return $notifyfrom;
+hunk ./lib/mail.php 131
+-# For confirming a Jabber address
+/**
-+ * Laconica, the distributed open-source microblogging tool
-hunk ./actions/outbox.php 5
-- * This program is free software: you can redistribute it and/or modify
-+ * action handler for message inbox
++ * sends email to a user
+ *
-+ * PHP version 5
++ * @param User &$user user to send email to
++ * @param string $subject subject of the email
++ * @param string $body body of the email
++ * @param string $address optional specification of email address
+ *
-+ * LICENCE: This program is free software: you can redistribute it and/or modify
-hunk ./actions/outbox.php 21
++ * @return boolean success flag
++ */
++
++function mail_to_user(&$user, $subject, $body, $address=null)
++{
++ if (!$address) {
++ $address = $user->email;
++ }
+hunk ./lib/mail.php 148
+-function mail_confirm_address($user, $code, $nickname, $address) {
++ $recipients = $address;
++ $profile = $user->getProfile();
+hunk ./lib/mail.php 151
+- $subject = _('Email address confirmation');
++ $headers['From'] = mail_notify_from();
++ $headers['To'] = $profile->getBestName() . ' <' . $address . '>';
++ $headers['Subject'] = $subject;
+hunk ./lib/mail.php 155
+- $body = sprintf(_("Hey, %s.\n\nSomeone just entered this email address on %s.\n\n" .
+- "If it was you, and you want to confirm your entry, use the URL below:\n\n\t%s\n\n" .
+- "If not, just ignore this message.\n\nThanks for your time, \n%s\n")
+- , $nickname, common_config('site', 'name')
+- , common_local_url('confirmaddress', array('code' => $code)), common_config('site', 'name'));
+- return mail_to_user($user, $subject, $body, $address);
++ return mail_send($recipients, $headers, $body);
+hunk ./lib/mail.php 158
+-function mail_subscribe_notify($listenee, $listener) {
+- $other = $listener->getProfile();
+- mail_subscribe_notify_profile($listenee, $other);
++/**
++ * Send an email to confirm a user's control of an email address
+ *
-+ * @category Message
-+ * @package Laconica
-+ * @author Evan Prodromou <evan@controlyourself.ca>
-+ * @copyright 2008 Control Yourself, Inc.
-+ * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
-+ * @link http://laconi.ca/
++ * @param User $user User claiming the email address
++ * @param string $code Confirmation code
++ * @param string $nickname Nickname of user
++ * @param string $address email address to confirm
++ *
++ * @see common_confirmation_code()
++ *
++ * @return success flag
+ */
+
-+if (!defined('LACONICA')) {
-+ exit(1);
-+}
++function mail_confirm_address($user, $code, $nickname, $address)
++{
++ $subject = _('Email address confirmation');
+
-+require_once INSTALLDIR.'/lib/mailbox.php';
++ $body = sprintf(_("Hey, %s.\n\n".
++ "Someone just entered this email address on %s.\n\n" .
++ "If it was you, and you want to confirm your entry, ".
++ "use the URL below:\n\n\t%s\n\n" .
++ "If not, just ignore this message.\n\n".
++ "Thanks for your time, \n%s\n"),
++ $nickname, common_config('site', 'name'),
++ common_local_url('confirmaddress', array('code' => $code)),
++ common_config('site', 'name'));
++ return mail_to_user($user, $subject, $body, $address);
+hunk ./lib/mail.php 187
+-function mail_subscribe_notify_profile($listenee, $other) {
+- if ($listenee->email && $listenee->emailnotifysub) {
+- // use the recipients localization
++/**
++ * notify a user of subscription by another user
++ *
++ * This is just a wrapper around the profile-based version.
++ *
++ * @param User $listenee user who is being subscribed to
++ * @param User $listener user who is subscribing
++ *
++ * @see mail_subscribe_notify_profile()
++ *
++ * @return void
++ */
++
++function mail_subscribe_notify($listenee, $listener)
++{
++ $other = $listener->getProfile();
++ mail_subscribe_notify_profile($listenee, $other);
++}
+
+/**
-+ * action handler for message outbox
-+ *
-+ * @category Message
-+ * @package Laconica
-+ * @author Evan Prodromou <evan@controlyourself.ca>
-+ * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0
-+ * @link http://laconi.ca/
-+ * @see MailboxAction
-hunk ./actions/outbox.php 47
--if (!defined('LACONICA')) { exit(1); }
-+class OutboxAction extends MailboxAction
++ * notify a user of subscription by a profile (remote or local)
++ *
++ * This function checks to see if the listenee has an email
++ * address and wants subscription notices.
++ *
++ * @param User $listenee user who's being subscribed to
++ * @param Profile $other profile of person who's listening
++ *
++ * @return void
++ */
++
++function mail_subscribe_notify_profile($listenee, $other)
+{
-+ /**
-+ * returns the title of the page
-+ *
-+ * @param User $user current user
-+ * @param int $page current page
-+ *
-+ * @return string localised title of the page
-+ *
-+ * @see MailboxAction::getTitle()
-+ */
++ if ($listenee->email && $listenee->emailnotifysub) {
+
-+ function getTitle($user, $page)
-+ {
-+ if ($page > 1) {
-+ $title = sprintf(_("Outbox for %s - page %d"), $user->nickname, $page);
-+ } else {
-+ $title = sprintf(_("Outbox for %s"), $user->nickname);
-+ }
-+ return $title;
-+ }
++ // use the recipient's localization
+hunk ./lib/mail.php 224
+- $profile = $listenee->getProfile();
+- $name = $profile->getBestName();
+- $long_name = ($other->fullname) ? ($other->fullname . ' (' . $other->nickname . ')') : $other->nickname;
+- $recipients = $listenee->email;
+- $headers['From'] = mail_notify_from();
+- $headers['To'] = $name . ' <' . $listenee->email . '>';
+- $headers['Subject'] = sprintf(_('%1$s is now listening to your notices on %2$s.'),
+
-+ /**
-+ * retrieve the messages for this user and this page
-+ *
-+ * Does a query for the right messages
-+ *
-+ * @param User $user The current user
-+ * @param int $page The page the user is on
-+ *
-+ * @return Message data object with stream for messages
-+ *
-+ * @see MailboxAction::getMessages()
-+ */
++ $profile = $listenee->getProfile();
+
-+ function getMessages($user, $page)
-+ {
-+ $message = new Message();
++ $name = $profile->getBestName();
+
-+ $message->from_profile = $user->id;
-+ $message->orderBy('created DESC, id DESC');
-+ $message->limit((($page-1)*MESSAGES_PER_PAGE), MESSAGES_PER_PAGE + 1);
++ $long_name = ($other->fullname) ?
++ ($other->fullname . ' (' . $other->nickname . ')') : $other->nickname;
+
-+ if ($message->find()) {
-+ return $message;
-+ } else {
-+ return null;
-+ }
-+ }
++ $recipients = $listenee->email;
+
-+ /**
-+ * returns the profile we want to show with the message
-+ *
-+ * For outboxes, we show the recipient.
-+ *
-+ * @param Message $message The message to get the profile for
-+ *
-+ * @return Profile The profile of the message recipient
-+ *
-+ * @see MailboxAction::getMessageProfile()
-+ */
-hunk ./actions/outbox.php 110
--require_once(INSTALLDIR.'/lib/mailbox.php');
-+ function getMessageProfile($message)
-+ {
-+ return $message->getTo();
++ $headers['From'] = mail_notify_from();
++ $headers['To'] = $name . ' <' . $listenee->email . '>';
++ $headers['Subject'] = sprintf(_('%1$s is now listening to '.
++ 'your notices on %2$s.'),
+hunk ./lib/mail.php 239
+- common_config('site', 'name'));
+- $body = sprintf(_('%1$s is now listening to your notices on %2$s.'."\n\n".
+- "\t".'%3$s'."\n\n".
+- '%4$s'.
+- '%5$s'.
+- '%6$s'.
+- "\n".'Faithfully yours,'."\n".'%7$s.'."\n\n".
+- "----\n".
+- "Change your email address or notification options at %8$s"),
+- $long_name,
+- common_config('site', 'name'),
+- $other->profileurl,
+- ($other->location) ? sprintf(_("Location: %s\n"), $other->location) : '',
+- ($other->homepage) ? sprintf(_("Homepage: %s\n"), $other->homepage) : '',
+- ($other->bio) ? sprintf(_("Bio: %s\n\n"), $other->bio) : '',
+- common_config('site', 'name'),
+- common_local_url('emailsettings'));
++ common_config('site', 'name'));
++
++ $body = sprintf(_('%1$s is now listening to your notices on %2$s.'."\n\n".
++ "\t".'%3$s'."\n\n".
++ '%4$s'.
++ '%5$s'.
++ '%6$s'.
++ "\n".'Faithfully yours,'."\n".'%7$s.'."\n\n".
++ "----\n".
++ "Change your email address or ".
++ "notification options at %8$s\n"),
++ $long_name,
++ common_config('site', 'name'),
++ $other->profileurl,
++ ($other->location) ?
++ sprintf(_("Location: %s\n"), $other->location) : '',
++ ($other->homepage) ?
++ sprintf(_("Homepage: %s\n"), $other->homepage) : '',
++ ($other->bio) ?
++ sprintf(_("Bio: %s\n\n"), $other->bio) : '',
++ common_config('site', 'name'),
++ common_local_url('emailsettings'));
++
+hunk ./lib/mail.php 264
+- mail_send($recipients, $headers, $body);
+- }
++ mail_send($recipients, $headers, $body);
+ }
-hunk ./actions/outbox.php 115
--class OutboxAction extends MailboxAction {
--
-- function get_title($user, $page) {
-- if ($page > 1) {
-- $title = sprintf(_("Outbox for %s - page %d"), $user->nickname, $page);
-- } else {
-- $title = sprintf(_("Outbox for %s"), $user->nickname);
+hunk ./lib/mail.php 268
+-function mail_new_incoming_notify($user) {
++/**
++ * notify a user of their new incoming email address
++ *
++ * User's email and incoming fields should already be updated.
++ *
++ * @param User $user user with the new address
++ *
++ * @return void
++ */
++
++function mail_new_incoming_notify($user)
++{
++ $profile = $user->getProfile();
+hunk ./lib/mail.php 282
+- $profile = $user->getProfile();
+- $name = $profile->getBestName();
++ $name = $profile->getBestName();
+hunk ./lib/mail.php 284
+- $headers['From'] = $user->incomingemail;
+- $headers['To'] = $name . ' <' . $user->email . '>';
+- $headers['Subject'] = sprintf(_('New email address for posting to %s'),
+- common_config('site', 'name'));
++ $headers['From'] = $user->incomingemail;
++ $headers['To'] = $name . ' <' . $user->email . '>';
++ $headers['Subject'] = sprintf(_('New email address for posting to %s'),
++ common_config('site', 'name'));
+hunk ./lib/mail.php 289
+- $body = sprintf(_("You have a new posting address on %1\$s.\n\n".
+- "Send email to %2\$s to post new messages.\n\n".
+- "More email instructions at %3\$s.\n\n".
+- "Faithfully yours,\n%4\$s"),
+- common_config('site', 'name'),
+- $user->incomingemail,
+- common_local_url('doc', array('title' => 'email')),
+- common_config('site', 'name'));
++ $body = sprintf(_("You have a new posting address on %1\$s.\n\n".
++ "Send email to %2\$s to post new messages.\n\n".
++ "More email instructions at %3\$s.\n\n".
++ "Faithfully yours,\n%4\$s"),
++ common_config('site', 'name'),
++ $user->incomingemail,
++ common_local_url('doc', array('title' => 'email')),
++ common_config('site', 'name'));
+hunk ./lib/mail.php 298
+- mail_send($user->email, $headers, $body);
++ mail_send($user->email, $headers, $body);
+hunk ./lib/mail.php 301
+-function mail_new_incoming_address() {
+- $prefix = common_confirmation_code(64);
+- $suffix = mail_domain();
+- return $prefix . '@' . $suffix;
++/**
++ * generate a new address for incoming messages
++ *
++ * @todo check the database for uniqueness
++ *
++ * @return string new email address for incoming messages
++ */
++
++function mail_new_incoming_address()
++{
++ $prefix = common_confirmation_code(64);
++ $suffix = mail_domain();
++ return $prefix . '@' . $suffix;
+hunk ./lib/mail.php 316
+-function mail_broadcast_notice_sms($notice) {
++/**
++ * broadcast a notice to all subscribers with SMS notification on
++ *
++ * This function sends SMS messages to all users who have sms addresses;
++ * have sms notification on; and have sms enabled for this particular
++ * subscription.
++ *
++ * @param Notice $notice The notice to broadcast
++ *
++ * @return success flag
++ */
+hunk ./lib/mail.php 328
+- # Now, get users subscribed to this profile
++function mail_broadcast_notice_sms($notice)
++{
++ // Now, get users subscribed to this profile
+hunk ./lib/mail.php 332
+- $user = new User();
++ $user = new User();
+hunk ./lib/mail.php 334
+- $user->query('SELECT nickname, smsemail, incomingemail ' .
+- 'FROM user JOIN subscription ' .
+- 'ON user.id = subscription.subscriber ' .
+- 'WHERE subscription.subscribed = ' . $notice->profile_id . ' ' .
+- 'AND user.smsemail IS NOT NULL ' .
+- 'AND user.smsnotify = 1 ' .
++ $user->query('SELECT nickname, smsemail, incomingemail ' .
++ 'FROM user JOIN subscription ' .
++ 'ON user.id = subscription.subscriber ' .
++ 'WHERE subscription.subscribed = ' . $notice->profile_id . ' ' .
++ 'AND user.smsemail IS NOT null ' .
++ 'AND user.smsnotify = 1 ' .
+hunk ./lib/mail.php 342
+- while ($user->fetch()) {
+- common_log(LOG_INFO,
+- 'Sending notice ' . $notice->id . ' to ' . $user->smsemail,
+- __FILE__);
+- $success = mail_send_sms_notice_address($notice, $user->smsemail, $user->incomingemail);
+- if (!$success) {
+- # XXX: Not sure, but I think that's the right thing to do
+- common_log(LOG_WARNING,
+- 'Sending notice ' . $notice->id . ' to ' . $user->smsemail . ' FAILED, cancelling.',
+- __FILE__);
+- return false;
- }
-- return $title;
- }
--
-- function get_messages($user, $page) {
-- $message = new Message();
-- $message->from_profile = $user->id;
-- $message->orderBy('created DESC, id DESC');
-- $message->limit((($page-1)*MESSAGES_PER_PAGE), MESSAGES_PER_PAGE + 1);
-+ /**
-+ * instructions for using this page
-+ *
-+ * @return string localised instructions for using the page
-+ */
-hunk ./actions/outbox.php 121
-- if ($message->find()) {
-- return $message;
-- } else {
-- return NULL;
-- }
++ while ($user->fetch()) {
++ common_log(LOG_INFO,
++ 'Sending notice ' . $notice->id . ' to ' . $user->smsemail,
++ __FILE__);
++ $success = mail_send_sms_notice_address($notice,
++ $user->smsemail,
++ $user->incomingemail);
++ if (!$success) {
++ // XXX: Not sure, but I think that's the right thing to do
++ common_log(LOG_WARNING,
++ 'Sending notice ' . $notice->id . ' to ' .
++ $user->smsemail . ' FAILED, cancelling.',
++ __FILE__);
++ return false;
++ }
++ }
+hunk ./lib/mail.php 359
+- $user->free();
+- unset($user);
++ $user->free();
++ unset($user);
+hunk ./lib/mail.php 362
+- return true;
++ return true;
+hunk ./lib/mail.php 365
+-function mail_send_sms_notice($notice, $user) {
+- return mail_send_sms_notice_address($notice, $user->smsemail, $user->incomingemail);
++/**
++ * send a notice to a user via SMS
++ *
++ * A convenience wrapper around mail_send_sms_notice_address()
++ *
++ * @param Notice $notice notice to send
++ * @param User $user user to receive notice
++ *
++ * @see mail_send_sms_notice_address()
++ *
++ * @return boolean success flag
++ */
++
++function mail_send_sms_notice($notice, $user)
++{
++ return mail_send_sms_notice_address($notice,
++ $user->smsemail,
++ $user->incomingemail);
+hunk ./lib/mail.php 385
+-function mail_send_sms_notice_address($notice, $smsemail, $incomingemail) {
++/**
++ * send a notice to an SMS email address from a given address
++ *
++ * We use the user's incoming email address as the "From" address to make
++ * replying to notices easier.
++ *
++ * @param Notice $notice notice to send
++ * @param string $smsemail email address to send to
++ * @param string $incomingemail email address to set as 'from'
++ *
++ * @return boolean success flag
++ */
++
++function mail_send_sms_notice_address($notice, $smsemail, $incomingemail)
++{
++ $to = $nickname . ' <' . $smsemail . '>';
++
++ $other = $notice->getProfile();
+hunk ./lib/mail.php 404
+- $to = $nickname . ' <' . $smsemail . '>';
+- $other = $notice->getProfile();
++ common_log(LOG_INFO, 'Sending notice ' . $notice->id .
++ ' to ' . $smsemail, __FILE__);
+hunk ./lib/mail.php 407
+- common_log(LOG_INFO, "Sending notice " . $notice->id . " to " . $smsemail, __FILE__);
++ $headers = array();
+hunk ./lib/mail.php 409
+- $headers = array();
+- $headers['From'] = (isset($incomingemail)) ? $incomingemail : mail_notify_from();
+- $headers['To'] = $to;
+- $headers['Subject'] = sprintf(_('%s status'),
+- $other->getBestName());
+- $body = $notice->content;
++ $headers['From'] = ($incomingemail) ? $incomingemail : mail_notify_from();
++ $headers['To'] = $to;
++ $headers['Subject'] = sprintf(_('%s status'),
++ $other->getBestName());
+hunk ./lib/mail.php 414
+- return mail_send($smsemail, $headers, $body);
++ $body = $notice->content;
++
++ return mail_send($smsemail, $headers, $body);
+hunk ./lib/mail.php 419
+-function mail_confirm_sms($code, $nickname, $address) {
++/**
++ * send a message to confirm a claim for an SMS number
++ *
++ * @param string $code confirmation code
++ * @param string $nickname nickname of user claiming number
++ * @param string $address email address to send the confirmation to
++ *
++ * @see common_confirmation_code()
++ *
++ * @return void
++ */
++
++function mail_confirm_sms($code, $nickname, $address)
++{
++ $recipients = $address;
+hunk ./lib/mail.php 435
+- $recipients = $address;
++ $headers['From'] = mail_notify_from();
++ $headers['To'] = $nickname . ' <' . $address . '>';
++ $headers['Subject'] = _('SMS confirmation');
+hunk ./lib/mail.php 439
+- $headers['From'] = mail_notify_from();
+- $headers['To'] = $nickname . ' <' . $address . '>';
+- $headers['Subject'] = _('SMS confirmation');
++ // FIXME: I18N
+hunk ./lib/mail.php 441
+- $body = "$nickname: confirm you own this phone number with this code:";
+- $body .= "\n\n";
+- $body .= $code;
+- $body .= "\n\n";
++ $body = "$nickname: confirm you own this phone number with this code:";
++ $body .= "\n\n";
++ $body .= $code;
++ $body .= "\n\n";
+hunk ./lib/mail.php 446
+- mail_send($recipients, $headers, $body);
++ mail_send($recipients, $headers, $body);
+hunk ./lib/mail.php 449
+-function mail_notify_nudge($from, $to) {
++/**
++ * send a mail message to notify a user of a 'nudge'
++ *
++ * @param User $from user nudging
++ * @param User $to user being nudged
++ *
++ * @return boolean success flag
++ */
++
++function mail_notify_nudge($from, $to)
++{
+hunk ./lib/mail.php 461
+- $subject = sprintf(_('You\'ve been nudged by %s'), $from->nickname);
++ $subject = sprintf(_('You\'ve been nudged by %s'), $from->nickname);
+hunk ./lib/mail.php 463
+- $from_profile = $from->getProfile();
++ $from_profile = $from->getProfile();
+hunk ./lib/mail.php 465
+- $body = sprintf(_("%1\$s (%2\$s) is wondering what you are up to these days and is inviting you to post some news.\n\n".
+- "So let's hear from you :)\n\n".
+- "%3\$s\n\n".
+- "Don't reply to this email; it won't get to them.\n\n".
+- "With kind regards,\n".
+- "%4\$s\n"),
+- $from_profile->getBestName(),
+- $from->nickname,
+- common_local_url('all', array('nickname' => $to->nickname)),
+- common_config('site', 'name'));
++ $body = sprintf(_("%1\$s (%2\$s) is wondering what you are up to ".
++ "these days and is inviting you to post some news.\n\n".
++ "So let's hear from you :)\n\n".
++ "%3\$s\n\n".
++ "Don't reply to this email; it won't get to them.\n\n".
++ "With kind regards,\n".
++ "%4\$s\n"),
++ $from_profile->getBestName(),
++ $from->nickname,
++ common_local_url('all', array('nickname' => $to->nickname)),
++ common_config('site', 'name'));
+hunk ./lib/mail.php 477
+- return mail_to_user($to, $subject, $body);
++ return mail_to_user($to, $subject, $body);
+hunk ./lib/mail.php 480
+-function mail_notify_message($message, $from=NULL, $to=NULL) {
++/**
++ * send a message to notify a user of a direct message (DM)
++ *
++ * This function checks to see if the recipient wants notification
++ * of DMs and has a configured email address.
++ *
++ * @param Message $message message to notify about
++ * @param User $from user sending message; default to sender
++ * @param User $to user receiving message; default to recipient
++ *
++ * @return boolean success code
++ */
+hunk ./lib/mail.php 493
+- if (is_null($from)) {
+- $from = User::staticGet('id', $message->from_profile);
- }
--
-- function get_message_profile($message) {
-- return $message->getTo();
++function mail_notify_message($message, $from=null, $to=null)
++{
++ if (is_null($from)) {
++ $from = User::staticGet('id', $message->from_profile);
++ }
+hunk ./lib/mail.php 499
+- if (is_null($to)) {
+- $to = User::staticGet('id', $message->to_profile);
- }
--
-- function get_instructions() {
-- return _('This is your outbox, which lists private messages you have sent.');
++ if (is_null($to)) {
++ $to = User::staticGet('id', $message->to_profile);
++ }
+hunk ./lib/mail.php 503
+- if (is_null($to->email) || !$to->emailnotifymsg) {
+- return true;
- }
--
-+ function getInstructions()
-+ {
-+ return _('This is your outbox, which lists private messages you have sent.');
++ if (is_null($to->email) || !$to->emailnotifymsg) {
++ return true;
+ }
-hunk ./lib/mailbox.php 22
-- * @category Action
-+ * @category Message
-hunk ./lib/mailbox.php 41
-- * @category Action
-+ * @category Message
-hunk ./lib/mailbox.php 94
-- $this->show_page($user, $page);
-+ $this->showPage($user, $page);
-hunk ./lib/mailbox.php 106
-- function get_title($user, $page)
-+ function getTitle($user, $page)
-hunk ./lib/mailbox.php 117
-- function get_instructions()
-+ function getInstructions()
-hunk ./lib/mailbox.php 128
-- function show_top()
-+ function showTop()
-hunk ./lib/mailbox.php 146
-- function show_page($user, $page)
-+ function showPage($user, $page)
-hunk ./lib/mailbox.php 148
-- common_show_header($this->get_title($user, $page),
-+ common_show_header($this->getTitle($user, $page),
-hunk ./lib/mailbox.php 150
-- array($this, 'show_top'));
-+ array($this, 'showTop'));
-hunk ./lib/mailbox.php 152
-- $this->show_box($user, $page);
-+ $this->showBox($user, $page);
-hunk ./lib/mailbox.php 157
-+ /**
-+ * retrieve the messages appropriate for this mailbox
-+ *
-+ * Does a query for the right messages
-+ *
-+ * @param User $user The current user
-+ * @param int $page The page the user is on
-+ *
-+ * @return Message data object with stream for messages
-+ */
+hunk ./lib/mail.php 508
+- $subject = sprintf(_('New private message from %s'), $from->nickname);
++ $subject = sprintf(_('New private message from %s'), $from->nickname);
+hunk ./lib/mail.php 510
+- $from_profile = $from->getProfile();
++ $from_profile = $from->getProfile();
+hunk ./lib/mail.php 512
+- $body = sprintf(_("%1\$s (%2\$s) sent you a private message:\n\n".
+- "------------------------------------------------------\n".
+- "%3\$s\n".
+- "------------------------------------------------------\n\n".
+- "You can reply to their message here:\n\n".
+- "%4\$s\n\n".
+- "Don't reply to this email; it won't get to them.\n\n".
+- "With kind regards,\n".
+- "%5\$s\n"),
+- $from_profile->getBestName(),
+- $from->nickname,
+- $message->content,
+- common_local_url('newmessage', array('to' => $from->id)),
+- common_config('site', 'name'));
++ $body = sprintf(_("%1\$s (%2\$s) sent you a private message:\n\n".
++ "------------------------------------------------------\n".
++ "%3\$s\n".
++ "------------------------------------------------------\n\n".
++ "You can reply to their message here:\n\n".
++ "%4\$s\n\n".
++ "Don't reply to this email; it won't get to them.\n\n".
++ "With kind regards,\n".
++ "%5\$s\n"),
++ $from_profile->getBestName(),
++ $from->nickname,
++ $message->content,
++ common_local_url('newmessage', array('to' => $from->id)),
++ common_config('site', 'name'));
+hunk ./lib/mail.php 528
+- return mail_to_user($to, $subject, $body);
++ return mail_to_user($to, $subject, $body);
+hunk ./lib/mail.php 531
+-function mail_notify_fave($other, $user, $notice) {
++/**
++ * notify a user that one of their notices has been chosen as a 'fave'
++ *
++ * Doesn't check that the user has an email address nor if they
++ * want to receive notification of faves. Maybe this happens higher
++ * up the stack...?
++ *
++ * @param User $other The user whose notice was faved
++ * @param User $user The user who faved the notice
++ * @param Notice $notice The notice that was faved
++ *
++ * @return void
++ */
+
-+ function getMessages($user, $page)
-+ {
-+ return null;
-+ }
++function mail_notify_fave($other, $user, $notice)
++{
++ $profile = $user->getProfile();
++
++ $bestname = $profile->getBestName();
+hunk ./lib/mail.php 551
+- $profile = $user->getProfile();
+- $bestname = $profile->getBestName();
+hunk ./lib/mail.php 552
+- $subject = sprintf(_('%s added your notice as a favorite'), $bestname);
+- $body = sprintf(_("%1\$s just added your notice from %2\$s as one of their favorites.\n\n" .
+- "In case you forgot, you can see the text of your notice here:\n\n" .
+- "%3\$s\n\n" .
+- "You can see the list of %1\$s's favorites here:\n\n" .
+- "%4\$s\n\n" .
+- "Faithfully yours,\n" .
+- "%5\$s\n"),
+- $bestname,
+- common_exact_date($notice->created),
+- common_local_url('shownotice', array('notice' => $notice->id)),
+- common_local_url('showfavorites', array('nickname' => $user->nickname)),
+- common_config('site', 'name'));
++
++ $subject = sprintf(_('%s added your notice as a favorite'), $bestname);
+
-hunk ./lib/mailbox.php 184
-- function show_box($user, $page)
-+ function showBox($user, $page)
-hunk ./lib/mailbox.php 186
-- $message = $this->get_messages($user, $page);
-+ $message = $this->getMessages($user, $page);
-hunk ./lib/mailbox.php 200
-- $this->show_message($message);
-+ $this->showMessage($message);
-hunk ./lib/mailbox.php 224
-- function get_message_profile($message)
-+ function getMessageProfile($message)
-hunk ./lib/mailbox.php 237
-- function show_message($message)
-+ function showMessage($message)
-hunk ./lib/mailbox.php 242
-- $profile = $this->get_message_profile($message);
-+ $profile = $this->getMessageProfile($message);
++ $body = sprintf(_("%1\$s just added your notice from %2\$s".
++ " as one of their favorites.\n\n" .
++ "In case you forgot, you can see the text".
++ " of your notice here:\n\n" .
++ "%3\$s\n\n" .
++ "You can see the list of %1\$s's favorites here:\n\n" .
++ "%4\$s\n\n" .
++ "Faithfully yours,\n" .
++ "%5\$s\n"),
++ $bestname,
++ common_exact_date($notice->created),
++ common_local_url('shownotice',
++ array('notice' => $notice->id)),
++ common_local_url('showfavorites',
++ array('nickname' => $user->nickname)),
++ common_config('site', 'name'));
+hunk ./lib/mail.php 573
+- mail_to_user($other, $subject, $body);
++ mail_to_user($other, $subject, $body);