Merge pull request #9397 from vinzv/9238-red-color-unread-messages-faded

[friendica.git] / src / Module / Attach.php

diff --git a/src/Module/Attach.php b/src/Module/Attach.php
index 24e0edc5552ad774d7187573249835acd31474b7..519de0cc9c44126bbe8dc1171c0e4d3528a6ee9c 100644 (file)
--- a/src/Module/Attach.php
+++ b/src/Module/Attach.php
@@ -1,30 +1,44 @@
 <?php
 /**
- * @file src/Module/Attach.php
+ * @copyright Copyright (C) 2020, Friendica
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * 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 the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
  */
 
-
 namespace Friendica\Module;
 
 use Friendica\BaseModule;
-use Friendica\Core\L10n;
 use Friendica\Core\Logger;
-use Friendica\Core\System;
+use Friendica\DI;
 use Friendica\Model\Attach as MAttach;
 
 /**
- * @brief Attach Module
+ * Attach Module
  */
 class Attach extends BaseModule
 {
        /**
-        * @brief Return to user an attached file given the id
+        * Return to user an attached file given the id
         */
-       public static function rawContent()
+       public static function rawContent(array $parameters = [])
        {
-               $a = self::getApp();
+               $a = DI::app();
                if ($a->argc != 2) {
-                       System::httpExit(400); // Bad Request.
+                       throw new \Friendica\Network\HTTPException\BadRequestException();
                }
 
                // @TODO: Replace with parameter from router
@@ -33,19 +47,19 @@ class Attach extends BaseModule
                // Check for existence
                $item = MAttach::exists(['id' => $item_id]);
                if ($item === false) {
-                       System::httpExit(404, ['description' => L10n::t('Item was not found.')]);
+                       throw new \Friendica\Network\HTTPException\NotFoundException(DI::l10n()->t('Item was not found.'));
                }
 
                // Now we'll fetch the item, if we have enough permisson
                $item = MAttach::getByIdWithPermission($item_id);
                if ($item === false) {
-                       System::httpExit(403, ['description' => L10n::t('Permission denied.')]);
+                       throw new \Friendica\Network\HTTPException\ForbiddenException(DI::l10n()->t('Permission denied.'));
                }
 
                $data = MAttach::getData($item);
                if (is_null($data)) {
                        Logger::log('NULL data for attachment with id ' . $item['id']);
-                       System::httpExit(404, ['description' => L10n::t('Item was not found.')]);
+                       throw new \Friendica\Network\HTTPException\NotFoundException(DI::l10n()->t('Item was not found.'));
                }
 
                // Use quotes around the filename to prevent a "multiple Content-Disposition"