From c0f23a2703edbc64486a00ea72b87f5be62cc445 Mon Sep 17 00:00:00 2001 From: Evan Prodromou Date: Sun, 18 Jan 2009 18:35:11 +0000 Subject: [PATCH] Update shownotice for new framework, make phpcs-clean --- actions/shownotice.php | 185 ++++++++++++++++++++++++++++++++--------- 1 file changed, 145 insertions(+), 40 deletions(-) diff --git a/actions/shownotice.php b/actions/shownotice.php index 82d4bd2701..52faba93ef 100644 --- a/actions/shownotice.php +++ b/actions/shownotice.php @@ -1,9 +1,12 @@ . + * + * @category Personal + * @package Laconica + * @author Evan Prodromou + * @copyright 2008-2009 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(INSTALLDIR.'/lib/stream.php'); +require_once INSTALLDIR.'/lib/personalgroupnav.php'; +require_once INSTALLDIR.'/lib/noticelist.php'; +require_once INSTALLDIR.'/lib/feedlist.php'; -class ShownoticeAction extends StreamAction +/** + * Show a single notice + * + * @category Personal + * @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/ + */ + +class ShownoticeAction extends Action { + /** + * Notice object to show + */ var $notice = null; + + /** + * Profile of the notice object + */ + var $profile = null; + + /** + * Avatar of the profile of the notice object + */ + var $avatar = null; + /** + * Load attributes based on database arguments + * + * Loads all the DB stuff + * + * @param array $args $_REQUEST array + * + * @return success flag + */ + function prepare($args) { - parent::prepare($args); $id = $this->arg('notice'); + $this->notice = Notice::staticGet($id); if (!$this->notice) { @@ -53,45 +100,112 @@ class ShownoticeAction extends StreamAction return true; } - function last_modified() + /** + * Is this action read-only? + * + * @return boolean true + */ + + function isReadOnly() + { + return true; + } + + /** + * Last-modified date for page + * + * When was the content of this page last modified? Based on notice, + * profile, avatar. + * + * @return int last-modified date as unix timestamp + */ + + function lastModified() { return max(strtotime($this->notice->created), strtotime($this->profile->modified), ($this->avatar) ? strtotime($this->avatar->modified) : 0); } + /** + * An entity tag for this page + * + * Shows the ETag for the page, based on the notice ID and timestamps + * for the notice, profile, and avatar. It's weak, since we change + * the date text "one hour ago", etc. + * + * @return string etag + */ + function etag() { + $avtime = ($this->avatar) ? + strtotime($this->avatar->modified) : 0; + return 'W/"' . implode(':', array($this->arg('action'), common_language(), $this->notice->id, strtotime($this->notice->created), strtotime($this->profile->modified), - ($this->avatar) ? strtotime($this->avatar->modified) : 0)) . '"'; + $avtime)) . '"'; } + /** + * Title of the page + * + * @return string title of the page + */ + + function title() + { + return sprintf(_('%1$s\'s status on %2$s'), + $this->profile->nickname, + common_exact_date($this->notice->created)); + } + + /** + * Handle input + * + * Only handles get, so just show the page. + * + * @param array $args $_REQUEST data (unused) + * + * @return void + */ + function handle($args) { - parent::handle($args); - common_show_header(sprintf(_('%1$s\'s status on %2$s'), - $this->profile->nickname, - common_exact_date($this->notice->created)), - array($this, 'show_header'), null, - array($this, 'show_top')); - - $this->elementStart('ul', array('id' => 'notices')); - $nli = new NoticeListItem($this->notice); - $nli->show(); - $this->elementEnd('ul'); - - common_show_footer(); + $this->showPage(); } - function show_header() - { + /** + * Fill the content area of the page + * + * Shows a single notice list item. + * + * @return void + */ + function showContent() + { + $this->elementStart('ul', array('id' => 'notices')); + $nli = new NoticeListItem($this->notice, $this); + $nli->show(); + $this->elementEnd('ul'); + } + + /** + * Extra content + * + * We show the microid(s) for the author, if any. + * + * @return void + */ + + function extraHead() + { $user = User::staticGet($this->profile->id); if (!$user) { @@ -99,26 +213,17 @@ class ShownoticeAction extends StreamAction } if ($user->emailmicroid && $user->email && $this->notice->uri) { + $id = new Microid('mailto:'. $user->email, + $this->notice->uri); $this->element('meta', array('name' => 'microid', - 'content' => "mailto+http:sha1:" . sha1(sha1('mailto:' . $user->email) . sha1($this->notice->uri)))); + 'content' => $id->toString())); } if ($user->jabbermicroid && $user->jabber && $this->notice->uri) { + $id = new Microid('xmpp:', $user->jabber, + $this->notice->uri); $this->element('meta', array('name' => 'microid', - 'content' => "xmpp+http:sha1:" . sha1(sha1('xmpp:' . $user->jabber) . sha1($this->notice->uri)))); + 'content' => $id->toString())); } } - - function show_top() - { - $cur = common_current_user(); - if ($cur && $cur->id == $this->profile->id) { - common_notice_form(); - } - } - - function no_such_notice() - { - $this->clientError(_('No such notice.')); - } }