AttachmentAction is a ManagedAction, don't implement handle()

This also gives us more freedom in Attachment_thumbnailAction for example

actions/attachment.php

@@ -92,21 +92,14 @@ class AttachmentAction extends ManagedAction
         return $a->title();
     }
 
-    /**
+    public function showPage()
-     * Handle input
-     *
-     * Only handles get, so just show the page.
-     *
-     * @return void
-     */
-    protected function handle()
     {
         if (empty($this->attachment->filename)) {
             // if it's not a local file, gtfo
             common_redirect($this->attachment->url, 303);
         }
 
-        parent::handle();
+        parent::showPage();
     }
 
     /**