keith/friendica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Remove obsolete Factory\Api\Mastodon\Error->logError method and related dependencies

Browse source
This commit is contained in:
Hypolite Petovan 2023-10-11 09:33:53 -04:00
parent 0a91484fa0
commit eb583330df
2 changed files with 1 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/Factory/Api/Mastodon/Error.php
View file

@ -21,35 +21,22 @@
namespace Friendica\Factory\Api\Mastodon; namespace Friendica\Factory\Api\Mastodon;
use Friendica\App\Arguments;
use Friendica\BaseFactory; use Friendica\BaseFactory;
use Friendica\Core\L10n; use Friendica\Core\L10n;
use Friendica\Core\System;
use Psr\Log\LoggerInterface; use Psr\Log\LoggerInterface;
/** @todo A Factory shouldn't return something to the frontpage, it's for creating content, not showing it */ /** @todo A Factory shouldn't return something to the frontpage, it's for creating content, not showing it */
class Error extends BaseFactory class Error extends BaseFactory
{ {
/** @var Arguments */
private $args;
/** @var string[] The $_SERVER array */
private $server;
/** @var L10n */ /** @var L10n */
private $l10n; private $l10n;
public function __construct(LoggerInterface $logger, Arguments $args, L10n $l10n, array $server) public function __construct(LoggerInterface $logger, L10n $l10n)
{ {
parent::__construct($logger); parent::__construct($logger);
$this->args = $args;
$this->server = $server;
$this->l10n = $l10n; $this->l10n = $l10n;
} }
private function logError(int $errorno, string $error)
{
$this->logger->info('API Error', ['no' => $errorno, 'error' => $error, 'method' => $this->args->getMethod(), 'command' => $this->args->getQueryString(), 'user-agent' => $this->server['HTTP_USER_AGENT'] ?? '']);
}
public function RecordNotFound(): \Friendica\Object\Api\Mastodon\Error public function RecordNotFound(): \Friendica\Object\Api\Mastodon\Error
{ {
$error = $this->l10n->t('Record not found'); $error = $this->l10n->t('Record not found');

5
static/dependencies.config.php
View file

@ -302,11 +302,6 @@ return [
['createClient', [], Dice::CHAIN_CALL], ['createClient', [], Dice::CHAIN_CALL],
], ],
], ],
Factory\Api\Mastodon\Error::class => [
'constructParams' => [
$_SERVER
],
],
ParsedLogIterator::class => [ ParsedLogIterator::class => [
'constructParams' => [ 'constructParams' => [
[Dice::INSTANCE => Util\ReversedFileReader::class], [Dice::INSTANCE => Util\ReversedFileReader::class],