X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FModule%2FAttach.php;h=b70e4342976e52259f2de84dd510eb04df581212;hb=e1863951986ba5be173758324a00652bc5af870c;hp=519de0cc9c44126bbe8dc1171c0e4d3528a6ee9c;hpb=8293d5ed0af78ae6cc5629bb1f982a7b29794d32;p=friendica.git diff --git a/src/Module/Attach.php b/src/Module/Attach.php index 519de0cc9c..b70e434297 100644 --- a/src/Module/Attach.php +++ b/src/Module/Attach.php @@ -1,6 +1,6 @@ argc != 2) { + if (empty($this->parameters['item'])) { throw new \Friendica\Network\HTTPException\BadRequestException(); } - // @TODO: Replace with parameter from router - $item_id = intval($a->argv[1]); - + $item_id = intval($this->parameters['item']); + // Check for existence $item = MAttach::exists(['id' => $item_id]); if ($item === false) { throw new \Friendica\Network\HTTPException\NotFoundException(DI::l10n()->t('Item was not found.')); } - // Now we'll fetch the item, if we have enough permisson + // Now we'll fetch the item, if we have enough permission $item = MAttach::getByIdWithPermission($item_id); if ($item === false) { throw new \Friendica\Network\HTTPException\ForbiddenException(DI::l10n()->t('Permission denied.')); @@ -58,7 +57,7 @@ class Attach extends BaseModule $data = MAttach::getData($item); if (is_null($data)) { - Logger::log('NULL data for attachment with id ' . $item['id']); + Logger::notice('NULL data for attachment with id ' . $item['id']); throw new \Friendica\Network\HTTPException\NotFoundException(DI::l10n()->t('Item was not found.')); } @@ -73,7 +72,7 @@ class Attach extends BaseModule } echo $data; - exit(); + System::exit(); // NOTREACHED } }