X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FModule%2FAttach.php;h=7d877c12655b0a11efa78413a0debaadc1e8d93c;hb=3bca4fe2a64671d09e08346456cdfa6c12f996e9;hp=519de0cc9c44126bbe8dc1171c0e4d3528a6ee9c;hpb=2c56d2f3360c08e312e5c167261af8e5d4b87af4;p=friendica.git diff --git a/src/Module/Attach.php b/src/Module/Attach.php index 519de0cc9c..7d877c1265 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 } }