1
0
mirror of https://github.com/vichan-devel/vichan.git synced 2025-02-26 06:58:26 +01:00

Context: extract dependency building from the container

This commit is contained in:
Zankaria 2024-04-09 11:20:04 +02:00
parent 3c49645af0
commit f93dd1fae5
2 changed files with 29 additions and 15 deletions

View File

@ -6,18 +6,20 @@ use Vichan\Driver\{HttpDriver, HttpDrivers, Log, LogDrivers};
defined('TINYBOARD') or exit; defined('TINYBOARD') or exit;
interface Context { interface DependencyFactory {
public function getLog(): Log; public function buildLogDriver(): Log;
public function getHttpDriver(): HttpDriver; public function buildHttpDriver(): HttpDriver;
} }
class AppContext implements Context { class WebDependencyFactory implements DependencyFactory {
private array $config; private array $config;
private ?Log $log;
private ?HttpDriver $http;
private function initLogDriver(): Log { public function __construct(array $config) {
$this->config = $config;
}
public function buildLogDriver(): Log {
$name = $this->config['log_system']['name']; $name = $this->config['log_system']['name'];
$level = $this->config['debug'] ? Log::DEBUG : Log::NOTICE; $level = $this->config['debug'] ? Log::DEBUG : Log::NOTICE;
$backend = $this->config['log_system']['type']; $backend = $this->config['log_system']['type'];
@ -36,21 +38,34 @@ class AppContext implements Context {
} }
} }
public function buildHttpDriver(): HttpDriver {
return HttpDrivers::getHttpDriver(
$this->config['upload_by_url_timeout'],
$this->config['max_filesize']
);
}
}
public function __construct(array $config) { class Context {
$this->config = $config; private DependencyFactory $factory;
private ?Log $log;
private ?HttpDriver $http;
public function __construct(DependencyFactory $factory) {
$this->factory = $factory;
} }
public function getLog(): Log { public function getLog(): Log {
if (is_null($this->log)) { if (is_null($this->log)) {
$this->log = $this->initLogDriver(); $this->log = $this->factory->buildLogDriver();
} }
return $this->log; return $this->log;
} }
public function getHttpDriver(): HttpDriver { public function getHttpDriver(): HttpDriver {
if (is_null($this->http)) { if (is_null($this->http)) {
$this->http = HttpDrivers::getHttpDriver($this->config['upload_by_url_timeout'], $this->config['max_filesize']); $this->http = $this->factory->buildHttpDriver();
} }
return $this->http; return $this->http;
} }

View File

@ -5,9 +5,8 @@
require_once 'inc/bootstrap.php'; require_once 'inc/bootstrap.php';
use Vichan\AppContext; use Vichan\{Context, WebDependencyFactory};
use Vichan\Driver\HttpDriver; use Vichan\Driver\{HttpDriver, Log};
use Vichan\Driver\Log;
use Vichan\Service\{RemoteCaptchaQuery, NativeCaptchaQuery}; use Vichan\Service\{RemoteCaptchaQuery, NativeCaptchaQuery};
/** /**
@ -172,7 +171,7 @@ function strip_image_metadata(string $img_path): int {
*/ */
$dropped_post = false; $dropped_post = false;
$context = new AppContext($config); $context = new Context(new WebDependencyFactory($config));
// Is it a post coming from NNTP? Let's extract it and pretend it's a normal post. // Is it a post coming from NNTP? Let's extract it and pretend it's a normal post.
if (isset($_GET['Newsgroups']) && $config['nntpchan']['enabled']) { if (isset($_GET['Newsgroups']) && $config['nntpchan']['enabled']) {