Consolidate common code in micro-apps custom notice type display actions.

[quix0rs-gnu-social.git] / actions / file.php

diff --git a/actions/file.php b/actions/file.php
index 271f57ab9634aeaa4a6d675d30ac46a2849f30c8..49ed8af1d6eff55f89ef6ef77dcb33952dd2a5a2 100644 (file)
--- a/actions/file.php
+++ b/actions/file.php
@@ -1,7 +1,7 @@
 <?php
 /*
- * Laconica - a distributed open-source microblogging tool
- * Copyright (C) 2008, 2009, Control Yourself, Inc.
+ * StatusNet - the distributed open-source microblogging tool
+ * Copyright (C) 2008, 2009, StatusNet, Inc.
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU Affero General Public License as published by
@@ -17,10 +17,11 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-if (!defined('LACONICA')) { exit(1); }
+if (!defined('STATUSNET') && !defined('LACONICA')) { exit(1); }
 
 require_once(INSTALLDIR.'/actions/shownotice.php');
 
+// @todo FIXME: Add documentation.
 class FileAction extends Action
 {
     var $id = null;
@@ -31,15 +32,18 @@ class FileAction extends Action
         parent::prepare($args);
         $this->id = $this->trimmed('notice');
         if (empty($this->id)) {
-            $this->clientError(_('No notice id'));
+            // TRANS: Client error displayed when no notice ID was given trying do display a file.
+            $this->clientError(_('No notice ID.'));
         }
         $notice = Notice::staticGet('id', $this->id);
         if (empty($notice)) {
-            $this->clientError(_('No notice'));
+            // TRANS: Client error displayed when an invalid notice ID was given trying do display a file.
+            $this->clientError(_('No notice.'));
         }
         $atts = $notice->attachments();
         if (empty($atts)) {
-            $this->clientError(_('No attachments'));
+            // TRANS: Client error displayed when trying do display a file for a notice without a file attachement.
+            $this->clientError(_('No attachments.'));
         }
         foreach ($atts as $att) {
             if (!empty($att->filename)) {
@@ -48,7 +52,10 @@ class FileAction extends Action
             }
         }
         if (empty($this->filerec)) {
-            $this->clientError(_('No uploaded attachments'));
+            // XXX: Is this translation hint correct? If yes, please remove comment, if no, please correct and remove comment.
+            // TRANS: Client error displayed when trying do display a file for a notice with file attachements
+            // TRANS: that could not be found.
+            $this->clientError(_('No uploaded attachments.'));
         }
         return true;
     }
@@ -56,5 +63,14 @@ class FileAction extends Action
     function handle() {
         common_redirect($this->filerec->url);
     }
-}
 
+    /**
+     * Is this action read-only?
+     *
+     * @return boolean true
+     */
+    function isReadOnly($args)
+    {
+        return true;
+    }
+}