Use rawContent for Special Options to avoid a protected options() method

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

diff --git a/src/Module/Attach.php b/src/Module/Attach.php
index 519de0cc9c44126bbe8dc1171c0e4d3528a6ee9c..60d1015b0aeb68295b05c2f9830973af958157bd 100644 (file)
--- a/src/Module/Attach.php
+++ b/src/Module/Attach.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -34,16 +34,15 @@ 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]);
                if ($item === false) {
@@ -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.'));
                }