X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=actions%2Fshownotice.php;h=28cb68c121f224c9b1b93f4c3f5eb09ba1d212bd;hb=a7a4eeef09ee8929bb9ca563eb2bc52d198d202f;hp=ea9041efb50bd6bf2e52218c44c861cfd20dd667;hpb=31b29fde50e9664e1b70064c043879ce87553883;p=quix0rs-gnu-social.git diff --git a/actions/shownotice.php b/actions/shownotice.php index ea9041efb5..28cb68c121 100644 --- a/actions/shownotice.php +++ b/actions/shownotice.php @@ -22,18 +22,14 @@ * @category Personal * @package StatusNet * @author Evan Prodromou - * @copyright 2008-2009 StatusNet, Inc. + * @copyright 2008-2011 StatusNet, Inc. * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 * @link http://status.net/ */ -if (!defined('STATUSNET') && !defined('LACONICA')) { - exit(1); -} +if (!defined('GNUSOCIAL')) { exit(1); } -require_once INSTALLDIR.'/lib/personalgroupnav.php'; require_once INSTALLDIR.'/lib/noticelist.php'; -require_once INSTALLDIR.'/lib/feedlist.php'; /** * Show a single notice @@ -44,7 +40,7 @@ require_once INSTALLDIR.'/lib/feedlist.php'; * @license http://www.fsf.org/licensing/licenses/agpl-3.0.html GNU Affero General Public License version 3.0 * @link http://status.net/ */ -class ShownoticeAction extends OwnerDesignAction +class ShownoticeAction extends Action { /** * Notice object to show @@ -70,7 +66,7 @@ class ShownoticeAction extends OwnerDesignAction * * @return success flag */ - function prepare($args) + protected function prepare(array $args=array()) { parent::prepare($args); if ($this->boolean('ajax')) { @@ -79,30 +75,30 @@ class ShownoticeAction extends OwnerDesignAction $this->notice = $this->getNotice(); - $cur = common_current_user(); - - if (!empty($cur)) { - $curProfile = $cur->getProfile(); - } else { - $curProfile = null; - } - - if (!$this->notice->inScope($curProfile)) { + if (!$this->notice->inScope($this->scoped)) { // TRANS: Client exception thrown when trying a view a notice the user has no access to. throw new ClientException(_('Not available.'), 403); } $this->profile = $this->notice->getProfile(); - if (empty($this->profile)) { + if (!$this->profile instanceof Profile) { // TRANS: Server error displayed trying to show a notice without a connected profile. $this->serverError(_('Notice has no profile.'), 500); - return false; } - $this->user = User::staticGet('id', $this->profile->id); + try { + $this->user = $this->profile->getUser(); + } catch (NoSuchUserException $e) { + // FIXME: deprecate $this->user stuff in extended classes + $this->user = null; + } - $this->avatar = $this->profile->getAvatar(AVATAR_PROFILE_SIZE); + try { + $this->avatar = $this->profile->getAvatar(AVATAR_PROFILE_SIZE); + } catch (Exception $e) { + $this->avatar = null; + } return true; } @@ -113,16 +109,16 @@ class ShownoticeAction extends OwnerDesignAction * * @return Notice */ - function getNotice() + protected function getNotice() { $id = $this->arg('notice'); - $notice = Notice::staticGet('id', $id); + $notice = Notice::getKV('id', $id); - if (empty($notice)) { + if (!$notice instanceof Notice) { // Did we used to have it, and it got deleted? - $deleted = Deleted_notice::staticGet($id); - if (!empty($deleted)) { + $deleted = Deleted_notice::getKV($id); + if ($deleted instanceof Deleted_notice) { // TRANS: Client error displayed trying to show a deleted notice. $this->clientError(_('Notice deleted.'), 410); } else { @@ -207,28 +203,13 @@ class ShownoticeAction extends OwnerDesignAction * * @return void */ - function handle($args) + protected function handle() { - parent::handle($args); + parent::handle(); - if ($this->boolean('ajax')) { + if (StatusNet::isAjax()) { $this->showAjax(); } else { - if ($this->notice->is_local == Notice::REMOTE_OMB) { - if (!empty($this->notice->url)) { - $target = $this->notice->url; - } else if (!empty($this->notice->uri) && preg_match('/^https?:/', $this->notice->uri)) { - // Old OMB posts saved the remote URL only into the URI field. - $target = $this->notice->uri; - } else { - // Shouldn't happen. - $target = false; - } - if ($target && $target != $this->selfUrl()) { - common_redirect($target, 301); - return false; - } - } $this->showPage(); } } @@ -250,9 +231,7 @@ class ShownoticeAction extends OwnerDesignAction function showAjax() { - header('Content-Type: text/xml;charset=utf-8'); - $this->xw->startDocument('1.0', 'UTF-8'); - $this->elementStart('html'); + $this->startHTML('text/xml;charset=utf-8'); $this->elementStart('head'); // TRANS: Title for page that shows a notice. $this->element('title', null, _m('TITLE','Notice')); @@ -261,7 +240,7 @@ class ShownoticeAction extends OwnerDesignAction $nli = new NoticeListItem($this->notice, $this); $nli->show(); $this->elementEnd('body'); - $this->elementEnd('html'); + $this->endHTML(); } /** @@ -290,7 +269,7 @@ class ShownoticeAction extends OwnerDesignAction */ function extraHead() { - $user = User::staticGet($this->profile->id); + $user = User::getKV($this->profile->id); if (!$user) { return; @@ -303,26 +282,8 @@ class ShownoticeAction extends OwnerDesignAction 'content' => $id->toString())); } - $this->element('link',array('rel'=>'alternate', - 'type'=>'application/json+oembed', - 'href'=>common_local_url( - 'oembed', - array(), - array('format'=>'json','url'=>$this->notice->uri)), - 'title'=>'oEmbed'),null); - $this->element('link',array('rel'=>'alternate', - 'type'=>'text/xml+oembed', - 'href'=>common_local_url( - 'oembed', - array(), - array('format'=>'xml','url'=>$this->notice->uri)), - 'title'=>'oEmbed'),null); - // Extras to aid in sharing notices to Facebook - $avatar = $this->profile->getAvatar(AVATAR_PROFILE_SIZE); - $avatarUrl = ($avatar) ? - $avatar->displayUrl() : - Avatar::defaultImage(AVATAR_PROFILE_SIZE); + $avatarUrl = $this->profile->avatarUrl(AVATAR_PROFILE_SIZE); $this->element('meta', array('property' => 'og:image', 'content' => $avatarUrl)); $this->element('meta', array('property' => 'og:description',