From 2d0aec255f3cc49e246d972a3482f444c5a44980 Mon Sep 17 00:00:00 2001 From: Evan Prodromou Date: Mon, 22 Dec 2008 15:13:04 -0500 Subject: [PATCH] bring messaging section (inbox, outbox, mailbox) into PEAR Code Standards compliance Actually refactored the method names on these classes to come into complete compliance with the code standards. Untested; maybe there are some bad method names now. darcs-hash:20081222201304-84dde-3effcce0f9ef25fbe0bfebd505037d9ff37252fa.gz --- _darcs/inventory | 10 +- ...effcce0f9ef25fbe0bfebd505037d9ff37252fa.gz | Bin 0 -> 1931 bytes _darcs/pristine/actions/inbox.php | 149 +++-- _darcs/pristine/actions/outbox.php | 149 +++-- _darcs/pristine/lib/mailbox.php | 48 +- _darcs/tentative_pristine | 604 +++++++++--------- actions/inbox.php | 149 +++-- actions/outbox.php | 149 +++-- lib/mailbox.php | 48 +- 9 files changed, 799 insertions(+), 507 deletions(-) create mode 100644 _darcs/patches/20081222201304-84dde-3effcce0f9ef25fbe0bfebd505037d9ff37252fa.gz diff --git a/_darcs/inventory b/_darcs/inventory index 7f21ac58a1..083e53fd01 100644 --- a/_darcs/inventory +++ b/_darcs/inventory @@ -35,4 +35,12 @@ Evan Prodromou **20081222173249] [reformat lib/language.php for PEAR Coding Standards Evan Prodromou **20081222193029] [bring mailbox.php into line with PEAR Coding Standards (mostly) -Evan Prodromou **20081222195041] \ No newline at end of file +Evan Prodromou **20081222195041] +[bring messaging section (inbox, outbox, mailbox) into PEAR Code Standards compliance +Evan Prodromou **20081222201304 + + Actually refactored the method names on these classes to come into + complete compliance with the code standards. Untested; maybe there are + some bad method names now. + +] \ No newline at end of file diff --git a/_darcs/patches/20081222201304-84dde-3effcce0f9ef25fbe0bfebd505037d9ff37252fa.gz b/_darcs/patches/20081222201304-84dde-3effcce0f9ef25fbe0bfebd505037d9ff37252fa.gz new file mode 100644 index 0000000000000000000000000000000000000000..60517eb793febf9c050c43f5f532bf91081db71a GIT binary patch literal 1931 zcmV;62Xy!!iwFP!000001MOOEZ`(Ey{#^Zv+t9&s;@32NS+WI9)2s`SW=7)<7>2>1 zCEDgHi!MdQ%ZC2<9Z5-+EYfk>bgvN%N0yFv$9tYTl85{}7A&5^oF)mGs=I^+lJOXP z7LWOFE#Rrtf9Hfnc+&(HOAdq6<2P`^LkgFY#32!30s)^dB1YnXdZ$+;hJoOr;B%hB zHxz$AU*x|#F`3h&ot?d2?^%BjuU`LAZ{GtCjsuyJD7pqgCnS(u&=BN|qI+`2Lr~)q zK|$J&$Oj3t2RO$C_)-cAI?Ao7V^VxSJ-8W9c5}{OsiP;^g$; zZX9yM*DowS4~V2wF0KJy8%NFK1qt3Muc3&!c}`NKionNU&49qbhyIC#Mw3oJjxsX^ z)(YnwX80)+%z;*-5esOXDAO}37YE($a=GkGl1Yb)Y1h0&T(dixbjfrPwI6kQotd0R z@ap0%98V@xaCk*yDo6x_bd17qp6SiQjze;T7$<#Y1|vOXbiS(u@XOm}6Y#$X>4e2J z^c&~LCqG=Aog6osO-z$rHI)9wq~C8Idb=N#yrAz>Cg_N(X`tWZ zZdPg(7>z?Eo)dg){;Y7b!{$KkcAynV{MA5Mae(Iur3R@AL5f4bj61BSA?2w9UWtImAQ!0mQpx?~J zbQ-9;NgA5~6*77jE#qY-pG8Ss3IQ{LBj`8FZ`AeB`1dU&3v^sg{E`2qMnRMRs{|Jo zJ@%{c%Vx{Ot9^vbey3Ke+~(m*=Let>a_g2>EV?U7SGQT1jlxXiLmrwnQV2$`XtuoT zBu%tAsfxP>9@E_Xvf}cR;{XEhQ;KPB22wCZG}pHUxrQ^kEy0EvL$o11n5oT(lWSS& zfsjZ7eEchJtjtCPNk~z3seD;HwX-hMncP6ULToel9_H2aTZrj0ci3+p<~qwl?IX!Y zxadxprb@;K7CMBv2&s5+?Kc9PtlERTJiR<=0W&v^GJjTsA~wgGFIK856TU;5=%P(=!-u2wC^aC#`b$N}Me>|}1<1pd5oH}|QgICF z=%n%>lNpxevznP}G~V2j$C=(To1M#5zu))T<&A^CapRB^%#s$l)jTrw#8{(7`v?br zhnoVwS=6)*em?QqTdtRAv;6=+WhY0nzG`T>2eqXFpFOyv6#`WitlvhheMwMS)YsZF z(AH1a=Q+0)DP!MVnMyC+6{#Y0b0w;Xe_9P%F99sZYM&aC=+LE!K6W{mHRn#nsj0ee z*_1(hdj*Xw#ZkdttBLEp$OJClo}U+RZvZ!kXO(M4&!z;kD=bgnuWA|j4SgcSkJ_a} z*11^DSTF+&o-CdRe6C`Kl)geH6svEik(sSkjI)YWSDm;U2keE#^k4oHeTC9g{zr|4 zGJyA;g@0dX!Qgk_TF}?V)r>;5T<@1HCutjaKZK%5NR>sfNH%=yg5 zYIV0HTS@+PUEI0Utw(v|rs5X+eV4&U>*e1*K?|(4SvhU&mrF-!=URicKEpDywvn#+ R9(o_VKLHW-PX$ma007O. + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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); } - -require_once(INSTALLDIR.'/lib/mailbox.php'); - -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; - } - - 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); - - if ($message->find()) { - return $message; - } else { - return NULL; - } - } - - function get_message_profile($message) { - return $message->getFrom(); - } - - function get_instructions() { - return _('This is your inbox, which lists your incoming private messages.'); - } +if (!defined('LACONICA')) { + exit(1); +} + +require_once INSTALLDIR.'/lib/mailbox.php'; + +/** + * action handler for message inbox + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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 + */ + +class InboxAction extends MailboxAction +{ + /** + * 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; + } + } + + /** + * 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() + */ + + function getMessageProfile($message) + { + return $message->getFrom(); + } + + /** + * instructions for using this page + * + * @return string localised instructions for using the page + */ + + function getInstructions() + { + return _('This is your inbox, which lists your incoming private messages.'); + } } diff --git a/_darcs/pristine/actions/outbox.php b/_darcs/pristine/actions/outbox.php index c48d9c2062..9fb6dbf9f8 100644 --- a/_darcs/pristine/actions/outbox.php +++ b/_darcs/pristine/actions/outbox.php @@ -1,9 +1,12 @@ . + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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); } - -require_once(INSTALLDIR.'/lib/mailbox.php'); - -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); - } - 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); - - if ($message->find()) { - return $message; - } else { - return NULL; - } - } - - function get_message_profile($message) { - return $message->getTo(); - } - - function get_instructions() { - return _('This is your outbox, which lists private messages you have sent.'); - } - +if (!defined('LACONICA')) { + exit(1); +} + +require_once INSTALLDIR.'/lib/mailbox.php'; + +/** + * action handler for message outbox + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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 + */ + +class OutboxAction extends MailboxAction +{ + /** + * 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(_("Outbox for %s - page %d"), $user->nickname, $page); + } else { + $title = sprintf(_("Outbox 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->from_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; + } + } + + /** + * 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() + */ + + function getMessageProfile($message) + { + return $message->getTo(); + } + + /** + * instructions for using this page + * + * @return string localised instructions for using the page + */ + + function getInstructions() + { + return _('This is your outbox, which lists private messages you have sent.'); + } } diff --git a/_darcs/pristine/lib/mailbox.php b/_darcs/pristine/lib/mailbox.php index 9039c5fdaa..bdc360a353 100644 --- a/_darcs/pristine/lib/mailbox.php +++ b/_darcs/pristine/lib/mailbox.php @@ -19,7 +19,7 @@ * You should have received a copy of the GNU Affero General Public License * along with this program. If not, see . * - * @category Action + * @category Message * @package Laconica * @author Evan Prodromou * @copyright 2008 Control Yourself, Inc. @@ -38,7 +38,7 @@ define('MESSAGES_PER_PAGE', 20); /** * common superclass for direct messages inbox and outbox * - * @category Action + * @category Message * @package Laconica * @author Evan Prodromou * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 @@ -91,7 +91,7 @@ class MailboxAction extends PersonalAction $page = 1; } - $this->show_page($user, $page); + $this->showPage($user, $page); } /** @@ -103,7 +103,7 @@ class MailboxAction extends PersonalAction * @return string localised title of the page */ - function get_title($user, $page) + function getTitle($user, $page) { return ''; } @@ -114,7 +114,7 @@ class MailboxAction extends PersonalAction * @return string localised instructions for using the page */ - function get_instructions() + function getInstructions() { return ''; } @@ -125,7 +125,7 @@ class MailboxAction extends PersonalAction * @return void */ - function show_top() + function showTop() { $cur = common_current_user(); @@ -143,17 +143,33 @@ class MailboxAction extends PersonalAction * @return void */ - function show_page($user, $page) + function showPage($user, $page) { - common_show_header($this->get_title($user, $page), + common_show_header($this->getTitle($user, $page), null, null, - array($this, 'show_top')); + array($this, 'showTop')); - $this->show_box($user, $page); + $this->showBox($user, $page); common_show_footer(); } + /** + * 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 + */ + + function getMessages($user, $page) + { + return null; + } + /** * show the messages for a mailbox in list format * @@ -165,9 +181,9 @@ class MailboxAction extends PersonalAction * @return void */ - function show_box($user, $page) + function showBox($user, $page) { - $message = $this->get_messages($user, $page); + $message = $this->getMessages($user, $page); if ($message) { @@ -181,7 +197,7 @@ class MailboxAction extends PersonalAction break; } - $this->show_message($message); + $this->showMessage($message); } common_element_end('ul'); @@ -205,7 +221,7 @@ class MailboxAction extends PersonalAction * @return Profile The profile that matches the message */ - function get_message_profile($message) + function getMessageProfile($message) { return null; } @@ -218,12 +234,12 @@ class MailboxAction extends PersonalAction * @return void */ - function show_message($message) + function showMessage($message) { common_element_start('li', array('class' => 'message_single', 'id' => 'message-' . $message->id)); - $profile = $this->get_message_profile($message); + $profile = $this->getMessageProfile($message); $avatar = $profile->getAvatar(AVATAR_STREAM_SIZE); common_element_start('a', array('href' => $profile->profileurl)); diff --git a/_darcs/tentative_pristine b/_darcs/tentative_pristine index dd4a35214c..5a45154e6d 100644 --- a/_darcs/tentative_pristine +++ b/_darcs/tentative_pristine @@ -1,393 +1,365 @@ -hunk ./lib/mailbox.php 2 +hunk ./actions/inbox.php 2 -/* - * Laconica - a distributed open-source microblogging tool - * Copyright (C) 2008, Controlez-Vous, Inc. +/** + * Laconica, the distributed open-source microblogging tool -hunk ./lib/mailbox.php 5 +hunk ./actions/inbox.php 5 - * This program is free software: you can redistribute it and/or modify -+ * common superclass for direct messages inbox and outbox ++ * action handler for message inbox + * + * PHP version 5 + * + * LICENCE: This program is free software: you can redistribute it and/or modify -hunk ./lib/mailbox.php 21 +hunk ./actions/inbox.php 21 + * -+ * @category Action ++ * @category Message + * @package Laconica + * @author Evan Prodromou + * @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/mailbox.php 30 --if (!defined('LACONICA')) { exit(1); } ++ */ ++ +if (!defined('LACONICA')) { + exit(1); +} -hunk ./lib/mailbox.php 34 --require_once(INSTALLDIR.'/lib/personal.php'); -+require_once INSTALLDIR.'/lib/personal.php'; -hunk ./lib/mailbox.php 38 --class MailboxAction extends PersonalAction { -- -- function handle($args) { ++ ++require_once INSTALLDIR.'/lib/mailbox.php'; ++ +/** -+ * common superclass for direct messages inbox and outbox ++ * action handler for message inbox + * -+ * @category Action ++ * @category Message + * @package Laconica + * @author Evan Prodromou + * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 + * @link http://laconi.ca/ -+ * @see InboxAction -+ * @see OutboxAction -+ */ -+ -+class MailboxAction extends PersonalAction ++ * @see MailboxAction +hunk ./actions/inbox.php 47 +-if (!defined('LACONICA')) { exit(1); } ++class InboxAction extends MailboxAction +{ + /** -+ * output page based on arguments -+ * -+ * @param array $args HTTP arguments (from $_REQUEST) -+ * -+ * @return void -+ */ -+ -+ function handle($args) -+ { -+ parent::handle($args); -+ -+ $nickname = common_canonical_nickname($this->arg('nickname')); -+ -+ $user = User::staticGet('nickname', $nickname); -+ -+ if (!$user) { -+ $this->client_error(_('No such user.'), 404); -+ return; -+ } -+ -+ $cur = common_current_user(); -+ -+ if (!$cur || $cur->id != $user->id) { -+ $this->client_error(_('Only the user can read their own mailboxes.'), -+ 403); -+ return; -+ } -+ -+ $profile = $user->getProfile(); -+ -+ if (!$profile) { -+ $this->server_error(_('User has no profile.')); -+ return; -+ } -+ -+ $page = $this->trimmed('page'); -+ -+ if (!$page) { -+ $page = 1; -+ } -+ -+ $this->show_page($user, $page); -+ } -+ -+ /** + * 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 get_title($user, $page) ++ function getTitle($user, $page) + { -+ return ''; ++ 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; ++ } ++ } ++ ++ /** ++ * 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; +- } +- +- 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 + */ -+ -+ function get_instructions() +hunk ./actions/inbox.php 122 +- if ($message->find()) { +- return $message; +- } else { +- return NULL; +- } +- } +- +- function get_message_profile($message) { +- return $message->getFrom(); +- } +- +- function get_instructions() { +- return _('This is your inbox, which lists your incoming private messages.'); +- } ++ function getInstructions() + { -+ return ''; ++ return _('This is your inbox, which lists your incoming private messages.'); + } +hunk ./actions/outbox.php 2 +-/* +- * Laconica - a distributed open-source microblogging tool +- * Copyright (C) 2008, Controlez-Vous, Inc. ++/** ++ * 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 ++ * ++ * PHP version 5 ++ * ++ * LICENCE: This program is free software: you can redistribute it and/or modify +hunk ./actions/outbox.php 21 ++ * ++ * @category Message ++ * @package Laconica ++ * @author Evan Prodromou ++ * @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); ++} ++ ++require_once INSTALLDIR.'/lib/mailbox.php'; ++ ++/** ++ * action handler for message outbox ++ * ++ * @category Message ++ * @package Laconica ++ * @author Evan Prodromou ++ * @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 ++{ + /** -+ * do structured output for the "instructions" are of the page ++ * returns the title of the page + * -+ * @return void ++ * @param User $user current user ++ * @param int $page current page ++ * ++ * @return string localised title of the page ++ * ++ * @see MailboxAction::getTitle() + */ + -+ function show_top() ++ function getTitle($user, $page) + { -+ $cur = common_current_user(); -+ -+ common_message_form(null, $cur, null); -+ -+ $this->views_menu(); ++ if ($page > 1) { ++ $title = sprintf(_("Outbox for %s - page %d"), $user->nickname, $page); ++ } else { ++ $title = sprintf(_("Outbox for %s"), $user->nickname); ++ } ++ return $title; + } + + /** -+ * show a full page of output ++ * 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 void ++ * @return Message data object with stream for messages ++ * ++ * @see MailboxAction::getMessages() + */ + -+ function show_page($user, $page) ++ function getMessages($user, $page) + { -+ common_show_header($this->get_title($user, $page), -+ null, null, -+ array($this, 'show_top')); ++ $message = new Message(); + -+ $this->show_box($user, $page); ++ $message->from_profile = $user->id; ++ $message->orderBy('created DESC, id DESC'); ++ $message->limit((($page-1)*MESSAGES_PER_PAGE), MESSAGES_PER_PAGE + 1); + -+ common_show_footer(); -+ } -+ -+ /** -+ * show the messages for a mailbox in list format -+ * -+ * Includes the pagination links (before, after). -+ * -+ * @param User $user The current user -+ * @param int $page The page the user is on -+ * -+ * @return void -+ */ -+ -+ function show_box($user, $page) -+ { -+ $message = $this->get_messages($user, $page); -+ -+ if ($message) { -+ -+ $cnt = 0; -+ common_element_start('ul', array('id' => 'messages')); -+ -+ while ($message->fetch() && $cnt <= MESSAGES_PER_PAGE) { -+ $cnt++; -hunk ./lib/mailbox.php 180 -- parent::handle($args); -+ if ($cnt > MESSAGES_PER_PAGE) { -+ break; -+ } -hunk ./lib/mailbox.php 184 -- $nickname = common_canonical_nickname($this->arg('nickname')); -- $user = User::staticGet('nickname', $nickname); -+ $this->show_message($message); -+ } -hunk ./lib/mailbox.php 187 -- if (!$user) { -- $this->client_error(_('No such user.'), 404); -- return; -- } -+ common_element_end('ul'); -hunk ./lib/mailbox.php 189 -- $cur = common_current_user(); -- -- if (!$cur || $cur->id != $user->id) { -- $this->client_error(_('Only the user can read their own mailboxes.'), 403); -- return; -- } -- -- $profile = $user->getProfile(); -+ common_pagination($page > 1, $cnt > MESSAGES_PER_PAGE, -+ $page, $this->trimmed('action'), -+ array('nickname' => $user->nickname)); -hunk ./lib/mailbox.php 193 -- if (!$profile) { -- $this->server_error(_('User has no profile.')); -- return; -- } -+ $message->free(); -+ unset($message); ++ if ($message->find()) { ++ return $message; ++ } else { ++ return null; + } + } -hunk ./lib/mailbox.php 198 -- $page = $this->trimmed('page'); -- -- if (!$page) { -- $page = 1; -- } -- -- $this->show_page($user, $page); -- } ++ + /** + * returns the profile we want to show with the message + * -+ * For inboxes, we show the sender; for outboxes, the recipient. ++ * For outboxes, we show the recipient. + * + * @param Message $message The message to get the profile for + * -+ * @return Profile The profile that matches the message ++ * @return Profile The profile of the message recipient ++ * ++ * @see MailboxAction::getMessageProfile() + */ -hunk ./lib/mailbox.php 208 +hunk ./actions/outbox.php 110 +-require_once(INSTALLDIR.'/lib/mailbox.php'); ++ function getMessageProfile($message) ++ { ++ return $message->getTo(); ++ } +hunk ./actions/outbox.php 115 +-class OutboxAction extends MailboxAction { +- - function get_title($user, $page) { -- return ''; +- if ($page > 1) { +- $title = sprintf(_("Outbox for %s - page %d"), $user->nickname, $page); +- } else { +- $title = sprintf(_("Outbox for %s"), $user->nickname); +- } +- return $title; - } -+ function get_message_profile($message) +- +- 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; +- } +- } +- +- function get_message_profile($message) { +- return $message->getTo(); +- } +- +- function get_instructions() { +- return _('This is your outbox, which lists private messages you have sent.'); +- } +- ++ function getInstructions() ++ { ++ return _('This is your outbox, which lists private messages you have sent.'); ++ } +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 ++ */ ++ ++ function getMessages($user, $page) + { + return null; + } -hunk ./lib/mailbox.php 213 -- function get_instructions() { -- return ''; -- } -+ /** -+ * show a single message in the list format -+ * -+ * @param Message $message the message to show -+ * -+ * @return void -+ */ -hunk ./lib/mailbox.php 221 -- function show_top() { -+ function show_message($message) -+ { -+ common_element_start('li', array('class' => 'message_single', -+ 'id' => 'message-' . $message->id)); -hunk ./lib/mailbox.php 226 -- $cur = common_current_user(); -- -- common_message_form(NULL, $cur, NULL); -- -- $this->views_menu(); -- } -- -- function show_page($user, $page) { -+ $profile = $this->get_message_profile($message); -hunk ./lib/mailbox.php 228 -- common_show_header($this->get_title($user, $page), -- NULL, NULL, -- array($this, 'show_top')); -- -- $this->show_box($user, $page); -- -- common_show_footer(); -- } -- -- function show_box($user, $page) { -- -- $message = $this->get_messages($user, $page); -- -- if ($message) { -- -- $cnt = 0; -- common_element_start('ul', array('id' => 'messages')); -- -- while ($message->fetch() && $cnt <= MESSAGES_PER_PAGE) { -- $cnt++; -- -- if ($cnt > MESSAGES_PER_PAGE) { -- break; -- } -- -- $this->show_message($message); -- } -+ $avatar = $profile->getAvatar(AVATAR_STREAM_SIZE); -+ common_element_start('a', array('href' => $profile->profileurl)); -+ common_element('img', array('src' => ($avatar) ? -+ common_avatar_display_url($avatar) : -+ common_default_avatar(AVATAR_STREAM_SIZE), -+ 'class' => 'avatar stream', -+ 'width' => AVATAR_STREAM_SIZE, -+ 'height' => AVATAR_STREAM_SIZE, -+ 'alt' => -+ ($profile->fullname) ? $profile->fullname : -+ $profile->nickname)); -+ common_element_end('a'); -+ common_element('a', array('href' => $profile->profileurl, -+ 'class' => 'nickname'), -+ $profile->nickname); -+ // FIXME: URL, image, video, audio -+ common_element_start('p', array('class' => 'content')); -+ common_raw($message->rendered); -+ common_element_end('p'); -hunk ./lib/mailbox.php 248 -- common_element_end('ul'); -- -- common_pagination($page > 1, $cnt > MESSAGES_PER_PAGE, -- $page, $this->trimmed('action'), -- array('nickname' => $user->nickname)); -- -- $message->free(); -- unset($message); -- } -- } -+ $messageurl = common_local_url('showmessage', -+ array('message' => $message->id)); -hunk ./lib/mailbox.php 251 -- # returns the profile we want to show with the message -- -- function get_message_profile($message) { -- return NULL; -- } -- -- function show_message($message) { -+ // XXX: we need to figure this out better. Is this right? -+ if (strcmp($message->uri, $messageurl) != 0 && -+ preg_match('/^http/', $message->uri)) { -+ $messageurl = $message->uri; -+ } -+ common_element_start('p', 'time'); -+ common_element('a', array('class' => 'permalink', -+ 'href' => $messageurl, -+ 'title' => common_exact_date($message->created)), -+ common_date_string($message->created)); -+ if ($message->source) { -+ common_text(_(' from ')); -+ $this->source_link($message->source); -+ } -hunk ./lib/mailbox.php 266 -- common_element_start('li', array('class' => 'message_single', -- 'id' => 'message-' . $message->id)); -+ common_element_end('p'); -hunk ./lib/mailbox.php 268 -- $profile = $this->get_message_profile($message); -- -- $avatar = $profile->getAvatar(AVATAR_STREAM_SIZE); -- common_element_start('a', array('href' => $profile->profileurl)); -- common_element('img', array('src' => ($avatar) ? common_avatar_display_url($avatar) : common_default_avatar(AVATAR_STREAM_SIZE), -- 'class' => 'avatar stream', -- 'width' => AVATAR_STREAM_SIZE, -- 'height' => AVATAR_STREAM_SIZE, -- 'alt' => -- ($profile->fullname) ? $profile->fullname : -- $profile->nickname)); -- common_element_end('a'); -- common_element('a', array('href' => $profile->profileurl, -- 'class' => 'nickname'), -- $profile->nickname); -- # FIXME: URL, image, video, audio -- common_element_start('p', array('class' => 'content')); -- common_raw($message->rendered); -- common_element_end('p'); -- -- $messageurl = common_local_url('showmessage', array('message' => $message->id)); -- -- # XXX: we need to figure this out better. Is this right? -- if (strcmp($message->uri, $messageurl) != 0 && preg_match('/^http/', $message->uri)) { -- $messageurl = $message->uri; -- } -- common_element_start('p', 'time'); -- common_element('a', array('class' => 'permalink', -- 'href' => $messageurl, -- 'title' => common_exact_date($message->created)), -- common_date_string($message->created)); -- if ($message->source) { -- common_text(_(' from ')); -- $this->source_link($message->source); -- } -- -- common_element_end('p'); -- -- common_element_end('li'); -- } -+ common_element_end('li'); -+ } ++ +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); diff --git a/actions/inbox.php b/actions/inbox.php index c752e404e9..da27814a6e 100644 --- a/actions/inbox.php +++ b/actions/inbox.php @@ -1,9 +1,12 @@ . + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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); } - -require_once(INSTALLDIR.'/lib/mailbox.php'); - -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; - } - - 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); - - if ($message->find()) { - return $message; - } else { - return NULL; - } - } - - function get_message_profile($message) { - return $message->getFrom(); - } - - function get_instructions() { - return _('This is your inbox, which lists your incoming private messages.'); - } +if (!defined('LACONICA')) { + exit(1); +} + +require_once INSTALLDIR.'/lib/mailbox.php'; + +/** + * action handler for message inbox + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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 + */ + +class InboxAction extends MailboxAction +{ + /** + * 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; + } + } + + /** + * 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() + */ + + function getMessageProfile($message) + { + return $message->getFrom(); + } + + /** + * instructions for using this page + * + * @return string localised instructions for using the page + */ + + function getInstructions() + { + return _('This is your inbox, which lists your incoming private messages.'); + } } diff --git a/actions/outbox.php b/actions/outbox.php index c48d9c2062..9fb6dbf9f8 100644 --- a/actions/outbox.php +++ b/actions/outbox.php @@ -1,9 +1,12 @@ . + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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); } - -require_once(INSTALLDIR.'/lib/mailbox.php'); - -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); - } - 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); - - if ($message->find()) { - return $message; - } else { - return NULL; - } - } - - function get_message_profile($message) { - return $message->getTo(); - } - - function get_instructions() { - return _('This is your outbox, which lists private messages you have sent.'); - } - +if (!defined('LACONICA')) { + exit(1); +} + +require_once INSTALLDIR.'/lib/mailbox.php'; + +/** + * action handler for message outbox + * + * @category Message + * @package Laconica + * @author Evan Prodromou + * @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 + */ + +class OutboxAction extends MailboxAction +{ + /** + * 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(_("Outbox for %s - page %d"), $user->nickname, $page); + } else { + $title = sprintf(_("Outbox 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->from_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; + } + } + + /** + * 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() + */ + + function getMessageProfile($message) + { + return $message->getTo(); + } + + /** + * instructions for using this page + * + * @return string localised instructions for using the page + */ + + function getInstructions() + { + return _('This is your outbox, which lists private messages you have sent.'); + } } diff --git a/lib/mailbox.php b/lib/mailbox.php index 9039c5fdaa..bdc360a353 100644 --- a/lib/mailbox.php +++ b/lib/mailbox.php @@ -19,7 +19,7 @@ * You should have received a copy of the GNU Affero General Public License * along with this program. If not, see . * - * @category Action + * @category Message * @package Laconica * @author Evan Prodromou * @copyright 2008 Control Yourself, Inc. @@ -38,7 +38,7 @@ define('MESSAGES_PER_PAGE', 20); /** * common superclass for direct messages inbox and outbox * - * @category Action + * @category Message * @package Laconica * @author Evan Prodromou * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 @@ -91,7 +91,7 @@ class MailboxAction extends PersonalAction $page = 1; } - $this->show_page($user, $page); + $this->showPage($user, $page); } /** @@ -103,7 +103,7 @@ class MailboxAction extends PersonalAction * @return string localised title of the page */ - function get_title($user, $page) + function getTitle($user, $page) { return ''; } @@ -114,7 +114,7 @@ class MailboxAction extends PersonalAction * @return string localised instructions for using the page */ - function get_instructions() + function getInstructions() { return ''; } @@ -125,7 +125,7 @@ class MailboxAction extends PersonalAction * @return void */ - function show_top() + function showTop() { $cur = common_current_user(); @@ -143,17 +143,33 @@ class MailboxAction extends PersonalAction * @return void */ - function show_page($user, $page) + function showPage($user, $page) { - common_show_header($this->get_title($user, $page), + common_show_header($this->getTitle($user, $page), null, null, - array($this, 'show_top')); + array($this, 'showTop')); - $this->show_box($user, $page); + $this->showBox($user, $page); common_show_footer(); } + /** + * 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 + */ + + function getMessages($user, $page) + { + return null; + } + /** * show the messages for a mailbox in list format * @@ -165,9 +181,9 @@ class MailboxAction extends PersonalAction * @return void */ - function show_box($user, $page) + function showBox($user, $page) { - $message = $this->get_messages($user, $page); + $message = $this->getMessages($user, $page); if ($message) { @@ -181,7 +197,7 @@ class MailboxAction extends PersonalAction break; } - $this->show_message($message); + $this->showMessage($message); } common_element_end('ul'); @@ -205,7 +221,7 @@ class MailboxAction extends PersonalAction * @return Profile The profile that matches the message */ - function get_message_profile($message) + function getMessageProfile($message) { return null; } @@ -218,12 +234,12 @@ class MailboxAction extends PersonalAction * @return void */ - function show_message($message) + function showMessage($message) { common_element_start('li', array('class' => 'message_single', 'id' => 'message-' . $message->id)); - $profile = $this->get_message_profile($message); + $profile = $this->getMessageProfile($message); $avatar = $profile->getAvatar(AVATAR_STREAM_SIZE); common_element_start('a', array('href' => $profile->profileurl));