From 4b65275e41453e71865a5ec7ca12d23d09bffd7f Mon Sep 17 00:00:00 2001 From: Mikael Nordfeldth Date: Sun, 25 Jan 2015 02:25:28 +0100 Subject: [PATCH] AttachmentAction is a ManagedAction, don't implement handle() This also gives us more freedom in Attachment_thumbnailAction for example --- actions/attachment.php | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/actions/attachment.php b/actions/attachment.php index 94ccf098e0..1126759832 100644 --- a/actions/attachment.php +++ b/actions/attachment.php @@ -92,21 +92,14 @@ class AttachmentAction extends ManagedAction return $a->title(); } - /** - * Handle input - * - * Only handles get, so just show the page. - * - * @return void - */ - protected function handle() + public function showPage() { if (empty($this->attachment->filename)) { // if it's not a local file, gtfo common_redirect($this->attachment->url, 303); } - parent::handle(); + parent::showPage(); } /**