X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FModule%2FAttach.php;h=519de0cc9c44126bbe8dc1171c0e4d3528a6ee9c;hb=213716d44ce64545972ebfc064758c065a20684a;hp=24e0edc5552ad774d7187573249835acd31474b7;hpb=b6b9e57488f77dc048887128adf3dc6d0c8892d4;p=friendica.git diff --git a/src/Module/Attach.php b/src/Module/Attach.php index 24e0edc555..519de0cc9c 100644 --- a/src/Module/Attach.php +++ b/src/Module/Attach.php @@ -1,30 +1,44 @@ . + * */ - 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"