Standards

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

diff --git a/src/Module/Attach.php b/src/Module/Attach.php
index 2acbf31b425ed7ded08fc89e637519c6fab016a7..bd785881c3d067a9a36cdfb0a17bf2ae7b84740c 100644 (file)
--- a/src/Module/Attach.php
+++ b/src/Module/Attach.php
@@ -34,15 +34,14 @@ class Attach extends BaseModule
        /**
         * Return to user an attached file given the id
         */
-       public static function rawContent(array $parameters = [])
+       protected function rawContent(array $request = [])
        {
                $a = DI::app();
-               if ($a->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]);
@@ -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.'));
                }